catch linter errors

Author: joyeecheung

components/jenkins.js

@@ -32,6 +32,7 @@ const COMMIT_TREE =
 const FANNED_TREE = `result,url,number,subBuilds[phaseName,${BUILD_FIELDS}]`;
 // hudson.matrix.MatrixBuild
 const BUILD_TREE = 'result,runs[url,number,result]';
+const LINTER_TREE = 'result,url,number';
 
 function getPath(url) {
   return url.replace(`https://${DOMAIN}/`, '').replace('api/json', '');
@@ -208,6 +209,8 @@ class CommitBuild extends Job {
     const promises = builds.failed.map(({jobName, buildNumber}) => {
       if (jobName.includes('fanned')) {
         return new FannedBuild(cli, request, jobName, buildNumber).getResults();
+      } else if (jobName.includes('linter')) {
+        return new LinterBuild(cli, request, jobName, buildNumber).getResults();
       }
       return new NormalBuild(cli, request, jobName, buildNumber).getResults();
     });
@@ -397,6 +400,34 @@ class FannedBuild extends Job {
   }
 }
 
+class LinterBuild extends Job {
+  constructor(cli, request, jobName, id) {
+    const path = `job/${jobName}/${id}/`;
+    const tree = LINTER_TREE;
+    super(cli, request, path, tree);
+
+    this.failures = [];
+  }
+
+  async getResults() {
+    const data = await this.getConsoleText();
+    for (const pattern of FAILURE_PATTERNS) {
+      const results = data.match(pattern.pattern);
+      if (results) {
+        const failures = pattern.filter.call(this, results, data);
+        this.failures = failures;
+        return failures;
+      }
+    }
+
+    this.failures = [{
+      url: this.jobUrl,
+      reason: 'Unknown'
+    }];
+    return this.failures;
+  }
+}
+
 class NormalBuild extends Job {
   constructor(cli, request, jobName, id) {
     const path = `job/${jobName}/${id}/`;
@@ -492,6 +523,10 @@ const FAILURE_PATTERNS = [{
   pattern: /ERROR: .+/g,
   filter: pickReason(-1, 0, 5),
   name: 'build failure'
+}, {
+  pattern: /Error: .+/g,
+  filter: pickReason(0, 0, 5),
+  name: 'tool error'
 }, {
   pattern: /fatal: .+/g,
   filter: mergeReasons,