Skip to content

Restores StageCheck to Explain Vulnerability

Darby Frey requested to merge add-stage-check-etv into master

What does this MR do and why?

This MR fixes an inconsistency identified in Remove Vulnerability Resolution from Experiment... (#471110 - closed), which raised the issue that the StageCheck isn't being applied consistently to both the Explain and Resolve Vulnerability features. Currently, the Explain feature is available when the Use experiment and beta Duo features is unchecked. This change restores the StageCheck to Explain Vulnerability.

With these features going to GA soon, they won't be limited by the Use experiment and beta Duo features checkbox, but this change will ensure they are controlled correctly by the ai_features_catalogue.

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.

How to set up and validate locally

  1. Checkout the master branch, load a vulnerability.
  2. Check Use experiment and beta Duo features in the project's group settings.
  3. Reload the vulnerability, you should see the split button.
  4. Uncheck Use experiment and beta Duo features in the project's group settings.
  5. Reload the vulnerability, you should only see the Resolve button.
  6. Checkout the branch for this MR.
  7. Reload the vulnerability, neither button should be visible.
  8. Check Use experiment and beta Duo features in the project's group settings.
  9. Reload the vulnerability, you should see the split button.
Edited by Darby Frey

Merge request reports