Skip to content

Commit

Permalink
Add Linkify test
Browse files Browse the repository at this point in the history
  • Loading branch information
deluan committed Oct 7, 2022
1 parent 53a4ea6 commit 62e7492
Show file tree
Hide file tree
Showing 2 changed files with 38 additions and 1 deletion.
5 changes: 4 additions & 1 deletion ui/src/common/Linkify.js
Expand Up @@ -54,7 +54,10 @@ const Linkify = ({ text, ...rest }) => {
// Push remaining text
if (text.length > lastIndex) {
elements.push(
<span dangerouslySetInnerHTML={{ __html: text.substring(lastIndex) }} />
<span
key={'last-span-key'}
dangerouslySetInnerHTML={{ __html: text.substring(lastIndex) }}
/>
)
}

Expand Down
34 changes: 34 additions & 0 deletions ui/src/common/Linkify.test.js
@@ -0,0 +1,34 @@
import React from 'react'
import { render, screen } from '@testing-library/react'
import '@testing-library/jest-dom/extend-expect'
import Linkify from './Linkify'

const URL = 'http://www.example.com'

const expectLink = (url) => {
const linkEl = screen.getByRole('link')
expect(linkEl).not.toBeNull()
expect(linkEl?.href).toBe(url)
}

describe('<Linkify />', () => {
it('should render link', () => {
render(<Linkify text={URL} />)
expectLink(`${URL}/`)
expect(screen.getByText(URL)).toBeInTheDocument()
})

it('should render link and text', () => {
render(<Linkify text={`foo ${URL} bar`} />)
expectLink(`${URL}/`)
expect(screen.getByText(/foo/i)).toBeInTheDocument()
expect(screen.getByText(URL)).toBeInTheDocument()
expect(screen.getByText(/bar/i)).toBeInTheDocument()
})

it('should render only text', () => {
render(<Linkify text={'foo bar'} />)
expect(screen.queryAllByRole('link')).toHaveLength(0)
expect(screen.getByText(/foo bar/i)).toBeInTheDocument()
})
})

0 comments on commit 62e7492

Please sign in to comment.