Add a check for the fixtures

dangerfile.ts

@@ -71,3 +71,12 @@ const missing = names.filter(n => !readme.includes(n))
 if (missing.length) {
   warn(`These providers are missing from the README: ${sentence(missing)}`)
 }
+
+console.log("before")
+danger.github.utils.fileContents("scripts/run-fixtures.js").then(fixtures => {
+  console.log("after")
+  console.log(fixtures)
+  if (fixtures.includes("const writeResults = true")) {
+    fail("Fixtures test script is still in write mode, edit `scripts/run-fixtures.js`.")
+  }
+})

scripts/run-fixtures.js

@@ -11,7 +11,7 @@ const expect = require("expect")
 
 // Toggle this on to update the JSON files for each run
 const writeResults = true
-
+console.log("If this script fails, you probably want to update the fixtures - just edit script/run-fixtures.js")
 const runnerFileJS = "distribution/commands/danger-runner.js"
 
 // Get all the fixtures

source/platforms/github/GitHubUtils.ts

@@ -35,16 +35,15 @@ const utils = (pr: GitHubPRDSL, api: GitHub): GitHubUtilsDSL => {
         repoSlug = pr.head.repo.full_name
         ref = pr.head.ref
       }
-
-      const data = await api.repos.getContent({
+      const opts = {
         ref,
         path,
         repo: repoSlug.split("/")[1],
         owner: repoSlug.split("/")[0],
-      })
-
-      if (data && data.type === "file") {
-        const buffer = new Buffer(data.content, data.encoding)
+      }
+      const response = await api.repos.getContent(opts)
+      if (response && response.data && response.data.type === "file") {
+        const buffer = new Buffer(response.data.content, "base64")
         return buffer.toString()
       } else {
         return ""