Commit ae099857 authored by Airat Shigapov's avatar Airat Shigapov

Rewrite events order spec to simulate wrong order without changing ids

parent e15f15d5
...@@ -959,23 +959,24 @@ describe API::API, api: true do ...@@ -959,23 +959,24 @@ describe API::API, api: true do
end end
end end
context 'when there are multiple events' do context 'when there are multiple events from different projects' do
let(:old_note) { create(:note_on_issue, project: project) } let(:another_project) { create(:empty_project) }
let(:new_event) { note.events.first } let(:notes) { create_list(:note_on_issue, 5, project: [project, another_project].sample) }
let(:old_event) { old_note.events.first }
before do before do
EventCreateService.new.leave_note(old_note, user) another_project.add_user(user, :developer)
new_event.update(id: 1000) notes.each { |note| EventCreateService.new.leave_note(note, user) }
old_event.update(id: 900)
end end
it 'returns them in the correct order (from newest to oldest determined by ID field)' do it 'returns them in the correct order (from newest to oldest determined by ID field)' do
get api("/users/#{user.id}/events", user) get api("/users/#{user.id}/events", user)
expect(json_response[0]['target_id']).to eq(note.id) comment_events = json_response.select { |e| e['action_name'] == 'commented on' }
expect(json_response[1]['target_id']).to eq(old_note.id)
notes.reverse.each_with_index do |note, i|
expect(note.id).to eq(comment_events[i]['target_id'])
end
end end
end end
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment