Skip to content

Update Explain This Vulnerability

Will Meek requested to merge explain_vuln_spec_fix into master

What does this MR do and why?

Update Explain This Vulnerability as used in specs

The UI has been refactored in https://gitlab.com/gitlab-org/gitlab/-/issues/476691, now click the "Explain or Resolve with AI" button for the explain this vulnerability spec

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

See https://gitlab.com/gitlab-org/gitlab/-/issues/476691

How to set up and validate locally

This is observed in Staging, so unless you have the "Explain this" button enabled locally, it can be ran there:

GITLAB_QA_USER_AGENT =<USER_AGENT> GITLAB_ADMIN_USERNAME=<ADMIN_USERNAME> GITLAB_ADMIN_PASSWORD="<ADMIN_PASSWORD>" GITLAB_USERNAME=<USERNAME> GITLAB_QA_ACCESS_TOKEN="<ACCESS_TOKEN>" GITLAB_PASSWORD="<GITLAB_PASSWORD>"  QA_DEBUG=true WEBDRIVER_HEADLESS=true bundle exec bin/qa Test::Instance::All https://staging.gitlab.com qa/specs/features/ee/browser_ui/10_govern/explain_this_vulnerability_spec.rb

Merge request reports

Loading