Commit ce9e35c295d4df4b6a093ffef7ce0ad3f5e1a24e
1 parent
4a60c377
Exists in
master
and in
4 other branches
API: Fixes tests for changed hook deletion
Showing
1 changed file
with
2 additions
and
7 deletions
Show diff stats
spec/requests/api/projects_spec.rb
| @@ -433,19 +433,14 @@ describe Gitlab::API do | @@ -433,19 +433,14 @@ describe Gitlab::API do | ||
| 433 | end | 433 | end |
| 434 | 434 | ||
| 435 | it "should return success when deleting hook" do | 435 | it "should return success when deleting hook" do |
| 436 | - delete api("/projects/#{project.id}/hooks", user), hook_id: hook.id | 436 | + delete api("/projects/#{project.id}/hooks/#{hook.id}", user) |
| 437 | response.status.should == 200 | 437 | response.status.should == 200 |
| 438 | end | 438 | end |
| 439 | 439 | ||
| 440 | it "should return success when deleting non existent hook" do | 440 | it "should return success when deleting non existent hook" do |
| 441 | - delete api("/projects/#{project.id}/hooks", user), hook_id: 42 | 441 | + delete api("/projects/#{project.id}/hooks/42", user) |
| 442 | response.status.should == 200 | 442 | response.status.should == 200 |
| 443 | end | 443 | end |
| 444 | - | ||
| 445 | - it "should return a 400 error if hook id not given" do | ||
| 446 | - delete api("/projects/#{project.id}/hooks", user) | ||
| 447 | - response.status.should == 400 | ||
| 448 | - end | ||
| 449 | end | 444 | end |
| 450 | 445 | ||
| 451 | describe "GET /projects/:id/repository/tags" do | 446 | describe "GET /projects/:id/repository/tags" do |