From 9b807a1091d7939e0b5206d6fd76a3a034d4cda4 Mon Sep 17 00:00:00 2001 From: Shinya Maeda Date: Wed, 7 Nov 2018 18:56:34 +0900 Subject: [PATCH] Improve test quality --- spec/features/projects/environments/environment_spec.rb | 6 +++--- spec/models/environment_status_spec.rb | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/spec/features/projects/environments/environment_spec.rb b/spec/features/projects/environments/environment_spec.rb index 0d82df6dc8e..9772a7bacac 100644 --- a/spec/features/projects/environments/environment_spec.rb +++ b/spec/features/projects/environments/environment_spec.rb @@ -25,7 +25,7 @@ describe 'Environment' do end context 'without deployments' do - it 'does show no deployments' do + it 'does not show deployments' do expect(page).to have_content('You don\'t have any deployments right now.') end end @@ -64,7 +64,7 @@ describe 'Environment' do create(:deployment, :running, environment: environment, deployable: build) end - it 'does show no deployments' do + it 'does not show deployments' do expect(page).to have_content('You don\'t have any deployments right now.') end end @@ -77,7 +77,7 @@ describe 'Environment' do create(:deployment, :failed, environment: environment, deployable: build) end - it 'does show no deployments' do + it 'does not show deployments' do expect(page).to have_content('You don\'t have any deployments right now.') end end diff --git a/spec/models/environment_status_spec.rb b/spec/models/environment_status_spec.rb index db9ee204849..90f7e4a4590 100644 --- a/spec/models/environment_status_spec.rb +++ b/spec/models/environment_status_spec.rb @@ -99,7 +99,7 @@ describe EnvironmentStatus do let(:user) { project.owner } before do - build.deployment&.update(sha: sha) + build.deployment&.update!(sha: sha) end context 'when environment is created on a forked project' do