diff options
author | Rémy Coutable <remy@rymai.me> | 2016-10-10 17:03:34 +0200 |
---|---|---|
committer | Rémy Coutable <remy@rymai.me> | 2016-10-10 17:07:58 +0200 |
commit | 4917bbd7ffc9e7e67d93a08650d95d02a0a67081 (patch) | |
tree | 9be1a023dd64ff69785fdad6bc70fc41e0f0325c | |
parent | b06acbf52d76e36f55dcee5dd9935b43f1b7bbd2 (diff) | |
download | gitlab-ce-speed-up-api-projects-spec.tar.gz |
Speed up specs for GET /projects/:id/eventsspeed-up-api-projects-spec
From 8.15s to 4.55s by grouping expectations
Signed-off-by: Rémy Coutable <remy@rymai.me>
-rw-r--r-- | spec/requests/api/projects_spec.rb | 28 |
1 files changed, 15 insertions, 13 deletions
diff --git a/spec/requests/api/projects_spec.rb b/spec/requests/api/projects_spec.rb index 5f19638b460..85717d274aa 100644 --- a/spec/requests/api/projects_spec.rb +++ b/spec/requests/api/projects_spec.rb @@ -588,37 +588,39 @@ describe API::API, api: true do before do note = create(:note_on_issue, note: 'What an awesome day!', project: project) EventCreateService.new.leave_note(note, note.author) - get api("/projects/#{project.id}/events", user) end - it { expect(response).to have_http_status(200) } + it 'returns all events' do + get api("/projects/#{project.id}/events", user) - context 'joined event' do - let(:json_event) { json_response[1] } + expect(response).to have_http_status(200) - it { expect(json_event['action_name']).to eq('joined') } - it { expect(json_event['project_id'].to_i).to eq(project.id) } - it { expect(json_event['author_username']).to eq(user3.username) } - it { expect(json_event['author']['name']).to eq(user3.name) } - end + first_event = json_response.first - context 'comment event' do - let(:json_event) { json_response.first } + expect(first_event['action_name']).to eq('commented on') + expect(first_event['note']['body']).to eq('What an awesome day!') - it { expect(json_event['action_name']).to eq('commented on') } - it { expect(json_event['note']['body']).to eq('What an awesome day!') } + last_event = json_response.last + + expect(last_event['action_name']).to eq('joined') + expect(last_event['project_id'].to_i).to eq(project.id) + expect(last_event['author_username']).to eq(user3.username) + expect(last_event['author']['name']).to eq(user3.name) end end it 'returns a 404 error if not found' do get api('/projects/42/events', user) + expect(response).to have_http_status(404) expect(json_response['message']).to eq('404 Project Not Found') end it 'returns a 404 error if user is not a member' do other_user = create(:user) + get api("/projects/#{project.id}/events", other_user) + expect(response).to have_http_status(404) end end |