test: tidy up tests and fix remaining errors (bad mocking of expo-con…

…stants)
nearform · Jan 10, 2025 · bf0d15b · bf0d15b
1 parent 9ca997d
commit bf0d15b
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/jest.setup.js b/jest.setup.js
@@ -19,6 +19,7 @@ jest.mock('@expo-google-fonts/fira-code', () => ({
 
 jest.mock('expo-auth-session/providers/google')
 jest.mock('expo-constants', () => ({
+  ...jest.requireActual('expo-constants'),
   manifest: { extra: { apiUrl: 'http://dummy.com/api' } },
 }))
 // jest.mock('firebase')

diff --git a/src/components/NotificationPermissionRequest.tsx b/src/components/NotificationPermissionRequest.tsx
@@ -81,7 +81,9 @@ export default function NotificationPermissionRequest() {
       }
     } else {
       setPermissionGranted(false)
-      console.log('permissions denied, not a device')
+      if (typeof process.env.JEST_WORKER_ID === 'undefined') {
+        console.log('permissions denied, not a device') // only log if not running tests
+      }
     }
   }
   async function sendPermissionRequest() {

diff --git a/src/screens/CreateTokenScreen.tsx b/src/screens/CreateTokenScreen.tsx
@@ -102,7 +102,9 @@ export const CreateTokenScreen = ({ route, navigation }: Props) => {
     return null
   }
 
-  console.log(expoToken)
+  if (typeof process.env.JEST_WORKER_ID === 'undefined') {
+    console.log(expoToken) // only log if not running tests
+  }
 
   return (
     <>