@telus-uds/components-base 0.0.2-prerelease.4 → 0.0.2-prerelease.8

package/__tests__/ActivityIndicator/ActivityIndicator.test.jsx

@@ -4,7 +4,7 @@ import Theme from '../../__fixtures__/Theme'
 import ActivityIndicator from '../../src/ActivityIndicator'
 
 // Used to remove the "Animated: `useNativeDriver` is not supported because the native animated module is missing." warnings
-jest.mock('react-native/Libraries/Animated/src/NativeAnimatedHelper')
+jest.mock('react-native/Libraries/Animated/NativeAnimatedHelper')
 
 const render = (jsx) => baseRender(jsx, { wrapper: Theme })
 

package/__tests__/Button/ButtonBase.test.jsx

@@ -1,4 +1,3 @@
-/* eslint-disable no-console */
 import React from 'react'
 import { render } from '@testing-library/react-native'
 import { Platform, Text } from 'react-native'
@@ -9,41 +8,13 @@ import ButtonBase from '../../src/Button/ButtonBase'
 
 beforeEach(() => jest.clearAllMocks())
 
-describe('ButtonBase', () => {
-  it('throws if both href and onPress are provided', async () => {
-    jest.spyOn(console, 'error').mockImplementation()
-    expect(() =>
-      render(
-        <Theme>
-          <ButtonBase href="example.com" onPress={jest.fn()}>
-            click me
-          </ButtonBase>
-        </Theme>
-      )
-    ).toThrow()
-    expect(console.error).toHaveBeenCalledTimes(1)
-  })
-
-  it('throws if neither href nor onPress are provided', async () => {
-    jest.spyOn(console, 'error').mockImplementation()
-    expect(() =>
-      render(
-        <Theme>
-          <ButtonBase>click me</ButtonBase>
-        </Theme>
-      )
-    ).toThrow()
-    expect(console.error).toHaveBeenCalledTimes(1)
-  })
-})
-
 describe('ButtonBase with render function child', () => {
-  it('recieves text styles in render function from button theme', () => {
+  it('receives tokens in render function child', () => {
     const { tokens } = testTheme.components.Button
-    const arbitraryWeight = '123'
+    const arbitraryWeight = '300'
     const { getByText } = render(
       <Theme>
-        <ButtonBase onPress={() => {}} tokens={{ fontWeight: arbitraryWeight }}>
+        <ButtonBase onPress={() => {}} tokens={{ ...tokens, fontWeight: arbitraryWeight }}>
           {({ textStyles }) => <Text style={textStyles}>Function child</Text>}
         </ButtonBase>
       </Theme>

package/__tests__/Button/ButtonGroup.test.jsx

@@ -106,7 +106,7 @@ describe('ButtonGroup (uncontrolled)', () => {
       </Theme>
     )
     expect(queryAllByA11yRole('checkbox')).toHaveLength(0)
-    expect(queryAllByA11yRole('radioGroup')).toHaveLength(1)
+    expect(queryAllByA11yRole('radiogroup')).toHaveLength(1)
     const radios = queryAllByA11yRole('radio')
     expect(radios).toHaveLength(4)
 
@@ -158,7 +158,7 @@ describe('ButtonGroup (uncontrolled)', () => {
       </Theme>
     )
 
-    expect(queryAllByA11yRole('radioGroup')).toHaveLength(0)
+    expect(queryAllByA11yRole('radiogroup')).toHaveLength(0)
     expect(queryAllByA11yRole('radio')).toHaveLength(0)
 
     const checks = queryAllByA11yRole('checkbox')

package/__tests__/Checkbox/Checkbox.test.jsx

@@ -0,0 +1,94 @@
+import React from 'react'
+import { fireEvent, render } from '@testing-library/react-native'
+
+import { Checkbox } from '../../src'
+import Theme from '../../__fixtures__/Theme'
+
+const checkedId = 'Checkbox-Icon'
+const inputId = 'Checkbox-Input'
+const setup = (props) =>
+  render(
+    <Theme>
+      <Checkbox {...props} />
+    </Theme>
+  )
+
+describe('Checkbox', () => {
+  it('renders with defaults', () => {
+    const { getByRole } = setup()
+
+    expect(getByRole('checkbox')).toBeTruthy()
+  })
+
+  it('renders the label', () => {
+    const label = 'Test label message'
+    const { queryByText } = setup({ label })
+
+    expect(queryByText(label)).toBeTruthy()
+  })
+
+  it('renders the feedback', () => {
+    const feedback = 'Test feedback message'
+    const { queryByText } = setup({ feedback })
+
+    expect(queryByText(feedback)).toBeTruthy()
+  })
+
+  it('renders error styling on validation failure', () => {
+    const errorBorderColor = '#e12339' // from __fixtures__/testTheme.js
+    const { getByTestId } = setup({ error: true })
+
+    const checkboxInput = getByTestId(inputId)
+    expect(checkboxInput).toHaveStyle({ borderColor: errorBorderColor })
+  })
+
+  it('triggers the callback', () => {
+    const onChange = jest.fn()
+
+    const { getByRole } = setup({ onChange })
+
+    const checkbox = getByRole('checkbox')
+    expect(onChange).not.toHaveBeenCalled()
+    fireEvent(checkbox, 'press')
+    expect(onChange).toHaveBeenCalledTimes(1)
+  })
+
+  it('cannot be changed if inactive', () => {
+    const { getByRole, queryByTestId } = setup({ inactive: true })
+
+    const checkbox = getByRole('checkbox')
+    expect(queryByTestId(checkedId)).toBeFalsy()
+    fireEvent(checkbox, 'press')
+    expect(queryByTestId(checkedId)).toBeFalsy()
+  })
+
+  describe('when uncontrolled', () => {
+    it('uses `defaultChecked` as the initial value', () => {
+      const { queryByTestId } = setup({ defaultChecked: true })
+
+      expect(queryByTestId(checkedId)).toBeTruthy()
+    })
+
+    it('changes value', () => {
+      const { getByRole, queryByTestId } = setup({ defaultChecked: false })
+
+      const checkbox = getByRole('checkbox')
+      expect(queryByTestId(checkedId)).toBeFalsy()
+      fireEvent(checkbox, 'press')
+      expect(queryByTestId(checkedId)).toBeTruthy()
+    })
+  })
+
+  describe('when controlled', () => {
+    it('calls `onChange` with the new value', () => {
+      const onChange = jest.fn()
+      const { getByRole, queryByTestId } = setup({ onChange, checked: false })
+
+      const checkbox = getByRole('checkbox')
+      expect(queryByTestId(checkedId)).toBeFalsy()
+      fireEvent(checkbox, 'press')
+      expect(onChange).toHaveBeenCalledTimes(1)
+      expect(onChange).toHaveBeenCalledWith(true)
+    })
+  })
+})

package/__tests__/Checkbox/CheckboxGroup.test.jsx

@@ -0,0 +1,247 @@
+import React from 'react'
+import { render, fireEvent } from '@testing-library/react-native'
+import { toHaveTextContent } from '@testing-library/jest-native'
+
+import Theme from '../../__fixtures__/Theme'
+import CheckboxGroup from '../../src/Checkbox/CheckboxGroup'
+
+const items = [
+  { label: 'One', id: 'one' },
+  { label: 'Two', id: 'two' },
+  { label: 'Three', id: 'three' },
+  { label: 'Four', id: 'four' }
+]
+const checked = { checked: true }
+
+// expect().toHaveTextContent doesn't work on arrays, expect().toContain etc doesn't match elements
+const containsText = (queryResult, text) =>
+  queryResult.some((testInstance) => toHaveTextContent(testInstance, text).pass)
+
+describe('CheckboxGroup', () => {
+  // eslint-disable-next-line no-console
+  const consoleError = console.error
+  beforeEach(() => {
+    // eslint-disable-next-line no-console
+    console.error = () => {}
+  })
+  afterEach(() => {
+    // eslint-disable-next-line no-console
+    console.error = consoleError
+  })
+
+  it('Throws if has `checkedIds` without `onChange`', () => {
+    expect(() =>
+      render(
+        <Theme>
+          <CheckboxGroup items={items} checkedIds={['one']} />
+        </Theme>
+      )
+    ).toThrow(/values.+without.+onChange/)
+  })
+  it("Doesn't throw if has `checkedIds` without `onChange` and is read only", () => {
+    expect(() =>
+      render(
+        <Theme>
+          <CheckboxGroup items={items} checkedIds={['one']} readOnly />
+        </Theme>
+      )
+    ).not.toThrow()
+  })
+  it('Throws if has both `checkedIds` and `initialCheckedIds`', () => {
+    expect(() =>
+      render(
+        <Theme>
+          <CheckboxGroup
+            items={items}
+            checkedIds={['one']}
+            initialCheckedIds={['one']}
+            onChange={() => {}}
+          />
+        </Theme>
+      )
+    ).toThrow(/both(?=.*initialValues)(?=.*values){2}/)
+  })
+})
+
+describe('CheckboxGroup (uncontrolled)', () => {
+  it('Selects unlimited items', async () => {
+    const { getByText, queryAllByA11yState } = render(
+      <Theme>
+        <CheckboxGroup items={items} />
+      </Theme>
+    )
+
+    expect(queryAllByA11yState(checked)).toHaveLength(0)
+
+    const one = getByText('One')
+    await fireEvent.press(one)
+    expect(queryAllByA11yState(checked)).toHaveLength(1)
+    expect(containsText(queryAllByA11yState(checked), 'One')).toBeTruthy()
+
+    const two = getByText('Two')
+    await fireEvent.press(two)
+    expect(queryAllByA11yState(checked)).toHaveLength(2)
+    expect(containsText(queryAllByA11yState(checked), 'Two')).toBeTruthy()
+    expect(containsText(queryAllByA11yState(checked), 'One')).toBeTruthy()
+
+    const three = getByText('Three')
+    await fireEvent.press(three)
+    expect(queryAllByA11yState(checked)).toHaveLength(3)
+    expect(containsText(queryAllByA11yState(checked), 'Three')).toBeTruthy()
+    expect(containsText(queryAllByA11yState(checked), 'Two')).toBeTruthy()
+    expect(containsText(queryAllByA11yState(checked), 'One')).toBeTruthy()
+
+    const four = getByText('Four')
+    await fireEvent.press(four)
+    expect(queryAllByA11yState(checked)).toHaveLength(4)
+    expect(containsText(queryAllByA11yState(checked), 'Four')).toBeTruthy()
+    expect(containsText(queryAllByA11yState(checked), 'Three')).toBeTruthy()
+    expect(containsText(queryAllByA11yState(checked), 'Two')).toBeTruthy()
+    expect(containsText(queryAllByA11yState(checked), 'One')).toBeTruthy()
+  })
+
+  it('Deselects if the selected item is pressed', async () => {
+    const { getByText, queryAllByA11yState } = render(
+      <Theme>
+        <CheckboxGroup items={items} />
+      </Theme>
+    )
+
+    expect(queryAllByA11yState(checked)).toHaveLength(0)
+
+    const three = getByText('Three')
+    await fireEvent.press(three)
+    expect(queryAllByA11yState(checked)).toHaveLength(1)
+    expect(containsText(queryAllByA11yState(checked), 'Three')).toBeTruthy()
+
+    await fireEvent.press(three)
+    expect(queryAllByA11yState(checked)).toHaveLength(0)
+    expect(containsText(queryAllByA11yState(checked), 'Three')).not.toBeTruthy()
+  })
+
+  it('Is accessible as checkboxes button', async () => {
+    const { queryAllByA11yRole, queryAllByA11yState } = render(
+      <Theme>
+        <CheckboxGroup items={items} />
+      </Theme>
+    )
+
+    expect(queryAllByA11yRole('radiogroup')).toHaveLength(0)
+    expect(queryAllByA11yRole('radio')).toHaveLength(0)
+
+    const checks = queryAllByA11yRole('checkbox')
+    expect(checks).toHaveLength(4)
+
+    expect(queryAllByA11yState(checked)).toHaveLength(0)
+    await fireEvent.press(checks[0])
+    expect(queryAllByA11yState(checked)).toHaveLength(1)
+  })
+
+  it('Auto-selects any provided initialCheckedIds', async () => {
+    const { getByText, queryAllByA11yState } = render(
+      <Theme>
+        <CheckboxGroup items={items} initialCheckedIds={['one', 'two']} />
+      </Theme>
+    )
+    expect(queryAllByA11yState(checked)).toHaveLength(2)
+    expect(containsText(queryAllByA11yState(checked), 'One')).toBeTruthy()
+    expect(containsText(queryAllByA11yState(checked), 'Two')).toBeTruthy()
+
+    const three = getByText('Three')
+    await fireEvent.press(three)
+
+    expect(queryAllByA11yState(checked)).toHaveLength(3)
+    expect(containsText(queryAllByA11yState(checked), 'Three')).toBeTruthy()
+    expect(containsText(queryAllByA11yState(checked), 'Two')).toBeTruthy()
+    expect(containsText(queryAllByA11yState(checked), 'One')).toBeTruthy()
+  })
+})
+
+describe('CheckboxGroup (controlled)', () => {
+  it('Calls onChange handler on press in controlled mode, providing pressed id', async () => {
+    const handleChange = jest.fn((arg) => arg)
+    const { getByText } = render(
+      <Theme>
+        <CheckboxGroup items={items} checkedIds={[]} onChange={handleChange} />
+      </Theme>
+    )
+
+    expect(handleChange).toHaveBeenCalledTimes(0)
+
+    const one = getByText('One')
+    await fireEvent.press(one)
+    expect(handleChange).toHaveBeenCalledTimes(1)
+
+    expect(handleChange.mock.calls[0][0]).toEqual(['one'])
+
+    const two = getByText('Two')
+    await fireEvent.press(two)
+    expect(handleChange).toHaveBeenCalledTimes(2)
+    expect(handleChange.mock.calls[1][0]).toEqual(['two'])
+  })
+
+  it("Doesn't change its own selection if `checkedIds` is passed", async () => {
+    const { getByText, queryAllByA11yState } = render(
+      <Theme>
+        <CheckboxGroup items={items} checkedIds={['one']} onChange={() => {}} />
+      </Theme>
+    )
+
+    expect(queryAllByA11yState(checked)).toHaveLength(1)
+    expect(containsText(queryAllByA11yState(checked), 'One')).toBeTruthy()
+
+    const one = getByText('One')
+    await fireEvent.press(one)
+    expect(queryAllByA11yState(checked)).toHaveLength(1)
+    expect(containsText(queryAllByA11yState(checked), 'One')).toBeTruthy()
+
+    const two = getByText('Two')
+    await fireEvent.press(two)
+    expect(queryAllByA11yState(checked)).toHaveLength(1)
+    expect(containsText(queryAllByA11yState(checked), 'Two')).not.toBeTruthy()
+    expect(containsText(queryAllByA11yState(checked), 'One')).toBeTruthy()
+  })
+})
+
+describe('CheckboxGroup (read-only)', () => {
+  it("Doesn't call onChange handler when read-only", async () => {
+    const handleChange = jest.fn((arg) => arg)
+    const { getByText } = render(
+      <Theme>
+        <CheckboxGroup items={items} checkedIds={[]} onChange={handleChange} readOnly />
+      </Theme>
+    )
+
+    expect(handleChange).toHaveBeenCalledTimes(0)
+
+    const one = getByText('One')
+    await fireEvent.press(one)
+    expect(handleChange).toHaveBeenCalledTimes(0)
+
+    const two = getByText('Two')
+    await fireEvent.press(two)
+    expect(handleChange).toHaveBeenCalledTimes(0)
+  })
+
+  it("Doesn't change its selection on press", async () => {
+    const { getByText, queryAllByA11yState } = render(
+      <Theme>
+        <CheckboxGroup items={items} checkedIds={['one']} readOnly />
+      </Theme>
+    )
+
+    expect(queryAllByA11yState(checked)).toHaveLength(1)
+    expect(containsText(queryAllByA11yState(checked), 'One')).toBeTruthy()
+
+    const one = getByText('One')
+    await fireEvent.press(one)
+    expect(queryAllByA11yState(checked)).toHaveLength(1)
+    expect(containsText(queryAllByA11yState(checked), 'One')).toBeTruthy()
+
+    const two = getByText('Two')
+    await fireEvent.press(two)
+    expect(queryAllByA11yState(checked)).toHaveLength(1)
+    expect(containsText(queryAllByA11yState(checked), 'Two')).not.toBeTruthy()
+    expect(containsText(queryAllByA11yState(checked), 'One')).toBeTruthy()
+  })
+})

package/__tests__/Divider/Divider.test.jsx

@@ -41,17 +41,38 @@ describe('Divider', () => {
     expect(divider).not.toHaveStyle(verticalStyles)
   })
 
-  it('renders a vertical line when vertical prop is passed', () => {
+  it('renders space above and below a horizontal line if space prop is passed', () => {
+    const expectedSize = 8 // from testTheme.js
+
     const { getByTestId } = render(
       <Theme>
         <Text>Before</Text>
-        <Divider vertical testID={testID} />
+        <Divider testID={testID} space={2} />
         <Text>After</Text>
       </Theme>
     )
-    const divider = getByTestId(testID)
-    expect(divider).toHaveStyle(verticalStyles)
-    expect(divider).not.toHaveStyle(horizontalStyles)
+    const before = getByTestId(`${testID}-Spacer-before`)
+    expect(before).toHaveStyle({ height: expectedSize })
+
+    const after = getByTestId(`${testID}-Spacer-after`)
+    expect(after).toHaveStyle({ height: expectedSize })
+  })
+
+  it('renders space left and right of a vertical line if space prop is passed', () => {
+    const expectedSize = 8 // from testTheme.js
+
+    const { getByTestId } = render(
+      <Theme>
+        <Text>Before</Text>
+        <Divider vertical testID={testID} space={2} />
+        <Text>After</Text>
+      </Theme>
+    )
+    const before = getByTestId(`${testID}-Spacer-before`)
+    expect(before).toHaveStyle({ width: expectedSize })
+
+    const after = getByTestId(`${testID}-Spacer-after`)
+    expect(after).toHaveStyle({ width: expectedSize })
   })
 
   it('renders the correct styles for a given variant', () => {

package/__tests__/Feedback/Feedback.test.jsx

@@ -0,0 +1,42 @@
+import React from 'react'
+import { Text } from 'react-native'
+import { render } from '@testing-library/react-native'
+
+import Feedback from '../../src/Feedback'
+import Theme from '../../__fixtures__/Theme'
+
+describe('Feedback', () => {
+  it('renders title', () => {
+    const { queryByText } = render(<Feedback title="Test title" />, { wrapper: Theme })
+
+    expect(queryByText('Test title')).toBeTruthy()
+  })
+
+  it('renders content when passed a string', () => {
+    const { queryByText } = render(<Feedback>Test content</Feedback>, { wrapper: Theme })
+
+    expect(queryByText('Test content')).toBeTruthy()
+  })
+
+  it('renders content when passed a React element', () => {
+    const { queryByText } = render(
+      <Feedback>
+        <Text>Test content</Text>
+      </Feedback>,
+      { wrapper: Theme }
+    )
+
+    expect(queryByText('Test content')).toBeTruthy()
+  })
+
+  it("renders content with theme's style prop when passed a function", () => {
+    const { getByText } = render(
+      <Feedback>{({ textStyles }) => <Text style={textStyles}>Test content</Text>}</Feedback>,
+      {
+        wrapper: Theme
+      }
+    )
+
+    expect(getByText('Test content')).toHaveStyle({ fontSize: 14 })
+  })
+})

package/__tests__/Icon/Icon.test.jsx

@@ -16,7 +16,7 @@ describe('Icon', () => {
   it('gets default styles from theme', () => {
     const { getByTestId } = render(
       <Theme>
-        <Icon IconSvg={InnerComponent} />
+        <Icon icon={InnerComponent} />
       </Theme>
     )
 
@@ -29,7 +29,7 @@ describe('Icon', () => {
   it('gets styles from variant', () => {
     const { getByTestId } = render(
       <Theme>
-        <Icon IconSvg={InnerComponent} variant={{ size: 'small' }} />
+        <Icon icon={InnerComponent} variant={{ size: 'small' }} />
       </Theme>
     )
 
@@ -46,7 +46,7 @@ describe('Icon', () => {
     const props = { tokens: { size: 30 } }
     const { getByTestId } = render(
       <Theme>
-        <Icon IconSvg={InnerComponent} {...props} />
+        <Icon icon={InnerComponent} {...props} />
       </Theme>
     )
 

package/__tests__/InputSupports/InputSupports.test.jsx

@@ -0,0 +1,50 @@
+import React from 'react'
+import { fireEvent, render } from '@testing-library/react-native'
+
+import InputSupports from '../../src/InputSupports'
+import Theme from '../../__fixtures__/Theme'
+import Viewport from '../../__fixtures__/Viewport'
+
+// eslint-disable-next-line react/prop-types
+const Wrapper = ({ children }) => (
+  <Viewport viewport="xs">
+    <Theme>{children}</Theme>
+  </Viewport>
+)
+
+describe('InputSupports', () => {
+  it('renders the label', () => {
+    const { queryByText } = render(<InputSupports label="Test label content" />, {
+      wrapper: Wrapper
+    })
+
+    expect(queryByText('Test label content')).toBeTruthy()
+  })
+
+  it('renders the hint', () => {
+    const { queryByText } = render(
+      <InputSupports label="Test label content" hint="Test hint content" />,
+      { wrapper: Wrapper }
+    )
+
+    expect(queryByText('Test hint content')).toBeTruthy()
+  })
+
+  it('renders the feedback', () => {
+    const { queryByText } = render(<InputSupports feedback="Test feedback content" />, {
+      wrapper: Wrapper
+    })
+
+    expect(queryByText('Test feedback content')).toBeTruthy()
+  })
+
+  it('renders the tooltip which content shows up when pressed', () => {
+    const { queryByText, getByRole } = render(
+      <InputSupports label="Test label content" tooltip="Test tooltip content" />,
+      { wrapper: Wrapper }
+    )
+
+    fireEvent(getByRole('button'), 'press')
+    expect(queryByText('Test tooltip content')).toBeTruthy()
+  })
+})

package/__tests__/List/List.test.jsx

@@ -0,0 +1,60 @@
+import React from 'react'
+import { Text } from 'react-native'
+import { render } from '@testing-library/react-native'
+import { List } from '../../src'
+import Theme from '../../__fixtures__/Theme'
+import testTheme from '../../__fixtures__/testTheme'
+
+const setup = ({ showDivider = false, icon, children } = {}) =>
+  render(
+    <Theme>
+      <List showDivider={showDivider}>
+        <List.Item icon={icon}>{children || 'Item'}</List.Item>
+        <List.Item icon={icon}>{children || 'Item'}</List.Item>
+        <List.Item icon={icon}>{children || 'Item'}</List.Item>
+        <List.Item icon={icon}>{children || 'Item'}</List.Item>
+      </List>
+    </Theme>
+  )
+
+describe('List', () => {
+  it('should render bullets by default', () => {
+    const { queryAllByTestId } = setup()
+    const bullet = queryAllByTestId('unordered-item-bullet')
+    expect(bullet.length).toBe(4)
+  })
+
+  it('should render dividers for all items except for the last one', () => {
+    const { getAllByRole } = setup({ showDivider: true })
+    const {
+      interItemMarginWithDivider: marginBottom,
+      dividerColor: borderColor,
+      dividerSize: borderBottomWidth
+    } = testTheme.components.List.tokens
+
+    const listItems = getAllByRole('item')
+
+    listItems.forEach((item, index) => {
+      if (index + 1 !== listItems.length) {
+        expect(item).toHaveStyle({ marginBottom, borderColor, borderBottomWidth })
+      } else {
+        expect(item).toHaveStyle({})
+      }
+    })
+  })
+
+  it('should render the icon when defined', () => {
+    const { queryAllByTestId } = setup({ icon: () => <Text testID="icon" /> })
+
+    const icons = queryAllByTestId('icon')
+
+    expect(icons.length).toBe(4)
+  })
+  it('should allow children nested components', () => {
+    const { queryAllByTestId } = setup({ children: <Text testID="child" /> })
+
+    const children = queryAllByTestId('child')
+
+    expect(children.length).toBe(4)
+  })
+})