Commit bddd1961 authored by Winnie Hellmann's avatar Winnie Hellmann Committed by Kamil Trzciński

Remove remaining TODOs from feature tests

parent 4643d944
...@@ -47,8 +47,7 @@ describe 'Feature Flags', :js do ...@@ -47,8 +47,7 @@ describe 'Feature Flags', :js do
it 'displays an error message' do it 'displays an error message' do
add_feature_flag(name, description, false) add_feature_flag(name, description, false)
# TODO: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/7433#note_104442383 expect(current_path).to eq project_feature_flags_path(project)
# expect(current_path).to eq new_project_feature_flag_path(project)
expect(page).to have_selector '.alert-danger', text: error_message expect(page).to have_selector '.alert-danger', text: error_message
end end
end end
...@@ -79,8 +78,7 @@ describe 'Feature Flags', :js do ...@@ -79,8 +78,7 @@ describe 'Feature Flags', :js do
it 'displays an error message' do it 'displays an error message' do
edit_feature_flag('feature-flag-to-edit', name, description, false) edit_feature_flag('feature-flag-to-edit', name, description, false)
# TODO: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/7433#note_104442383 expect(current_path).to eq project_feature_flags_path(project)
# expect(current_path).to eq new_project_feature_flag_path(project)
expect(page).to have_selector '.alert-danger', text: error_message expect(page).to have_selector '.alert-danger', text: error_message
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