Skip to content

Commit

Permalink
Fixed CI failures on Circle CI introduced by commit 8f04e32. This is …
Browse files Browse the repository at this point in the history
…a follow-up to original issue #2534 - Agents should always be able to re-open closed tickets, regardless of group.follow_up_possible.
  • Loading branch information
Billy Zhou authored and znuny-robo committed May 6, 2019
1 parent c16c86e commit d4b0d95
Showing 1 changed file with 26 additions and 12 deletions.
38 changes: 26 additions & 12 deletions spec/requests/ticket_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2088,6 +2088,7 @@
describe 'reopening a ticket' do
shared_examples 'successfully reopen a ticket' do
it 'succeeds' do
authenticated_as(user)
put "/api/v1/tickets/#{ticket.id}",
params: { state_id: Ticket::State.find_by(name: 'open').id },
as: :json
Expand All @@ -2099,6 +2100,7 @@

shared_examples 'fail to reopen a ticket' do
it 'fails' do
authenticated_as(user)
put "/api/v1/tickets/#{ticket.id}",
params: { state_id: Ticket::State.find_by(name: 'open').id },
as: :json
Expand All @@ -2111,32 +2113,44 @@
context 'when ticket.group.follow_up_possible = "yes"' do
before { ticket.group.update(follow_up_possible: 'yes') }

context 'as admin', authenticated_as: :admin do
include_examples 'successfully reopen a ticket'
context 'as admin' do
include_examples 'successfully reopen a ticket' do
let(:user) { admin }
end
end

context 'as agent', authenticated_as: :agent do
include_examples 'successfully reopen a ticket'
context 'as agent' do
include_examples 'successfully reopen a ticket' do
let(:user) { agent }
end
end

context 'as customer', authenticated_as: :customer do
include_examples 'successfully reopen a ticket'
context 'as customer' do
include_examples 'successfully reopen a ticket' do
let(:user) { customer }
end
end
end

context 'when ticket.group.follow_up_possible = "new_ticket"' do
before { ticket.group.update(follow_up_possible: 'new_ticket') }

context 'as admin', authenticated_as: :admin do
include_examples 'successfully reopen a ticket'
context 'as admin' do
include_examples 'successfully reopen a ticket' do
let(:user) { admin }
end
end

context 'as agent', authenticated_as: :agent do
include_examples 'successfully reopen a ticket'
context 'as agent' do
include_examples 'successfully reopen a ticket' do
let(:user) { agent }
end
end

context 'as customer', authenticated_as: :customer do
include_examples 'fail to reopen a ticket'
context 'as customer' do
include_examples 'fail to reopen a ticket' do
let(:user) { customer }
end
end
end
end
Expand Down

0 comments on commit d4b0d95

Please sign in to comment.