Refactor mr_widget_security_reports_spec.js file
Summary
ee/spec/frontend/vue_merge_request_widget/extensions/security_reports/mr_widget_security_reports_spec.js
contains many occurrences of mounting - creating and expanding the mr widget logic.
This issue is to abstract that functionality into a function.
Implementation plan
Abstract occurences of:
mockWithData();
createComponent({
mountFn: mountExtended,
});
await waitForPromises();
// Click on the toggle button to expand data
wrapper.findByRole('button', { name: 'Show details' }).trigger('click');
await nextTick();
located in ee/spec/frontend/vue_merge_request_widget/extensions/security_reports/mr_widget_security_reports_spec.js
into a separate function and re-use that function.
More context here: !104699 (comment 1181456876)
Verification steps
-
Checkout latest master
-
Run tests in watch mode: yarn jest --watch
-
Hit the p
button to filter by file name and type:mr_widget_security_reports_spec
Tests should be passing.
Edited by Lorenz van Herwaarden