Migrate Breadcrumb to TypeScript

.changeset/proud-lamps-marry.md

@@ -0,0 +1,5 @@
+---
+"@primer/components": patch
+---
+
+Migrate `Breadcrumb` to TypeScript

package.json

@@ -46,6 +46,7 @@
     "@styled-system/prop-types": "5.1.2",
     "@styled-system/props": "5.1.4",
     "@styled-system/theme-get": "5.1.2",
+    "@types/history": "4.7.8",
     "@types/styled-components": "5.1.7",
     "@types/styled-system": "5.1.10",
     "@types/styled-system__css": "5.0.14",

src/Breadcrumb.js → src/Breadcrumb.tsx

@@ -1,11 +1,13 @@
-import React from 'react'
-import PropTypes from 'prop-types'
 import classnames from 'classnames'
+import * as History from 'history'
+import PropTypes from 'prop-types'
+import React from 'react'
 import styled from 'styled-components'
-import sx from './sx'
-import {COMMON, FLEX, get} from './constants'
-import theme from './theme'
 import Box from './Box'
+import {COMMON, FLEX, get, SystemCommonProps, SystemFlexProps} from './constants'
+import sx, {SxProp} from './sx'
+import theme from './theme'
+import {ComponentProps} from './utils/types'
 
 const SELECTED_CLASS = 'selected'
 
@@ -30,31 +32,39 @@ const Wrapper = styled.li`
   }
 `
 
-const BreadcrumbBase = ({className, children, theme, ...rest}) => {
+const BreadcrumbBase = styled.nav<SystemFlexProps & SystemCommonProps & SxProp>`
+  display: flex;
+  justify-content: space-between;
+  ${COMMON};
+  ${FLEX};
+  ${sx};
+`
+
+export type BreadcrumbProps = ComponentProps<typeof BreadcrumbBase>
+
+function Breadcrumb({className, children, theme, ...rest}: React.PropsWithChildren<BreadcrumbProps>) {
   const classes = classnames(className, 'Breadcrumb')
   const wrappedChildren = React.Children.map(children, child => <Wrapper theme={theme}>{child}</Wrapper>)
   return (
-    <nav className={classes} aria-label="breadcrumb" {...rest}>
+    <BreadcrumbBase className={classes} aria-label="breadcrumb" theme={theme} {...rest}>
       <Box as="ol" my={0} pl={0}>
         {wrappedChildren}
       </Box>
-    </nav>
+    </BreadcrumbBase>
   )
 }
 
-const Breadcrumb = styled(BreadcrumbBase)`
-  display: flex;
-  justify-content: space-between;
-  ${COMMON};
-  ${FLEX};
-  ${sx};
-`
+type StyledBreadcrumbItemProps = {
+  to?: History.LocationDescriptor
+  selected?: boolean
+} & SystemCommonProps &
+  SxProp
 
-Breadcrumb.Item = styled.a.attrs(props => ({
+const BreadcrumbItem = styled.a.attrs<StyledBreadcrumbItemProps>(props => ({
   activeClassName: typeof props.to === 'string' ? 'selected' : '',
   className: classnames(props.selected && SELECTED_CLASS, props.className),
   'aria-current': props.selected ? 'page' : null
-}))`
+}))<StyledBreadcrumbItemProps>`
   color: ${get('colors.blue.5')};
   display: inline-block;
   font-size: ${get('fontSizes.1')};
@@ -81,18 +91,18 @@ Breadcrumb.propTypes = {
 
 Breadcrumb.displayName = 'Breadcrumb'
 
-Breadcrumb.Item.defaultProps = {
+BreadcrumbItem.defaultProps = {
   theme
 }
 
-Breadcrumb.Item.propTypes = {
-  as: PropTypes.oneOfType([PropTypes.string, PropTypes.elementType]),
+BreadcrumbItem.propTypes = {
   href: PropTypes.string,
   selected: PropTypes.bool,
   ...sx.propTypes,
   ...COMMON.propTypes
 }
 
-Breadcrumb.Item.displayName = 'Breadcrumb.Item'
+BreadcrumbItem.displayName = 'Breadcrumb.Item'
 
-export default Breadcrumb
+export type BreadcrumbItemProps = ComponentProps<typeof BreadcrumbItem>
+export default Object.assign(Breadcrumb, {Item: BreadcrumbItem})

src/__tests__/BreadcrumbItem.js → src/__tests__/BreadcrumbItem.tsx

@@ -1,10 +1,10 @@
+import {cleanup, render as HTMLRender} from '@testing-library/react'
+import 'babel-polyfill'
+import {axe, toHaveNoViolations} from 'jest-axe'
 import React from 'react'
 import {Breadcrumb} from '..'
-import {render, behavesAsComponent} from '../utils/testing'
 import {COMMON} from '../constants'
-import {render as HTMLRender, cleanup} from '@testing-library/react'
-import {axe, toHaveNoViolations} from 'jest-axe'
-import 'babel-polyfill'
+import {behavesAsComponent, render} from '../utils/testing'
 expect.extend(toHaveNoViolations)
 
 describe('Breadcrumb.Item', () => {
@@ -26,9 +26,7 @@ describe('Breadcrumb.Item', () => {
   })
 
   it('adds activeClassName={SELECTED_CLASS} when it gets a "to" prop', () => {
-    const Mock = jest.fn(() => <div />)
-    render(<Breadcrumb.Item as={Mock} to="#" />)
-    expect(Mock.mock.calls[0][0].to).toEqual('#')
-    expect(Mock.mock.calls[0][0].activeClassName).toEqual('selected')
+    const Link = ({theme, ...props}: any) => <div {...props} />
+    expect(render(<Breadcrumb.Item as={Link} to="#" />)).toMatchSnapshot()
   })
 })

src/__tests__/__snapshots__/BreadcrumbItem.js.snap → src/__tests__/__snapshots__/BreadcrumbItem.tsx.snap

@@ -1,5 +1,32 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`Breadcrumb.Item adds activeClassName={SELECTED_CLASS} when it gets a "to" prop 1`] = `
+.c0 {
+  color: #0366d6;
+  display: inline-block;
+  font-size: 14px;
+  -webkit-text-decoration: none;
+  text-decoration: none;
+}
+
+.c0:hover {
+  -webkit-text-decoration: underline;
+  text-decoration: underline;
+}
+
+.c0.selected {
+  color: #444d56;
+  pointer-events: none;
+}
+
+<div
+  activeClassName="selected"
+  aria-current={null}
+  className="c0"
+  to="#"
+/>
+`;
+
 exports[`Breadcrumb.Item renders consistently 1`] = `
 .c0 {
   color: #0366d6;

yarn.lock

@@ -1820,6 +1820,11 @@
   dependencies:
     "@types/node" "*"
 
+"@types/history@4.7.8":
+  version "4.7.8"
+  resolved "https://registry.yarnpkg.com/@types/history/-/history-4.7.8.tgz#49348387983075705fe8f4e02fb67f7daaec4934"
+  integrity sha512-S78QIYirQcUoo6UJZx9CSP0O2ix9IaeAXwQi26Rhr/+mg7qqPy8TzaxHSUut7eGjL8WmLccT7/MXf304WjqHcA==
+
 "@types/hoist-non-react-statics@*":
   version "3.3.1"
   resolved "https://registry.yarnpkg.com/@types/hoist-non-react-statics/-/hoist-non-react-statics-3.3.1.tgz#1124aafe5118cb591977aeb1ceaaed1070eb039f"