Как высушить эту спецификацию контроллера RSpec?

Question

Как я могу сушить эту спецификацию?

describe Api::TasksController, type: :controller do 
    it 'allows the creator of a task to destroy it' do 
    set_request_auth_header @user 
    delete :destroy, id: @task.id 
    expect(response).to be_success 
    expect(Task.count).to eq 0 
    end 

    it 'does not allow the assignee of a task to destroy it' do 
    set_request_auth_header @assignee 
    delete :destroy, id: @task.id 
    expect(response).to be_forbidden 
    expect(Task.count).to eq 1 
    end 

    it 'does not allow anyone unrelated to a task to destroy it' do 
    set_request_auth_header @spy 
    delete :destroy, id: @task.id 
    expect(response).to be_forbidden 
    expect(Task.count).to eq 1 
    end 
end 

Answer 1

Вы можете извлечь некоторые методы:

describe Api::TasksController, type: :controller do 
    it 'allows the creator of a task to destroy it' do 
    expect_delete_to_succeed @user 
    end 

    it 'does not allow the assignee of a task to destroy it' do 
    expect_delete_to_be_forbidden @assignee 
    end 

    it 'does not allow anyone unrelated to a task to destroy it' do 
    expect_delete_to_be_forbidden @spy 
    end 

    def expect_delete_to_succeed(requester) 
    delete_task requester 
    expect(response).to be_success 
    expect(Task.count).to eq 0 
    end 

    def expect_delete_to_be_forbidden(requester) 
    delete_task requester 
    expect(response).to be_forbidden 
    expect(Task.count).to eq 1 
    end 

    def delete_task(requester) 
    set_request_auth_header requester 
    delete :destroy, id: @task.id 
    end 

end 

Боковые ноты:

• Использование экземпляра (@) переменные не является текущей стандартной практикой в ​​RSpec. Посмотрите на let 'variables'.
• Task.exists? @task.id кажется более четким способом проверки удаления, чем проверка Task.count.

Answer 2

у вас нет. вуаля, пожалуйста.