feat(jest): resolve undefined snapshots (#4446)

* feat(jest): revamp tests * feat(jest): spy on functions and remove undefined snapshots
Satellite-im · Aug 25, 2022 · fac1d65 · fac1d65
1 parent 171db0b
commit fac1d65
Show file tree

Hide file tree

Showing 8 changed files with 16 additions and 55 deletions.
diff --git a/components/mixins/Touch/__snapshots__/index.test.ts.snap b/components/mixins/Touch/__snapshots__/index.test.ts.snap
diff --git a/components/mixins/Touch/index.test.ts b/components/mixins/Touch/index.test.ts
@@ -2,8 +2,11 @@ import * as index from '~/components/mixins/Touch/index'
 
 describe('index.Touch.created', () => {
   test('0', () => {
+    const localSpy = jest.spyOn(index.Touch, 'created')
     const result: any = index.Touch.created()
-    expect(result).toMatchSnapshot()
+
+    expect(localSpy).toHaveBeenCalled()
+    expect(result).toBe(undefined)
   })
 })
 

diff --git a/components/mixins/UserPermissions/__snapshots__/index.test.ts.snap b/components/mixins/UserPermissions/__snapshots__/index.test.ts.snap
@@ -1,9 +1,5 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`index.UserPermissions.created 0 1`] = `undefined`;
-
-exports[`index.UserPermissions.created 0 2`] = `undefined`;
-
 exports[`index.exportForTesting.formatDevices 0 1`] = `
 Array [
   Object {

diff --git a/components/mixins/UserPermissions/index.test.ts b/components/mixins/UserPermissions/index.test.ts
@@ -2,8 +2,11 @@ import * as index from '~/components/mixins/UserPermissions/index'
 
 describe('index.UserPermissions.created', () => {
   test('0', () => {
+    const localSpy = jest.spyOn(index.UserPermissions, 'created')
     const result: any = index.UserPermissions.created()
-    expect(result).toMatchSnapshot()
+
+    expect(localSpy).toHaveBeenCalled()
+    expect(result).toBe(undefined)
   })
 })
 describe('index.UserPermissions.methods.getUserPermissions', () => {
@@ -55,8 +58,11 @@ describe('index.exportForTesting.formatDevices', () => {
 
 describe('index.UserPermissions.created', () => {
   test('0', () => {
+    const localSpy = jest.spyOn(index.UserPermissions, 'created')
     const result: any = index.UserPermissions.created()
-    expect(result).toMatchSnapshot()
+
+    expect(localSpy).toHaveBeenCalled()
+    expect(result).toBe(undefined)
   })
 })
 

diff --git a/store/sounds/__snapshots__/actions.test.ts.snap b/store/sounds/__snapshots__/actions.test.ts.snap
diff --git a/store/sounds/actions.test.ts b/store/sounds/actions.test.ts
@@ -36,8 +36,6 @@ describe('Manage sounds', () => {
       },
       Sounds.CALL,
     )
-
-    expect(result).toMatchSnapshot()
   })
 
   test('sound does not play', () => {
@@ -68,7 +66,8 @@ describe('Manage sounds', () => {
       Sounds.CALL,
     )
 
-    expect(result).toMatchSnapshot()
+    const localSpy = jest.spyOn(window.HTMLMediaElement.prototype, 'play')
+    expect(localSpy).not.toHaveBeenCalled()
   })
 
   test('sound stops', () => {
@@ -88,8 +87,6 @@ describe('Manage sounds', () => {
       },
       [Sounds.CALL],
     )
-
-    expect(result).toMatchSnapshot()
   })
 
   test('sound does not stop', () => {
@@ -110,7 +107,8 @@ describe('Manage sounds', () => {
       [Sounds.CALL],
     )
 
-    expect(result).toMatchSnapshot()
+    const localSpy = jest.spyOn(window.HTMLMediaElement.prototype, 'play')
+    expect(localSpy).not.toHaveBeenCalled()
   })
 
   test('setMuteSounds', () => {

diff --git a/store/ui/__snapshots__/types.test.ts.snap b/store/ui/__snapshots__/types.test.ts.snap
diff --git a/store/ui/types.test.ts b/store/ui/types.test.ts