1 files changed, 9 insertions, 9 deletions

diff --git a/packages/main/src/controllers/__tests__/initConfig.spec.ts b/packages/main/src/controllers/__tests__/initConfig.spec.ts
index 7b6d6ab..9a5f85e 100644
--- a/packages/main/src/controllers/__tests__/initConfig.spec.ts
+++ b/packages/main/src/controllers/__tests__/initConfig.spec.ts
@@ -56,7 +56,7 @@ describe('when initializing', () => {
 
     it('should create a new config file', async () => {
       await initConfig(config, persistenceService);
-      expect(persistenceService.writeConfig).toBeCalledTimes(1);
+      expect(persistenceService.writeConfig).toHaveBeenCalledTimes(1);
     });
 
     it('should bail if there is an an error creating the config file', async () => {
@@ -81,7 +81,7 @@ describe('when initializing', () => {
 
     it('should read the existing config file is there is one', async () => {
       await initConfig(config, persistenceService);
-      expect(persistenceService.writeConfig).not.toBeCalled();
+      expect(persistenceService.writeConfig).not.toHaveBeenCalled();
       expect(config.themeSource).toBe('dark');
     });
 
@@ -138,14 +138,14 @@ describe('when it has loaded the config', () => {
     jest.advanceTimersByTime(lessThanThrottleMs);
     config.setThemeSource('light');
     jest.advanceTimersByTime(throttleMs);
-    expect(persistenceService.writeConfig).toBeCalledTimes(1);
+    expect(persistenceService.writeConfig).toHaveBeenCalledTimes(1);
   });
 
   it('should handle config writing errors gracefully', () => {
     mocked(persistenceService.writeConfig).mockRejectedValue(new Error('boo'));
     config.setThemeSource('dark');
     jest.advanceTimersByTime(throttleMs);
-    expect(persistenceService.writeConfig).toBeCalledTimes(1);
+    expect(persistenceService.writeConfig).toHaveBeenCalledTimes(1);
   });
 
   it('should read the config file when it has changed', async () => {
@@ -157,7 +157,7 @@ describe('when it has loaded the config', () => {
     });
     await configChangedCallback();
     // Do not write back the changes we have just read.
-    expect(persistenceService.writeConfig).not.toBeCalled();
+    expect(persistenceService.writeConfig).not.toHaveBeenCalled();
     expect(config.themeSource).toBe('dark');
   });
 
@@ -172,9 +172,9 @@ describe('when it has loaded the config', () => {
     expect(config.themeSource).not.toBe(-1);
   });
 
-  it('should handle config writing errors gracefully', async () => {
+  it('should handle config reading errors gracefully', async () => {
     mocked(persistenceService.readConfig).mockRejectedValue(new Error('boo'));
-    await configChangedCallback();
+    await expect(configChangedCallback()).resolves.not.toThrow();
   });
 
   describe('when it was disposed', () => {
@@ -183,13 +183,13 @@ describe('when it has loaded the config', () => {
     });
 
     it('should dispose the watcher', () => {
-      expect(watcherDisposer).toBeCalled();
+      expect(watcherDisposer).toHaveBeenCalled();
     });
 
     it('should not listen to store changes any more', () => {
       config.setThemeSource('dark');
       jest.advanceTimersByTime(2 * throttleMs);
-      expect(persistenceService.writeConfig).not.toBeCalled();
+      expect(persistenceService.writeConfig).not.toHaveBeenCalled();
     });
   });
 });