使用 Rspec 测试 DELETE 请求的最佳方法？

Question

我开始使用这种方式：

  describe "DELETE /v1/categories/{id}" do
   before(:each) do
     #   Login User/Token
   end
   it 'deletes a category' do
     category = Fabricate(:category)
     category2 = Fabricate(:category)

     get "/v1/categories"
     expect(response.status).to eq 200
     expect(JSON.parse(response.body)).to eq([YAML.load(category.to_json),YAML.load(category2.to_json),])

     delete "/v1/categories/#{category.id}"
     expect(response.status).to eq 200

     get "/v1/categories"
     expect(JSON.parse(response.body)).to eq([YAML.load(category2.to_json)])
   end
 end

我不确定是否是测试删除数据的 API 请求的最佳方法。

Answer 1

到目前为止，您的测试确保了这一点：

• 删除前获取请求的响应
• get 请求的状态码
• 删除请求的响应
• 删除请求的状态码
• 删除后get请求的响应
• get 请求的状态码

这个测试覆盖的内容比删除请求要多，但我认为这很好。有这种测试总比没有测试好。

为了改进这个测试，我会做的是在测试时拆分路线。 我将进行 1 次测试以确保索引路由按预期工作并且 1 次测试以确保删除路由正常工作。这样索引路由上的错误不会破坏您的删除规范。 =)

我会有这样的东西：

describe "GET /v1/categories" do
    before(:each) do
        #   Login User/Token
        category = Fabricate(:category)
        category2 = Fabricate(:category)
        get "/v1/categories"
    end

    it 'should return status 200' do
        expect(response.status).to eq 200
    end

    it 'list all categories' do
        expect(JSON.parse(response.body)).to eq([YAML.load(category.to_json),YAML.load(category2.to_json),])
    end
end

describe "DELETE /v1/categories/:category_id" do
    before(:each) do
        #   Login User/Token
        category = Fabricate(:category)
        category2 = Fabricate(:category)
        delete "/v1/categories/#{category.id}"
    end

    it 'should return status 200' do
        expect(response.status).to eq 200
    end

    it 'should delete the category' do
        expect(Category.all).to eq category2
    end
end