Make sure tests actually fail (#25)

* Make sure tests actually fail * Fix mailgun test
vercel · Nov 20, 2018 · f926990 · f926990
1 parent 5eeacd0
commit f926990
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 10 deletions.
diff --git a/test/index.test.js b/test/index.test.js
@@ -4,16 +4,11 @@ const ncc = require("../");
 for (const integrationTest of fs.readdirSync(__dirname + "/integration")) {
   it(`should evaluate ${integrationTest} without errors`, async () => {
     const code = await ncc(__dirname + "/integration/" + integrationTest);
-    try {
-      module.exports = null;
-      eval(code);
-      if ("function" !== typeof module.exports) {
-        throw new Error(`Integration test "${integrationTest}" evaluation failed. It does not export a function`)
-      }
-      await module.exports();
-    } catch (err) {
-      errored = true;
-      console.error(`Integration test "${integrationTest}" execution failed`, err);
+    module.exports = null;
+    eval(code);
+    if ("function" !== typeof module.exports) {
+      throw new Error(`Integration test "${integrationTest}" evaluation failed. It does not export a function`)
     }
+    await module.exports();
   })
 }
diff --git a/test/integration/mailgun.js b/test/integration/mailgun.js
@@ -1 +1,3 @@
 const mailgun = require('mailgun');
+
+module.exports = () => {}
Original file line number	Diff line number	Diff line change
		@@ -1 +1,3 @@
		const mailgun = require('mailgun');

		module.exports = () => {}