Skip to content

Fix logic for waiting on CI YAML content

Heinrich Lee Yu requested to merge 370204-fix-broken-master into master

What does this MR do and why?

Fixes a race condition where we're clicking "Validate" before the editor is populated with the YAML contents.

We can see in the failure screenshot that we're getting the Please provide content of .gitlab-ci.yml error.

Screenshots or screen recordings

ci_lint_yaml_parsing_behaves_like_validates_the_yaml_yaml_is_correct_parses_yaml_and_displays_the_jobs

How to set up and validate locally

Run bin/rspec spec/features/projects/ci/lint_spec.rb.

Specs here fail consistently for me locally before this change.

MR acceptance checklist

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

Related to #370204 (closed)

Edited by Heinrich Lee Yu

Merge request reports