i think i'm close

Author: jsjoeio

test/register.test.ts

@@ -27,9 +27,7 @@ describe("register", () => {
   })
 
   afterEach(() => {
-    // jest.resetModules()
-    // jest.resetAllMocks()
-    // jest.clearAllMocks()
+    jest.resetModules()
   })
 
   afterAll(() => {
@@ -44,28 +42,23 @@ describe("register", () => {
     // expect spy to have been called
     expect(spy).toHaveBeenCalled()
     expect(spy).toHaveBeenCalledTimes(1)
+    spy.mockClear()
   })
 
   it("should log an error if something doesn't work", () => {
-    mockRegisterFn.mockImplementation(() => console.log("frick ya"))
     loggerSpy = jest.spyOn(logger, "error")
     const message = "Can't find browser"
     const error = new Error(message)
-    global.navigator.serviceWorker.register = () => {
-      console.log("errror here")
-      throw error
-    }
-    console.log(global.navigator.serviceWorker.register.toString())
-    spy = jest.spyOn(global.navigator.serviceWorker, "register")
-    spy.mockResolvedValue(() => {
+
+    mockRegisterFn.mockImplementation(() => {
       throw error
     })
 
     // Load service worker like you would in the browser
     require("../src/browser/register")
 
-    expect(loggerSpy).toHaveBeenCalled()
     expect(spy).toHaveBeenCalled()
+    expect(loggerSpy).toHaveBeenCalled()
     // expect(loggerSpy).toHaveBeenCalledTimes(1)
     // Because we use logError, it will log the prefix along with the error message
     // expect(loggerSpy).toHaveBeenCalledWith(`[Service Worker] registration: ${error.message} ${error.stack}`)