Skip to content

Update the On Demand DAST E2E Spec

Will Meek requested to merge dast_spec_edit_delete into master

What does this MR do and why?

As per gitlab-org/quality/quality-engineering/team-tasks#2827 (closed) , there was a frontend issue raised around edit/delete buttons not being visible for On Demand DAST that went unnoticed due to test coverage - #468670 (closed)

We have an existing On Demand DAST E2E spec that checks that a scan is present with correct values on this page

After discussion - https://gitlab.slack.com/archives/CKWJP0ZS7/p1719476050700609 - it was decided we could use this test and extend it to cover this test gap

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

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

n/a

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/on_demand_dast_spec.rb

Merge request reports