Update lib/ci/build-types/citgm_comparison_build.js

Co-authored-by: Richard Lau <riclau@uk.ibm.com>
nodejs · Jul 20, 2020 · 8b99bba · 8b99bba
1 parent 40395ce
commit 8b99bba
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/lib/ci/build-types/citgm_comparison_build.js b/lib/ci/build-types/citgm_comparison_build.js
@@ -32,7 +32,7 @@ class CITGMComparisonBuild {
 
     const failures = {};
     for (const platform in baseFailures) {
-      const { modules: baseModules } = comparisonFailures[platform];
+      const { modules: baseModules } = baseFailures[platform];
       const { modules: comparisonModules } = comparisonFailures[platform];
 
       const newFailures = comparisonModules.filter(f => {

diff --git a/lib/ci/build-types/test_build.js b/lib/ci/build-types/test_build.js
@@ -47,6 +47,13 @@ class TestBuild extends Job {
     this.builtOn = builtOn;
   }
 
+  setDailyBuildData({ result, changeSet, actions, timestamp, builtOn }) {
+    this.change = changeSet.items[0] || {};
+    this.date = new Date(timestamp);
+    this.result = result;
+    this.builtOn = builtOn;
+  }
+
   get sourceURL() {
     const { params } = this;
 
@@ -125,7 +132,9 @@ class TestBuild extends Job {
   displayBuilds() {
     const { cli, failures, builds } = this;
     for (const failure of failures) {
-      this.displayFailure(failure);
+      if (failure !== undefined) {
+        this.displayFailure(failure);
+      }
     }
     cli.separator('Other builds');
     for (const aborted of builds.aborted) {
@@ -151,6 +160,7 @@ class TestBuild extends Job {
     const { failures } = this;
     let output = `Failures in job ${this.jobUrl}\n\n`;
     for (const failure of failures) {
+      if (failure === undefined) continue;
       output += `#### [${getNodeName(failure.url)}](${failure.url})`;
       if (!failure.reason.includes('\n') && failure.reason.length < 20) {
         const builtOn = failure.builtOn ? `On ${failure.builtOn}: ` : '';