Commit 2cea1fe8 authored by Will Meek's avatar Will Meek

Fix for project_security_dashboard_spec

As per issue
https://gitlab.com/gitlab-org/gitlab/-/issues/233362
it appears that it is possible that the user
is not logged in when visiting the project,
and therefore the security dashboard is
unavailable

This MR adds a sign_in_unless_signed_in
call before visiting the project. It
also removes a retry that was added
previously for the same issue as it
turns out that the user not being
logged in is the root cause
parent 0a4f597a
...@@ -60,10 +60,9 @@ module QA ...@@ -60,10 +60,9 @@ module QA
end end
it 'shows vulnerability details' do it 'shows vulnerability details' do
Flow::Login.sign_in_unless_signed_in
@project.visit! @project.visit!
Support::Retrier.retry_on_exception do Page::Project::Menu.perform(&:click_on_security_dashboard)
Page::Project::Menu.perform(&:click_on_security_dashboard)
end
EE::Page::Project::Secure::SecurityDashboard.perform do |security_dashboard| EE::Page::Project::Secure::SecurityDashboard.perform do |security_dashboard|
expect(security_dashboard).to have_vulnerability(description: vulnerability_name) expect(security_dashboard).to have_vulnerability(description: vulnerability_name)
......
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