Update snapshots and tests

Author: torkinos

package-lock.json

@@ -116,12 +116,19 @@
     "@types/react": "^19.2.4",
     "babel-jest": "29.7.0",
     "babel-plugin-macros": "3.1.0",
+    "eslint": "9.39.2",
     "eslint-config-expo": "8.0.1",
-    "tether-dev-docs": "file:packages/tether-dev-docs",
+    "eslint-config-prettier": "10.1.8",
+    "eslint-plugin-eslint-plugin": "6.5.0",
+    "eslint-plugin-import": "2.32.0",
+    "eslint-plugin-prettier": "5.5.4",
+    "eslint-plugin-react": "7.37.5",
     "husky": "^9.1.7",
     "jest": "29.7.0",
     "jest-environment-jsdom": "^30.2.0",
+    "prettier": "3.7.4",
     "react-test-renderer": "19.0.0",
+    "tether-dev-docs": "file:packages/tether-dev-docs",
     "typescript": "~5.8.3"
   },
   "private": true

package.json

@@ -62,6 +62,7 @@ exports[`CustomFields component renders correctly with note custom fields 1`] =
             "width": "100%",
           }
         }
+        testID="note-0"
       >
         <View
           style={
@@ -197,6 +198,7 @@ exports[`CustomFields component renders correctly with note custom fields 1`] =
                     },
                   ]
                 }
+                testID="note-0-input"
                 value="test-note-value"
               />
             </View>
@@ -381,6 +383,7 @@ exports[`CustomFields component renders correctly with note custom fields 1`] =
             "width": "100%",
           }
         }
+        testID="note-1"
       >
         <View
           style={
@@ -516,6 +519,7 @@ exports[`CustomFields component renders correctly with note custom fields 1`] =
                     },
                   ]
                 }
+                testID="note-1-input"
                 value="test-note-value"
               />
             </View>

src/components/CustomFields/__snapshots__/index.test.jsx.snap

@@ -145,6 +145,7 @@ exports[`Folder component renders create new folder option without count 1`] = `
       "paddingBottom": 16,
     }
   }
+  testID="sidebar-create-new"
 >
   <View
     style={

src/components/Folder/__snapshots__/index.test.jsx.snap

@@ -11,7 +11,7 @@ jest.mock('./styles', () => {
 
   return {
     ListItemContainer: (props) => (
-      <TouchableOpacity testID="vault-container" {...props}>
+      <TouchableOpacity {...props} testID="vault-container">
         {props.children}
       </TouchableOpacity>
     ),
@@ -29,17 +29,29 @@ jest.mock('./styles', () => {
         {props.children}
       </Text>
     ),
-    ListItemActions: (props) => <View testID="vault-actions" {...props} />
+    ListItemActions: (props) => <View testID="vault-actions" {...props} />,
+    SelectedListItemIconContainer: (props) => (
+      <View testID="selected-icon-container" {...props} />
+    )
   }
 })
 
 jest.mock('pearpass-lib-ui-react-native-components', () => ({
   BrushIcon: () => 'BrushIcon',
+  CheckIcon: () => 'CheckIcon',
   DeleteIcon: () => 'DeleteIcon',
   LockCircleIcon: () => 'LockCircleIcon',
   ShareIcon: () => 'ShareIcon'
 }))
 
+jest.mock('pearpass-lib-ui-theme-provider/native', () => ({
+  colors: {
+    primary400: { mode1: '#000000' },
+    black: { mode1: '#000000' },
+    white: { mode1: '#ffffff' }
+  }
+}))
+
 describe('ListItem', () => {
   const mockProps = {
     name: 'Test Vault',
@@ -65,15 +77,13 @@ describe('ListItem', () => {
     expect(mockProps.onPress).toHaveBeenCalled()
   })
 
-  it('renders actions when showActions is true', () => {
-    const { getByTestId } = render(
-      <ListItem {...mockProps} showActions={true} />
-    )
+  it('renders actions when action callbacks are provided', () => {
+    const { getByTestId } = render(<ListItem {...mockProps} />)
 
     expect(getByTestId('vault-actions')).toBeTruthy()
   })
 
-  it('does not render actions when showActions is false', () => {
+  it('does not render action icons when callbacks are not provided', () => {
     const mockWithoutActionsProps = {
       name: 'Test Vault',
       date: '2023-01-01',
@@ -82,8 +92,31 @@ describe('ListItem', () => {
 
     const { queryByTestId } = render(<ListItem {...mockWithoutActionsProps} />)
 
-    expect(queryByTestId('vault-actions')?.props.children[0]).toBeUndefined()
-    expect(queryByTestId('vault-actions')?.props.children[1]).toBeUndefined()
-    expect(queryByTestId('vault-actions')?.props.children[2]).toBeUndefined()
+    const actionsContainer = queryByTestId('vault-actions')
+    expect(actionsContainer).toBeTruthy()
+    // When no callbacks are provided, the actions container should have no meaningful children
+    // React filters out falsy values, so children should be undefined, null, or an array of falsy values
+    const children = actionsContainer.props.children
+    const hasNoChildren =
+      !children ||
+      (Array.isArray(children) && children.every((child) => !child)) ||
+      children === null
+    expect(hasNoChildren).toBeTruthy()
+  })
+
+  it('renders CheckIcon when isSelected is true', () => {
+    const { getByTestId } = render(
+      <ListItem {...mockProps} isSelected={true} />
+    )
+
+    expect(getByTestId('selected-icon-container')).toBeTruthy()
+  })
+
+  it('renders correctly when isLoading is true', () => {
+    const { getByTestId } = render(<ListItem {...mockProps} isLoading={true} />)
+
+    // Component should still render name and date when loading
+    expect(getByTestId('vault-name').props.children).toBe('Test Vault')
+    expect(getByTestId('vault-date')).toBeTruthy()
   })
 })

src/components/ListItem/index.test.jsx

@@ -28,7 +28,7 @@ describe('readFileContent', () => {
     const result = await readFileContent(['.csv'])
 
     expect(DocumentPicker.getDocumentAsync).toHaveBeenCalledWith({
-      type: ['text/csv']
+      type: ['text/csv', 'text/comma-separated-values']
     })
     expect(FileSystem.readAsStringAsync).toHaveBeenCalledWith(
       'file://test.csv',
@@ -72,7 +72,12 @@ describe('readFileContent', () => {
     const result = await readFileContent(['.csv', '.json', 'text/plain'])
 
     expect(DocumentPicker.getDocumentAsync).toHaveBeenCalledWith({
-      type: ['text/csv', 'application/json', 'text/plain']
+      type: [
+        'text/csv',
+        'text/comma-separated-values',
+        'application/json',
+        'text/plain'
+      ]
     })
     expect(result).toEqual({
       fileContent: 'plain text',