Commit 42c9bde5 authored by Rémy Coutable's avatar Rémy Coutable

Merge branch '2079-transient-failure-environments' into 'master'

Fixes transient failure: waits for asyncronous update

Closes #2079

See merge request !1567
parents 2019c376 8388d13a
...@@ -30,6 +30,7 @@ feature 'Environments page', :feature, :js do ...@@ -30,6 +30,7 @@ feature 'Environments page', :feature, :js do
describe 'in available tab page' do describe 'in available tab page' do
it 'should show one environment' do it 'should show one environment' do
visit namespace_project_environments_path(project.namespace, project, scope: 'available') visit namespace_project_environments_path(project.namespace, project, scope: 'available')
expect(page).to have_css('.environments-container')
expect(page.all('tbody > tr').length).to eq(1) expect(page.all('tbody > tr').length).to eq(1)
end end
end end
...@@ -37,6 +38,7 @@ feature 'Environments page', :feature, :js do ...@@ -37,6 +38,7 @@ feature 'Environments page', :feature, :js do
describe 'in stopped tab page' do describe 'in stopped tab page' do
it 'should show no environments' do it 'should show no environments' do
visit namespace_project_environments_path(project.namespace, project, scope: 'stopped') visit namespace_project_environments_path(project.namespace, project, scope: 'stopped')
expect(page).to have_css('.environments-container')
expect(page).to have_content('You don\'t have any environments right now') expect(page).to have_content('You don\'t have any environments right now')
end end
end end
...@@ -48,6 +50,7 @@ feature 'Environments page', :feature, :js do ...@@ -48,6 +50,7 @@ feature 'Environments page', :feature, :js do
describe 'in available tab page' do describe 'in available tab page' do
it 'should show no environments' do it 'should show no environments' do
visit namespace_project_environments_path(project.namespace, project, scope: 'available') visit namespace_project_environments_path(project.namespace, project, scope: 'available')
expect(page).to have_css('.environments-container')
expect(page).to have_content('You don\'t have any environments right now') expect(page).to have_content('You don\'t have any environments right now')
end end
end end
...@@ -55,6 +58,7 @@ feature 'Environments page', :feature, :js do ...@@ -55,6 +58,7 @@ feature 'Environments page', :feature, :js do
describe 'in stopped tab page' do describe 'in stopped tab page' do
it 'should show one environment' do it 'should show one environment' do
visit namespace_project_environments_path(project.namespace, project, scope: 'stopped') visit namespace_project_environments_path(project.namespace, project, scope: 'stopped')
expect(page).to have_css('.environments-container')
expect(page.all('tbody > tr').length).to eq(1) expect(page.all('tbody > tr').length).to eq(1)
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