Skip to content

Update Secure Configuration SAST test

Will Meek requested to merge sast_spec_update into master

What does this MR do and why?

As per !125267 (merged) - SAST Analyzers were removed from the UI

Update Secure Configuration SAST test

Remove SAST analyser step

  • /qa/qa/specs/features/ee/browser_ui/13_secure/enable_scanning_from_configuration_spec.rb
  • /qa/qa/ee/page/project/secure/configuration_form.rb

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After
image image

How to set up and validate locally

CHROME_HEADLESS=false bundle exec bin/qa Test::Instance::All http://gdk.test:3000/ ./qa/specs/features/ee/browser_ui/13_secure/enable_scanning_from_configuration_spec.rb:104

MR acceptance checklist

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

Merge request reports