@coreui/react 5.4.1 → 5.6.0

package/src/components/alert/__tests__/CAlert.spec.tsx

@@ -1,41 +1,62 @@
 import * as React from 'react'
-import { render, fireEvent } from '@testing-library/react'
+import { act, render, fireEvent } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CAlert } from '../../../index'
+import { CAlert } from '../index'
 
 test('loads and displays CAlert component', async () => {
-  const { container } = render(<CAlert color="primary">Test</CAlert>)
-  expect(container).toMatchSnapshot()
+  let container: HTMLElement
+  await act(async () => {
+    const renderResult = render(<CAlert color="primary">Test</CAlert>)
+    container = renderResult.container
+  })
+  expect(container!).toMatchSnapshot()
 })
 
 test('CAlert customize', async () => {
-  const { container } = render(
-    <CAlert color="secondary" className="bazinga" dismissible={true} variant="solid" visible={true}>
-      Test
-    </CAlert>,
-  )
-  expect(container).toMatchSnapshot()
-  expect(container.firstChild).toHaveClass('bazinga')
-  expect(container.firstChild).toHaveClass('bg-secondary')
-  expect(container.firstChild).toHaveClass('text-white')
-  expect(container.firstChild).toHaveClass('alert-dismissible')
+  let container: HTMLElement
+  await act(async () => {
+    const renderResult = render(
+      <CAlert
+        color="secondary"
+        className="bazinga"
+        dismissible={true}
+        variant="solid"
+        visible={true}
+      >
+        Test
+      </CAlert>
+    )
+    container = renderResult.container
+  })
+  expect(container!).toMatchSnapshot()
+  expect(container!.firstChild).toHaveClass('bazinga')
+  expect(container!.firstChild).toHaveClass('bg-secondary')
+  expect(container!.firstChild).toHaveClass('text-white')
+  expect(container!.firstChild).toHaveClass('alert-dismissible')
 })
 
 test('CAlert click close button', async () => {
   jest.useFakeTimers()
+
   const onClose = jest.fn()
   render(
     <CAlert color="primary" dismissible onClose={onClose}>
       Test
-    </CAlert>,
+    </CAlert>
   )
+
   expect(onClose).toHaveBeenCalledTimes(0)
+
   const btn = document.querySelector('.btn-close')
+
   if (btn !== null) {
-    fireEvent.click(btn)
+    act(() => {
+      fireEvent.click(btn)
+      jest.runAllTimers()
+    })
   }
+
   expect(onClose).toHaveBeenCalledTimes(1)
-  jest.runAllTimers()
-  expect(onClose).toHaveBeenCalledTimes(1)
+
   jest.useRealTimers()
 })

package/src/components/alert/__tests__/CAlertHeading.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CAlertHeading } from '../../../index'
+import { CAlertHeading } from '../index'
 
 test('loads and displays CAlertHeading component', async () => {
   const { container } = render(<CAlertHeading>Test</CAlertHeading>)

package/src/components/alert/__tests__/CAlertLink.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CAlertLink } from '../../../index'
+import { CAlertLink } from '../index'
 
 test('loads and displays CAlertLink component', async () => {
   const { container } = render(<CAlertLink>Test</CAlertLink>)

package/src/components/avatar/__tests__/CAvatar.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CAvatar } from '../../../index'
+import { CAvatar } from '../index'
 
 test('loads and displays CAvatar component', async () => {
   const { container } = render(<CAvatar color="primary">Test</CAvatar>)

package/src/components/backdrop/__tests__/CBackdrop.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CBackdrop } from '../../../index'
+import { CBackdrop } from '../index'
 
 test('loads and displays CBackdrop component', async () => {
   const { container } = render(<CBackdrop>Test</CBackdrop>)

package/src/components/badge/CBadge.tsx

@@ -96,7 +96,7 @@ export const CBadge: PolymorphicRefForwardingComponent<'span', CBadgeProps> = fo
 )
 
 CBadge.propTypes = {
-  as: PropTypes.string,
+  as: PropTypes.elementType,
   children: PropTypes.node,
   className: PropTypes.string,
   color: colorPropType,

package/src/components/badge/__tests__/CBadge.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CBadge } from '../../../index'
+import { CBadge } from '../index'
 
 test('loads and displays CBadge component', async () => {
   const { container } = render(<CBadge color="primary">Test</CBadge>)

package/src/components/breadcrumb/__tests__/CBreadcrumb.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CBreadcrumb, CBreadcrumbItem } from '../../../index'
+import { CBreadcrumb, CBreadcrumbItem } from '../index'
 
 test('loads and displays CBreadcrumb component', async () => {
   const { container } = render(<CBreadcrumb></CBreadcrumb>)

package/src/components/breadcrumb/__tests__/CBreadcrumbItem.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CBreadcrumbItem } from '../../../index'
+import { CBreadcrumbItem } from '../index'
 
 test('loads and displays CBreadcrumbItem component', async () => {
   const { container } = render(<CBreadcrumbItem>Test</CBreadcrumbItem>)

package/src/components/button/__tests__/CButton.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CButton } from '../../../index'
+import { CButton } from '../index'
 
 test('loads and displays CButton component', async () => {
   const { container } = render(<CButton color="primary">Test</CButton>)

package/src/components/button-group/__tests__/CButtonGroup.spec.tsx

@@ -1,7 +1,8 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CButtonGroup, CButton } from '../../../index'
+import { CButton } from '../../button/CButton'
+import { CButtonGroup } from '../index'
 
 test('loads and displays CButtonGroup component', async () => {
   const { container } = render(<CButtonGroup></CButtonGroup>)

package/src/components/button-group/__tests__/CButtonToolbar.spec.tsx

@@ -1,7 +1,8 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CButtonToolbar, CButtonGroup, CButton } from '../../../index'
+import { CButton } from '../../button/CButton'
+import { CButtonToolbar, CButtonGroup } from '../index'
 
 test('loads and displays CButtonToolbar component', async () => {
   const { container } = render(<CButtonToolbar></CButtonToolbar>)

package/src/components/callout/__tests__/CCallout.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CCallout } from '../../../index'
+import { CCallout } from '../index'
 
 test('loads and displays CCallout component', async () => {
   const { container } = render(<CCallout>Test</CCallout>)

package/src/components/card/__tests__/CCard.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CCard } from '../../../index'
+import { CCard } from '../index'
 
 test('loads and displays CCard component', async () => {
   const { container } = render(<CCard>Test</CCard>)

package/src/components/card/__tests__/CCardBody.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CCardBody } from '../../../index'
+import { CCardBody } from '../index'
 
 test('loads and displays CCardBody component', async () => {
   const { container } = render(<CCardBody>Test</CCardBody>)

package/src/components/card/__tests__/CCardFooter.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CCardFooter } from '../../../index'
+import { CCardFooter } from '../index'
 
 test('loads and displays CCardFooter component', async () => {
   const { container } = render(<CCardFooter>Test</CCardFooter>)

package/src/components/card/__tests__/CCardGroup.spec.tsx

@@ -12,7 +12,7 @@ import {
   CCardTitle,
   CCardText,
   CCardGroup,
-} from '../../../index'
+} from '../index'
 
 test('loads and displays CCardGroup component', async () => {
   const { container } = render(<CCardGroup>Test</CCardGroup>)

package/src/components/card/__tests__/CCardHeader.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CCardHeader } from '../../../index'
+import { CCardHeader } from '../index'
 
 test('loads and displays CCardHeader component', async () => {
   const { container } = render(<CCardHeader>Test</CCardHeader>)

package/src/components/card/__tests__/CCardImage.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CCardImage } from '../../../index'
+import { CCardImage } from '../index'
 
 test('loads and displays CCardImage component', async () => {
   const { container } = render(<CCardImage />)

package/src/components/card/__tests__/CCardImageOverlay.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CCardImageOverlay } from '../../../index'
+import { CCardImageOverlay } from '../index'
 
 test('loads and displays CCardImageOverlay component', async () => {
   const { container } = render(<CCardImageOverlay />)

package/src/components/card/__tests__/CCardLink.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CCardLink } from '../../../index'
+import { CCardLink } from '../index'
 
 test('loads and displays CCardLink component', async () => {
   const { container } = render(<CCardLink>Test</CCardLink>)

package/src/components/card/__tests__/CCardSubtitle.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CCardSubtitle } from '../../../index'
+import { CCardSubtitle } from '../index'
 
 test('loads and displays CCardSubtitle component', async () => {
   const { container } = render(<CCardSubtitle>Test</CCardSubtitle>)

package/src/components/card/__tests__/CCardText.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CCardText } from '../../../index'
+import { CCardText } from '../index'
 
 test('loads and displays CCardText component', async () => {
   const { container } = render(<CCardText>Test</CCardText>)

package/src/components/card/__tests__/CCardTitle.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CCardTitle } from '../../../index'
+import { CCardTitle } from '../index'
 
 test('loads and displays CCardTitle component', async () => {
   const { container } = render(<CCardTitle>Test</CCardTitle>)

package/src/components/carousel/CCarousel.tsx

@@ -1,6 +1,5 @@
 import React, {
   Children,
-  createContext,
   forwardRef,
   HTMLAttributes,
   TouchEvent,
@@ -14,6 +13,8 @@ import classNames from 'classnames'
 import { isInViewport } from '../../utils'
 import { useForkedRef } from '../../hooks'
 
+import { CCarouselContext } from './CCarouselContext'
+
 export interface CCarouselProps extends HTMLAttributes<HTMLDivElement> {
   /**
    * index of the active item.
@@ -71,13 +72,6 @@ interface DataType {
   timeout?: null | ReturnType<typeof setTimeout>
 }
 
-export interface ContextProps {
-  setAnimating: (a: boolean) => void
-  setCustomInterval: (a: boolean | number) => void
-}
-
-export const CCarouselContext = createContext({} as ContextProps)
-
 export const CCarousel = forwardRef<HTMLDivElement, CCarouselProps>(
   (
     {
@@ -96,7 +90,7 @@ export const CCarousel = forwardRef<HTMLDivElement, CCarouselProps>(
       wrap = true,
       ...rest
     },
-    ref,
+    ref
   ) => {
     const carouselRef = useRef<HTMLDivElement>(null)
     const forkedRef = useForkedRef(ref, carouselRef)
@@ -141,7 +135,7 @@ export const CCarousel = forwardRef<HTMLDivElement, CCarouselProps>(
       if (typeof interval === 'number') {
         data.timeout = setTimeout(
           () => nextItemWhenVisible(),
-          typeof customInterval === 'number' ? customInterval : interval,
+          typeof customInterval === 'number' ? customInterval : interval
         )
       }
     }
@@ -228,7 +222,7 @@ export const CCarousel = forwardRef<HTMLDivElement, CCarouselProps>(
           {
             'carousel-fade': transition === 'crossfade',
           },
-          className,
+          className
         )}
         {...(dark && { 'data-coreui-theme': 'dark' })}
         onMouseEnter={_pause}
@@ -288,7 +282,7 @@ export const CCarousel = forwardRef<HTMLDivElement, CCarouselProps>(
         </CCarouselContext.Provider>
       </div>
     )
-  },
+  }
 )
 
 CCarousel.propTypes = {

package/src/components/carousel/CCarouselCaption.tsx

@@ -12,7 +12,7 @@ export interface CCarouselCaptionProps extends HTMLAttributes<HTMLDivElement> {
 export const CCarouselCaption = forwardRef<HTMLDivElement, CCarouselCaptionProps>(
   ({ className, ...rest }, ref) => {
     return <div className={classNames('carousel-caption', className)} {...rest} ref={ref} />
-  },
+  }
 )
 
 CCarouselCaption.propTypes = {

package/src/components/carousel/CCarouselContext.ts

@@ -0,0 +1,8 @@
+import { createContext } from 'react'
+
+export interface CCarouselContextProps {
+  setAnimating: (a: boolean) => void
+  setCustomInterval: (a: boolean | number) => void
+}
+
+export const CCarouselContext = createContext({} as CCarouselContextProps)

package/src/components/carousel/CCarouselItem.tsx

@@ -3,7 +3,9 @@ import PropTypes from 'prop-types'
 import classNames from 'classnames'
 
 import { useForkedRef } from '../../hooks'
-import { CCarouselContext } from './CCarousel'
+
+import { CCarouselContext } from './CCarouselContext'
+
 export interface CCarouselItemProps extends HTMLAttributes<HTMLDivElement> {
   /**
    * @ignore
@@ -29,7 +31,7 @@ export const CCarouselItem = forwardRef<HTMLDivElement, CCarouselItemProps>(
     const carouselItemRef = useRef<HTMLDivElement>(null)
     const forkedRef = useForkedRef(ref, carouselItemRef)
 
-    const prevActive = useRef<boolean>()
+    const prevActive = useRef<boolean>(undefined)
     const [directionClassName, setDirectionClassName] = useState<string>()
     const [orderClassName, setOrderClassName] = useState<string>()
     const [activeClassName, setActiveClassName] = useState(active && 'active')
@@ -101,7 +103,7 @@ export const CCarouselItem = forwardRef<HTMLDivElement, CCarouselItemProps>(
           activeClassName,
           directionClassName,
           orderClassName,
-          className,
+          className
         )}
         ref={forkedRef}
         {...rest}
@@ -109,7 +111,7 @@ export const CCarouselItem = forwardRef<HTMLDivElement, CCarouselItemProps>(
         {children}
       </div>
     )
-  },
+  }
 )
 
 CCarouselItem.propTypes = {

package/src/components/carousel/__tests__/CCarousel.spec.tsx

@@ -1,8 +1,7 @@
 import React from 'react'
-import { render, fireEvent } from '@testing-library/react'
-import { getByText } from '@testing-library/dom'
+import { render, fireEvent, getByText } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CCarousel, CCarouselCaption, CCarouselItem } from '../../../index'
+import { CCarousel, CCarouselCaption, CCarouselItem } from '../index'
 
 test('loads and displays CCarousel component', async () => {
   const { container } = render(

package/src/components/close-button/__tests__/CCloseButton.spec.tsx

@@ -1,7 +1,7 @@
 import * as React from 'react'
 import { render } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CCloseButton } from '../../../index'
+import { CCloseButton } from '../index'
 
 test('loads and displays CCloseButton component', async () => {
   const { container } = render(<CCloseButton />)

package/src/components/collapse/__tests__/CCollapse.spec.tsx

@@ -1,7 +1,7 @@
 import React from 'react'
-import { render, screen } from '@testing-library/react'
+import { render, screen, act } from '@testing-library/react'
 import '@testing-library/jest-dom'
-import { CCollapse } from '../../../index'
+import { CCollapse } from '../index'
 
 test('loads and displays CCollapse component', async () => {
   const { container } = render(<CCollapse>Test</CCollapse>)
@@ -15,26 +15,45 @@ test('CCollapse customize', async () => {
 })
 
 test('CCollapse use case test', async () => {
+  jest.useFakeTimers()
+
   const { rerender } = render(<CCollapse visible={false}>Test</CCollapse>)
+
   expect(screen.getByText('Test')).toHaveClass('collapse')
   expect(screen.getByText('Test')).not.toHaveClass('show')
   expect(screen.getByText('Test')).not.toHaveClass('collapsing')
-  rerender(<CCollapse visible={true}>Test</CCollapse>)
+
+  act(() => {
+    rerender(<CCollapse visible={true}>Test</CCollapse>)
+  })
+
   expect(screen.getByText('Test')).not.toHaveClass('collapse')
   expect(screen.getByText('Test')).not.toHaveClass('show')
   expect(screen.getByText('Test')).toHaveClass('collapsing')
-  await new Promise((r) => setTimeout(r, 1000))
+
+  act(() => {
+    jest.runAllTimers()
+  })
+
   expect(screen.getByText('Test')).toHaveClass('collapse')
   expect(screen.getByText('Test')).toHaveClass('show')
   expect(screen.getByText('Test')).not.toHaveClass('collapsing')
-  rerender(<CCollapse visible={false}>Test</CCollapse>)
+
+  act(() => {
+    rerender(<CCollapse visible={false}>Test</CCollapse>)
+  })
+
   expect(screen.getByText('Test')).not.toHaveClass('collapse')
   expect(screen.getByText('Test')).not.toHaveClass('show')
   expect(screen.getByText('Test')).toHaveClass('collapsing')
-  await new Promise((r) => setTimeout(r, 1000))
+
+  act(() => {
+    jest.runAllTimers()
+  })
+
   expect(screen.getByText('Test')).toHaveClass('collapse')
   expect(screen.getByText('Test')).not.toHaveClass('show')
   expect(screen.getByText('Test')).not.toHaveClass('collapsing')
-  jest.runAllTimers()
+
   jest.useRealTimers()
 })