Apply comment from code review to simplify test, thanks @greyseer256

finos · Feb 21, 2022 · 7a10fdd · 7a10fdd
1 parent 3f0b876
commit 7a10fdd
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/test/Methods.test.ts b/test/Methods.test.ts
@@ -45,6 +45,10 @@ expect.extend({
   },
 });
 
+beforeEach(() => {
+  jest.resetAllMocks();
+});
+
 describe('test ES6 module', () => {
   describe('without `window.fdc3` global', () => {
     test('open should reject', async () => {
@@ -203,7 +207,7 @@ describe('test ES6 module', () => {
     test('getSystemChannels should delegate to window.fdc3.getUserChannels', async () => {
       await getSystemChannels();
 
-      expect(window.fdc3.getUserChannels).toHaveBeenCalledTimes(2); //was already called in previous test, hence this should be the second call to getUserChannels
+      expect(window.fdc3.getUserChannels).toHaveBeenCalledTimes(1);
       expect(window.fdc3.getUserChannels).toHaveBeenCalledWith();
     });
 
@@ -221,7 +225,7 @@ describe('test ES6 module', () => {
 
       await joinUserChannel(channelId);
 
-      expect(window.fdc3.joinUserChannel).toHaveBeenCalledTimes(2);
+      expect(window.fdc3.joinUserChannel).toHaveBeenCalledTimes(1);
       expect(window.fdc3.joinUserChannel).toHaveBeenCalledWith(channelId);
     });