Skip to content

Update test and remove from quarantine

Mark Lapierre requested to merge ml-fix-and-unquarantine-e2e-test into master

What does this MR do and why?

This removes a test from quarantine. It was quarantined because of a bug, which has since been fixed. See #386434 (closed)

There was also a problem with the test, possibly due to a UI change made after the test was quarantined. The test wouldn't pass because of an Ambiguous match error. This fixes that error by using a more specfic selector. Unfortunately, it had to be a CSS selector because the single QA selector gets added to multiple elements.

How to set up and validate locally

From the qa directory, assuming you have a GitLab instance running at http://gdk.test:3000:

bundle exec bin/qa Test::Instance::All http://gdk.test:3000 -- qa/specs/features/ee/browser_ui/10_govern/vulnerability_management_spec.rb

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Mark Lapierre

Merge request reports