Fix a testing issue and update coverage

MetaMask · Nov 2, 2023 · 2ea6431 · 2ea6431
1 parent 009cb2a
commit 2ea6431
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/packages/snaps-execution-environments/coverage.json b/packages/snaps-execution-environments/coverage.json
@@ -1,6 +1,6 @@
 {
-  "branches": 81.29,
-  "functions": 92.59,
-  "lines": 92.03,
-  "statements": 91.71
+  "branches": 80.85,
+  "functions": 92.7,
+  "lines": 91.95,
+  "statements": 91.65
 }
diff --git a/packages/snaps-execution-environments/src/common/endowments/network.test.ts b/packages/snaps-execution-environments/src/common/endowments/network.test.ts
@@ -12,7 +12,6 @@ describe('Network endowments', () => {
   describe('fetch', () => {
     beforeEach(() => {
       fetchMock.enableMocks();
-      factoryOptions.notify.mockClear();
     });
 
     afterEach(() => {
@@ -28,18 +27,19 @@ describe('Network endowments', () => {
     });
 
     it('send notification about outbound request and response', async () => {
+      const notify = jest.fn();
       const RESULT = 'OK';
       fetchMock.mockOnce(async () => Promise.resolve(RESULT));
-      const { fetch } = network.factory(factoryOptions);
+      const { fetch } = network.factory({ notify });
       const result = await (await fetch('foo.com')).text();
       expect(result).toStrictEqual(RESULT);
-      expect(factoryOptions.notify).toHaveBeenCalledWith({
+      expect(notify).toHaveBeenCalledWith({
         method: 'OutboundRequest',
       });
-      expect(factoryOptions.notify).toHaveBeenCalledWith({
+      expect(notify).toHaveBeenCalledWith({
         method: 'OutboundResponse',
       });
-      expect(factoryOptions.notify).toHaveBeenCalledTimes(4);
+      expect(notify).toHaveBeenCalledTimes(4);
     });
 
     it('can use AbortController normally', async () => {