cozy-ui 110.8.1 → 110.8.2

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [110.8.2](https://github.com/cozy/cozy-ui/compare/v110.8.1...v110.8.2) (2024-07-11)
+
+
+### Bug Fixes
+
+* **ViewerControls:** children can contain undefined values ([506fd4c](https://github.com/cozy/cozy-ui/commit/506fd4c))
+
 ## [110.8.1](https://github.com/cozy/cozy-ui/compare/v110.8.0...v110.8.1) (2024-07-10)
 
 

package/jest.config.js

@@ -14,7 +14,9 @@ module.exports = {
     'react-pdf/dist/esm/entry.webpack': 'react-pdf',
     '^cozy-client$': 'cozy-client/dist/index'
   },
-  transformIgnorePatterns: ['node_modules/(?!(react-styleguidist)/)'],
+  transformIgnorePatterns: [
+    'node_modules/(?!(react-styleguidist|cozy-harvest-lib)/)'
+  ],
   testPathIgnorePatterns: ['/node_modules/', '/transpiled/', '/dist/'],
   transform: {
     '^.+\\.(ts|tsx|js|jsx)?$': 'babel-jest'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-ui",
-  "version": "110.8.1",
+  "version": "110.8.2",
   "description": "Cozy apps UI SDK",
   "main": "./index.js",
   "bin": {

package/react/Viewer/components/ViewerControls.jsx

@@ -95,8 +95,8 @@ class ViewerControls extends Component {
 
     return React.Children.map(children, child => {
       if (
-        child.type.name === 'ViewerByFile' ||
-        child.type.displayName === 'ViewerByFile'
+        child?.type?.name === 'ViewerByFile' ||
+        child?.type?.displayName === 'ViewerByFile'
       ) {
         return React.cloneElement(child, {
           gestures: this.state.gestures,

package/react/Viewer/components/ViewerControls.spec.jsx

@@ -0,0 +1,54 @@
+import React from 'react'
+import { render, screen } from '@testing-library/react'
+
+import ViewerControls from './ViewerControls'
+import ViewerByFile from './ViewerByFile'
+
+jest.mock('../ViewersByFile/AudioViewer', () => () => <div>AudioViewer</div>)
+jest.mock('../providers/EncryptedProvider', () => ({
+  useEncrypted: () => ({ url: 'random' })
+}))
+
+describe('ViewerControls', () => {
+  const file = {
+    _id: 'audio',
+    class: 'audio',
+    mime: 'audio/mp3',
+    name: 'sample.mp3'
+  }
+
+  const setup = ({ children } = {}) => {
+    render(
+      <ViewerControls
+        file={file}
+        onClose={() => {}}
+        hasPrevious={false}
+        hasNext={false}
+        onPrevious={() => {}}
+        onNext={() => {}}
+        expanded={false}
+        toolbarProps={{
+          showToolbar: false,
+          showClose: false,
+          showFilePath: false,
+          toolbarRef: undefined
+        }}
+        showNavigation={false}
+      >
+        {children}
+      </ViewerControls>
+    )
+  }
+
+  it('should only render children if they are ViewerByFile', () => {
+    setup({
+      children: [
+        undefined,
+        <div key="notViewer">not ViewerByFile</div>,
+        <ViewerByFile key="viewer" file={file} onClose={() => {}} />
+      ]
+    })
+    expect(screen.queryByText('not ViewerByFile')).not.toBeInTheDocument()
+    expect(screen.getByText('AudioViewer')).toBeInTheDocument()
+  })
+})

package/transpiled/react/Viewer/components/ViewerControls.js

@@ -141,7 +141,9 @@ var ViewerControls = /*#__PURE__*/function (_Component) {
 
       if (!children) return null;
       return React.Children.map(children, function (child) {
-        if (child.type.name === 'ViewerByFile' || child.type.displayName === 'ViewerByFile') {
+        var _child$type, _child$type2;
+
+        if ((child === null || child === void 0 ? void 0 : (_child$type = child.type) === null || _child$type === void 0 ? void 0 : _child$type.name) === 'ViewerByFile' || (child === null || child === void 0 ? void 0 : (_child$type2 = child.type) === null || _child$type2 === void 0 ? void 0 : _child$type2.displayName) === 'ViewerByFile') {
           return /*#__PURE__*/React.cloneElement(child, {
             gestures: _this2.state.gestures,
             gesturesRef: _this2.wrapped,