Refactor ready_to_merge widget tests to be user centric
Summary
The tests for the ready_to_merge widget are wirtten mostly on computed properties and do some weird shortcut like setting the pipeline to true
instead of giving a real pipeline data. This make the tests brittle and possibly giving false positives.
Improvements
Make the test closer to the actual user experience and ensure that all states are covered properly.
Risks
Breaking the tests or removing some that feel obsolete but could were covering some obscure case.
Involved tests
-
spec/frontend/vue_mr_widget/components/states/mr_widget_ready_to_merge_spec.js
-
ee/spec/frontend/vue_mr_widget/components/states/mr_widget_ready_to_merge_spec.js
Edited by Payton Burdette