diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2020-01-09 21:07:48 +0000 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2020-01-09 21:07:48 +0000 |
commit | e98d69bc8b8b926a727d36e37d2ee30c9fa28907 (patch) | |
tree | 1e86398c19b6ddd0602a99fb449cf75652ac62cf /spec/lib | |
parent | 007aba8b3b5583c5ef31670369f32f9f1f72555d (diff) | |
download | gitlab-ce-e98d69bc8b8b926a727d36e37d2ee30c9fa28907.tar.gz |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'spec/lib')
-rw-r--r-- | spec/lib/sentry/client/issue_spec.rb | 43 |
1 files changed, 39 insertions, 4 deletions
diff --git a/spec/lib/sentry/client/issue_spec.rb b/spec/lib/sentry/client/issue_spec.rb index b5ee5063e86..f4796a27b4d 100644 --- a/spec/lib/sentry/client/issue_spec.rb +++ b/spec/lib/sentry/client/issue_spec.rb @@ -6,7 +6,9 @@ describe Sentry::Client::Issue do include SentryClientHelpers let(:token) { 'test-token' } + let(:sentry_url) { 'https://sentrytest.gitlab.com/api/0' } let(:client) { Sentry::Client.new(sentry_url, token) } + let(:issue_id) { 503504 } describe '#list_issues' do shared_examples 'issues have correct return type' do |klass| @@ -225,15 +227,11 @@ describe Sentry::Client::Issue do ) end - let(:issue_id) { 503504 } - let(:sentry_url) { 'https://sentrytest.gitlab.com/api/0' } let(:sentry_request_url) { "#{sentry_url}/issues/#{issue_id}/" } let!(:sentry_api_request) { stub_sentry_request(sentry_request_url, body: issue_sample_response) } subject { client.issue_details(issue_id: issue_id) } - it_behaves_like 'calls sentry api' - it 'escapes issue ID' do allow(CGI).to receive(:escape).and_call_original @@ -290,4 +288,41 @@ describe Sentry::Client::Issue do end end end + + describe '#update_issue' do + let(:sentry_url) { 'https://sentrytest.gitlab.com/api/0' } + let(:sentry_request_url) { "#{sentry_url}/issues/#{issue_id}/" } + + before do + stub_sentry_request(sentry_request_url, :put) + end + + let(:params) do + { + status: 'resolved' + } + end + + subject { client.update_issue(issue_id: issue_id, params: params) } + + it_behaves_like 'calls sentry api' do + let(:sentry_api_request) { stub_sentry_request(sentry_request_url, :put) } + end + + it 'returns a truthy result' do + expect(subject).to be_truthy + end + + context 'error encountered' do + let(:error) { StandardError.new('error') } + + before do + allow(client).to receive(:update_issue).and_raise(error) + end + + it 'raises the error' do + expect { subject }.to raise_error(error) + end + end + end end |