Skip to content

Commit

Permalink
Renaming get_commits to commits
Browse files Browse the repository at this point in the history
  • Loading branch information
dmgarland committed Mar 29, 2016
1 parent 59d06c3 commit 4aa27d1
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion lib/bitbucket_rest_api/repos/pull_request.rb
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ def update(user_name, repo_name, pull_request_id, params={})
return response unless block_given?
end

def get_commits(user_name, repo_name, pull_request_id, params={})
def commits(user_name, repo_name, pull_request_id, params={})
_update_user_repo_params(user_name, repo_name)
_validate_user_repo_params(user, repo) unless user? && repo?
normalize! params
Expand Down
6 changes: 3 additions & 3 deletions spec/bitbucket_rest_api/repos/pull_request_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@
end
end

describe '.get_commits' do
describe '.commits' do
before do
expect(subject).to receive(:request).with(
:get,
Expand All @@ -136,11 +136,11 @@
end

it 'makes a GET request for the commits' do
subject.get_commits('mock_user', 'mock_repo', 'mock_id')
subject.commits('mock_user', 'mock_repo', 'mock_id')
end
end

describe '.get_commits' do
describe '.commits' do
before do
expect(subject).to receive(:request).with(
:post,
Expand Down

1 comment on commit 4aa27d1

@iconnor
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tagging issue #44 and PR #55

Please sign in to comment.