Update the Secure E2E specs to use commit rather than projectpush

What does this MR do and why?

Update the Secure E2E specs to use commit rather than projectpush

As per gitlab-org/quality/quality-engineering/team-tasks#1279 (closed) - commit is faster than projectpush.

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

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

added QA Quality devopssecure teamTest Engineering maintenancerefactor typemaintenance labels

assigned to @willmeek

added pipelinetier-3 label

added sectionsec label

added 1123 commits

• 955a53cf...439ce447 - 1122 commits from branch master
• 30283aa1 - Update secure specs to use commit

Compare with previous version

	1 Warning
	This merge request does not refer to an existing milestone.

Reviewer roulette

Category	Reviewer	Maintainer
QA	@dchevalier2 (UTC-4, 5 hours behind author)	@chloeliu (UTC-7, 8 hours behind author)

Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.

If needed, you can retry the danger-review job that generated this comment.

Generated by Danger

E2E Test Result Summary

allure-report-publisher generated test report!

e2e-test-on-gdk: test report for 30283aa1

expand test summary

+-------------------------------------------------------------+
|                       suites summary                        |
+--------+--------+--------+---------+-------+-------+--------+
|        | passed | failed | skipped | flaky | total | result |
+--------+--------+--------+---------+-------+-------+--------+
| Secure | 1      | 0      | 0       | 0     | 1     | ✅     |
+--------+--------+--------+---------+-------+-------+--------+
| Total  | 1      | 0      | 0       | 0     | 1     | ✅     |
+--------+--------+--------+---------+-------+-------+--------+

e2e-package-and-test: test report for 30283aa1

expand test summary

+-------------------------------------------------------------+
|                       suites summary                        |
+--------+--------+--------+---------+-------+-------+--------+
|        | passed | failed | skipped | flaky | total | result |
+--------+--------+--------+---------+-------+-------+--------+
| Secure | 12     | 0      | 3       | 0     | 15    | ✅     |
+--------+--------+--------+---------+-------+-------+--------+
| Total  | 12     | 0      | 3       | 0     | 15    | ✅     |
+--------+--------+--------+---------+-------+-------+--------+

Hi @dchevalier2 could you please review this small piece of tidyup work for Secure specs? Thanks

requested review from @dchevalier2

mentioned in issue gitlab-org/quality/quality-engineering/team-tasks#1279 (closed)

mentioned in merge request !157056 (merged)

approved this merge request

requested review from @chloeliu and removed review request for @dchevalier2

added pipeline:mr-approved label

approved this merge request

resolved all threads

started a merge train

removed this merge request from the merge train because the pipeline did not succeed. Learn more.

enabled automatic add to merge train when checks pass

started a merge train