Fix TypeScript tests

dangerfile.js

@@ -1,6 +1,6 @@
 import fs from "fs"
 import includes from "lodash.includes"
-fs.f
+
 // Request a CHANGELOG entry if not declared #trivial
 const hasChangelog = includes(danger.git.modified_files, "changelog.md")
 const isTrivial = includes((danger.github.pr.body + danger.github.pr.title), "#trivial")

source/runner/DangerfileRunner.ts

@@ -56,7 +56,7 @@ async function dangerJestConfig() {
   // so if you can make you tests run right,
   // then it's pretty likely that Danger can do it too.
   const jestConfig = await readConfig([], process.cwd())
-
+  // console.log(jestConfig)
   return {
     cacheDirectory: os.tmpdir(),
     setupFiles: [],
@@ -65,12 +65,14 @@ async function dangerJestConfig() {
       defaultPlatform: "danger-js"
     },
     moduleNameMapper: [],
+    moduleDirectories: [ "node_modules" ],
     moduleFileExtensions: ["js", ...jestConfig.moduleFileExtensions],
     transform: [["js$", "babel-jest"], ...jestConfig.transform],
-    transformIgnorePatterns: [],
+    testPathIgnorePatterns: jestConfig.testPathIgnorePatterns,
     cache: null,
     testRegex: "",
-    testPathDirs: [process.cwd()]
+    testPathDirs: [process.cwd()],
+    transformIgnorePatterns: [ "/node_modules/" ]
   }
 }
 

source/runner/_tests/DangerRunner.test.ts

@@ -101,17 +101,20 @@ describe("with fixtures", () => {
     })
   })
 
-  it.skip("can execute async/await scheduled functions", async () => {
-    // this test takes *forever* because of babel-polyfill being required
-    const context = await setupDangerfileContext()
-    const runtime = await createDangerfileRuntimeEnvironment(context)
-    const results = await runDangerfileEnvironment(resolve(fixtures, "__DangerfileAsync.js"), runtime)
-    expect(results.warnings).toEqual([{
-      message: "Async Function"
-    }, {
-      message: "After Async Function"
-    }])
-  })
+  // This adds > 6 seconds to the tests! Only orta should be forced into that.
+  if (process.env["USER"] === "orta") {
+     it("can execute async/await scheduled functions", async () => {
+      // this test takes *forever* because of babel-polyfill being required
+      const context = await setupDangerfileContext()
+      const runtime = await createDangerfileRuntimeEnvironment(context)
+      const results = await runDangerfileEnvironment(resolve(fixtures, "__DangerfileAsync.js"), runtime)
+      expect(results.warnings).toEqual([{
+        message: "Async Function"
+      }, {
+        message: "After Async Function"
+      }])
+    })
+  }
 
   it("can schedule callback-based promised", async () => {
     const context = await setupDangerfileContext()
@@ -126,7 +129,7 @@ describe("with fixtures", () => {
     const context = await setupDangerfileContext()
     const runtime = await createDangerfileRuntimeEnvironment(context)
     const results = await runDangerfileEnvironment(resolve(fixtures, "__DangerfileTypeScript.ts"), runtime)
-    expect(results.warnings).toEqual([{
+    expect(results.messages).toEqual([{
       message: "Honey, we got Types",
     }])
   })

source/runner/_tests/fixtures/__DangerfileTypeScript.ts

@@ -3,5 +3,4 @@
 // This doesn't exist in JS-world
 interface MyThing {}
 
-// Removed import
 message("Honey, we got Types")