diff options
author | Sebastian Ziebell <sebastian.ziebell@asquera.de> | 2013-02-27 18:12:02 +0100 |
---|---|---|
committer | Sebastian Ziebell <sebastian.ziebell@asquera.de> | 2013-02-27 18:12:02 +0100 |
commit | d269d107d86c600ab2add651f47cced8f601ae84 (patch) | |
tree | 7ed4d1c587152825272ddae2bc38dcd5e748885b | |
parent | 3bb342b961f2a02098175e24855ab56fc1785c43 (diff) | |
download | gitlab-ce-d269d107d86c600ab2add651f47cced8f601ae84.tar.gz |
API: fixed adding a note
Now the correct attribute is checked if it's available or not. Also fixed a test.
-rw-r--r-- | lib/api/notes.rb | 2 | ||||
-rw-r--r-- | spec/requests/api/projects_spec.rb | 3 |
2 files changed, 2 insertions, 3 deletions
diff --git a/lib/api/notes.rb b/lib/api/notes.rb index 759fd3a9819..097cc7ea475 100644 --- a/lib/api/notes.rb +++ b/lib/api/notes.rb @@ -93,7 +93,7 @@ module Gitlab # POST /projects/:id/issues/:noteable_id/notes # POST /projects/:id/snippets/:noteable_id/notes post ":id/#{noteables_str}/:#{noteable_id_str}/notes" do - required_attributes! [:"#{noteable_id_str}"] + required_attributes! [:body] @noteable = user_project.send(:"#{noteables_str}").find(params[:"#{noteable_id_str}"]) @note = @noteable.notes.new(note: params[:body]) diff --git a/spec/requests/api/projects_spec.rb b/spec/requests/api/projects_spec.rb index 71a33f85101..994c8d5ef2c 100644 --- a/spec/requests/api/projects_spec.rb +++ b/spec/requests/api/projects_spec.rb @@ -466,8 +466,7 @@ describe Gitlab::API do response.status.should == 200 json_response.should be_an Array - #json_response.first['id'].should == project.repository.commit.id - json_response.size.should == 1 + json_response.first['id'].should == project.repository.commit.id end end |