Merge pull request #233 from pact-foundation/fix/issue-231

fix(interaction-failure): clear interactions on any verify()
pact-foundation · Oct 17, 2018 · e375894 · e375894
2 parents ac27a1e + fbc5ac0
commit e375894
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/pact.spec.ts b/src/pact.spec.ts
@@ -245,7 +245,7 @@ describe("Pact", () => {
  const verifyPromise = b.verify();
  return Promise.all([
  expect(verifyPromise).to.eventually.be.rejectedWith(Error),
- verifyPromise.catch(() => expect(removeInteractionsStub).to.callCount(0)),
+ verifyPromise.catch(() => expect(removeInteractionsStub).to.callCount(1)),
  ]);
  });
  });

diff --git a/src/pact.ts b/src/pact.ts
@@ -127,15 +127,18 @@ export class Pact {
  return this.mockService.verify()
  .then(() => this.mockService.removeInteractions())
  .catch((e: any) => {
+
  // Properly format the error
  /* tslint:disable: no-console */
  console.error("");
  console.error(clc.red("Pact verification failed!"));
  console.error(clc.red(e));
  /* tslint:enable: */
 
- throw new Error("Pact verification failed - expected interactions did not match actual.");
- });
+ return this.mockService.removeInteractions().then(() => {
+ throw new Error("Pact verification failed - expected interactions did not match actual.");
+ })
+ })
  }
 
  /**