Skip to content
Snippets Groups Projects
Commit 09ea1874 authored by Savas Vedova's avatar Savas Vedova
Browse files

Merge branch 'ph/removeExtraWidgetBorder' into 'master'

Removes an extra border in the merge request widget

See merge request !95680
parents c489ac20 f01b334a
No related branches found
No related tags found
2 merge requests!96059Draft: Add GraphQL query for deployment details,!95680Removes an extra border in the merge request widget
Pipeline #617668413 passed
......@@ -15,7 +15,12 @@ export default {
</script>
<template>
<section role="region" :aria-label="__('Merge request reports')" data-testid="mr-widget-app">
<section
v-if="widgets.length"
role="region"
:aria-label="__('Merge request reports')"
data-testid="mr-widget-app"
>
<component
:is="widget"
v-for="(widget, index) in widgets"
......
......@@ -14,9 +14,9 @@ describe('MR Widget App', () => {
});
};
it('mounts the component', () => {
it('does not mount if widgets array is empty', () => {
createComponent();
expect(wrapper.findByTestId('mr-widget-app').exists()).toBe(true);
expect(wrapper.findByTestId('mr-widget-app').exists()).toBe(false);
});
describe.each`
......
......@@ -12,8 +12,8 @@ describe('MR Widget App', () => {
});
};
it('mounts the component', () => {
it('does not mount if widgets array is empty', () => {
createComponent();
expect(wrapper.findByTestId('mr-widget-app').exists()).toBe(true);
expect(wrapper.findByTestId('mr-widget-app').exists()).toBe(false);
});
});
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment