Commit 59328941 authored by Tom Quirk's avatar Tom Quirk

Remove redundant test in item_spec

Tests "hides comment count" and
"renders item with no status icon for none event"
were testing the same state against snapshots
that are equivalent. The test "hides comment count"
has been removed in favour of the other.
parent 0f63ba44
// Jest Snapshot v1, https://goo.gl/fbAQLP // Jest Snapshot v1, https://goo.gl/fbAQLP
exports[`Design management list item component with no notes hides comment count 1`] = `
<router-link-stub
class="card cursor-pointer text-plain js-design-list-item design-list-item"
to="[object Object]"
>
<div
class="card-body p-0 d-flex-center overflow-hidden position-relative"
>
<!---->
<img
alt="test"
class="block ml-auto mr-auto mw-100 mh-100 design-img"
data-qa-selector="design_image"
src="http://via.placeholder.com/300"
/>
</div>
<div
class="card-footer d-flex w-100"
>
<div
class="d-flex flex-column str-truncated-100"
>
<span
class="bold str-truncated-100"
data-qa-selector="design_file_name"
>
test
</span>
<span
class="str-truncated-100"
>
Updated
<timeago-stub
cssclass=""
time="01-01-2019"
tooltipplacement="bottom"
/>
</span>
</div>
<!---->
</div>
</router-link-stub>
`;
exports[`Design management list item component with no notes renders item with correct status icon for creation event 1`] = ` exports[`Design management list item component with no notes renders item with correct status icon for creation event 1`] = `
<router-link-stub <router-link-stub
class="card cursor-pointer text-plain js-design-list-item design-list-item" class="card cursor-pointer text-plain js-design-list-item design-list-item"
......
...@@ -57,7 +57,7 @@ describe('Design management list item component', () => { ...@@ -57,7 +57,7 @@ describe('Design management list item component', () => {
}); });
describe('with no notes', () => { describe('with no notes', () => {
it('hides comment count', () => { it('renders item with no status icon for none event', () => {
createComponent(); createComponent();
expect(wrapper.element).toMatchSnapshot(); expect(wrapper.element).toMatchSnapshot();
...@@ -81,12 +81,6 @@ describe('Design management list item component', () => { ...@@ -81,12 +81,6 @@ describe('Design management list item component', () => {
expect(wrapper.element).toMatchSnapshot(); expect(wrapper.element).toMatchSnapshot();
}); });
it('renders item with no status icon for none event', () => {
createComponent();
expect(wrapper.element).toMatchSnapshot();
});
it('renders loading spinner when isLoading is true', () => { it('renders loading spinner when isLoading is true', () => {
createComponent({ isLoading: true }); createComponent({ isLoading: true });
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment