Refactor pipeline editor tabs spec with new validation tab
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
topipeline_editor_tabs_spec.rb
since this name seems to make more sense. - We introduced a new tab
Validate
behind FFsimulate_pipeline
(to replaceLint
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.
-
I have evaluated the MR acceptance checklist for this MR.
Edited by Tiffany Rea