Update toHaveBeenCalledAfter to fail if second mock is never called

src/matchers/toHaveBeenCalledAfter.js

@@ -37,7 +37,7 @@ export function toHaveBeenCalledAfter(actual, expected) {
 const smallest = ns => ns.reduce((acc, n) => (acc < n ? acc : n));
 
 const predicate = (firstInvocationCallOrder, secondInvocationCallOrder) => {
-  if (firstInvocationCallOrder.length === 0) return true;
+  if (firstInvocationCallOrder.length === 0) return false;
   if (secondInvocationCallOrder.length === 0) return false;
 
   const firstSmallest = smallest(firstInvocationCallOrder);

test/matchers/__snapshots__/toHaveBeenCalledAfter.test.js.snap

@@ -1,14 +1,5 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`.not.toHaveBeenCalledAfter fails when given a first mock has not been called 1`] = `
-"<dim>expect(</><red>received</><dim>).not.toHaveBeenCalledAfter(</><green>expected</><dim>)</>
-
-Expected first mock to not have been called after, invocationCallOrder:
-  <green>[]</>
-Received second mock with invocationCallOrder:
-  <red>[]</>"
-`;
-
 exports[`.not.toHaveBeenCalledAfter fails when given first mock is called after second mock 1`] = `
 "<dim>expect(</><red>received</><dim>).not.toHaveBeenCalledAfter(</><green>expected</><dim>)</>
 
@@ -27,6 +18,15 @@ Received second mock with invocationCallOrder:
   <red>[25]</>"
 `;
 
+exports[`.toHaveBeenCalledAfter fails when given first mock has not been called 1`] = `
+"<dim>expect(</><red>received</><dim>).toHaveBeenCalledAfter(</><green>expected</><dim>)</>
+
+Expected first mock to have been called after, invocationCallOrder:
+  <green>[]</>
+Received second mock with invocationCallOrder:
+  <red>[]</>"
+`;
+
 exports[`.toHaveBeenCalledAfter fails when given first mock is called before multiple calls to second mock 1`] = `
 "<dim>expect(</><red>received</><dim>).toHaveBeenCalledAfter(</><green>expected</><dim>)</>
 

test/matchers/toHaveBeenCalledAfter.test.js

@@ -3,10 +3,11 @@ import * as matcher from 'src/matchers/toHaveBeenCalledAfter';
 expect.extend(matcher);
 
 describe('.toHaveBeenCalledAfter', () => {
-  test('passes when given a first mock has not been called', () => {
+  test('fails when given first mock has not been called', () => {
     const mock1 = jest.fn();
     const mock2 = jest.fn();
-    expect(mock1).toHaveBeenCalledAfter(mock2);
+
+    expect(() => expect(mock1).toHaveBeenCalledAfter(mock2)).toThrowErrorMatchingSnapshot();
   });
 
   test('fails when given first mock that has been called and a second mock that has not been called', () => {
@@ -87,12 +88,6 @@ describe('.toHaveBeenCalledAfter', () => {
 });
 
 describe('.not.toHaveBeenCalledAfter', () => {
-  test('fails when given a first mock has not been called', () => {
-    const mock1 = jest.fn();
-    const mock2 = jest.fn();
-    expect(() => expect(mock1).not.toHaveBeenCalledAfter(mock2)).toThrowErrorMatchingSnapshot();
-  });
-
   test('passes when given first mock that has been called and a second mock that has not been called', () => {
     const mock1 = jest.fn();
     const mock2 = jest.fn();