Skip to content

Refactor pipeline editor tabs spec with new validation tab

Tiffany Rea requested to merge add-simulate-pipeline-ff-case into master

What does this MR do and why?

Describe in detail what your merge request does and why.

  • Changed spec name from pipeline_editor_lint_spec.rb to pipeline_editor_tabs_spec.rb since this name seems to make more sense.
  • We introduced a new tab Validate behind FF simulate_pipeline (to replace Lint tab eventually), so I create scenarios for both when FF is enabled vs disabled. Most of the code are redundant now but they will be removed when the flag is removed so I think it's okay not to spend too much time on shrinking them.

MR acceptance checklist

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

Edited by Tiffany Rea

Merge request reports