Move test case from `pipelines_table_spec.js` to `pipelines_table_row_spec.js`
In !44896 (merged) we removed this piece of test from pipelines_table_spec
describe('pipline actions', () => {
it('should set the "Re-deploy" title', () => {
const pipelines = [{ ...pipeline, flags: { cancelable: false, retryable: true } }];
createComponent({ ...defaultProps, pipelines });
expect(findGlButtons().length).toBe(1);
expect(
findGlButtons()
.at(0)
.attributes('title'),
).toMatch('Retry');
});
it('should set the "Cancel" title', () => {
const pipelines = [{ ...pipeline, flags: { cancelable: true, retryable: false } }];
createComponent({ ...defaultProps, pipelines });
expect(findGlButtons().length).toBe(1);
expect(
findGlButtons()
.at(0)
.attributes('title'),
).toMatch('Cancel');
});
});
This should be re-worked in pipelines_table_row_spec
to avoid losing coverage
-
@nmezzopera started a discussion: (+1 comment) @ntepluhina Thank you for fixing this, should we quickly open a followup issue to move the removed test case to the correct spec file:
spec/frontend/pipelines/pipelines_table_row_spec.js
?
Edited by Nicolò Maria Mezzopera