Fixing rejection tests

jestjs · Jan 15, 2018 · 5ae2641 · 5ae2641
1 parent 85fad97
commit 5ae2641
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/packages/jest-mock/src/__tests__/jest_mock.test.js b/packages/jest-mock/src/__tests__/jest_mock.test.js
@@ -422,7 +422,9 @@ describe('moduleMocker', () => {
 
       expect(promise).toBeInstanceOf(Promise);
 
-      return promise.catch(rejection => expect(rejection).toBe(err));
+      return promise
+        .then(() => Promise.reject(new Error('did not reject')))
+        .catch(rejection => expect(rejection).toBe(err));
     });
 
     it('supports mocking rejectable async functions only once', () => {
@@ -432,10 +434,12 @@ describe('moduleMocker', () => {
       fn.mockRejectedValue(defaultErr);
       fn.mockRejectedValueOnce(err);
 
-      const promise1 = fn().catch(rejection => expect(rejection).toBe(err));
-      const promise2 = fn().catch(rejection =>
-        expect(rejection).toBe(defaultErr),
-      );
+      const promise1 = fn()
+        .then(() => Promise.reject(new Error('did not reject')))
+        .catch(rejection => expect(rejection).toBe(err));
+      const promise2 = fn()
+        .then(() => Promise.reject(new Error('did not reject')))
+        .catch(rejection => expect(rejection).toBe(defaultErr));
 
       return Promise.all([promise1, promise2]);
     });