diff options
Diffstat (limited to 'spec/frontend/releases/components/release_block_assets_spec.js')
-rw-r--r-- | spec/frontend/releases/components/release_block_assets_spec.js | 36 |
1 files changed, 4 insertions, 32 deletions
diff --git a/spec/frontend/releases/components/release_block_assets_spec.js b/spec/frontend/releases/components/release_block_assets_spec.js index 4f94e4dfd55..6d53bf5a49e 100644 --- a/spec/frontend/releases/components/release_block_assets_spec.js +++ b/spec/frontend/releases/components/release_block_assets_spec.js @@ -123,42 +123,14 @@ describe('Release block assets', () => { }); }); - describe('external vs internal links', () => { + describe('links', () => { const containsExternalSourceIndicator = () => wrapper.find('[data-testid="external-link-indicator"]').exists(); - describe('when a link is external', () => { - beforeEach(() => { - defaultProps.assets.sources = []; - defaultProps.assets.links = [ - { - ...defaultProps.assets.links[0], - external: true, - }, - ]; - createComponent(defaultProps); - }); - - it('renders the link with an "external source" indicator', () => { - expect(containsExternalSourceIndicator()).toBe(true); - }); - }); + beforeEach(() => createComponent(defaultProps)); - describe('when a link is internal', () => { - beforeEach(() => { - defaultProps.assets.sources = []; - defaultProps.assets.links = [ - { - ...defaultProps.assets.links[0], - external: false, - }, - ]; - createComponent(defaultProps); - }); - - it('renders the link without the "external source" indicator', () => { - expect(containsExternalSourceIndicator()).toBe(false); - }); + it('renders with an external source indicator (except for sections with no title)', () => { + expect(containsExternalSourceIndicator()).toBe(true); }); }); }); |