Skip to content

Commit c7c500e

Browse files
saraycpdanidoni
authored andcommitted
Write state and status in specs as well
We need to keep writing to state in specs as well. Otherwise, some of them will fail until we finally start reading only from status.
1 parent d60c400 commit c7c500e

File tree

3 files changed

+65
-3
lines changed

3 files changed

+65
-3
lines changed

src/api/spec/cassettes/Requests/review/for_user/opens_a_review_and_accepts_it.yml

Lines changed: 62 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/api/spec/features/webui/requests_spec.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -329,7 +329,7 @@
329329
end
330330

331331
it 'when request is in a final state' do
332-
bs_request.update(status: :accepted)
332+
bs_request.update(state: :accepted, status: :accepted)
333333
visit request_show_path(bs_request)
334334
expect(page).to have_text("Auto-accept was set to #{I18n.l(bs_request.accept_at, format: :only_date)}.")
335335
end
@@ -369,7 +369,7 @@
369369

370370
describe 'for a request with a deleted target project' do
371371
let!(:delete_bs_request) do
372-
create(:delete_bs_request, target_project: target_project, description: 'a long text - ' * 200, creator: submitter, status: :accepted)
372+
create(:delete_bs_request, target_project: target_project, description: 'a long text - ' * 200, creator: submitter, state: :accepted, status: :accepted)
373373
end
374374

375375
before do

src/api/spec/models/project/staging_project_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@
9393

9494
context 'when request got revoked' do
9595
before do
96-
submit_request.update(status: 'revoked')
96+
submit_request.update(state: 'revoked', status: 'revoked')
9797
end
9898

9999
it { expect(staging_project.overall_state).to eq(:unacceptable) }

0 commit comments

Comments
 (0)