build: do not fail integration test when service is flaky (#874)

build: do not fail integration test when service is flaky

Author: germanattanasio

.gitignore

@@ -42,3 +42,5 @@ sdk.js
 **/*.js.map
 coverage.lcov
 .swagger-codegen-ignore
+*.log
+*.tmp

.travis.yml

@@ -14,8 +14,8 @@ before_install:
 - npm install -g typescript
 script:
 - tsc
-- if [ "${TRAVIS_PULL_REQUEST}" = "false" ]; then npm run test-travis; else npm run
-  test-unit-travis; fi
+- npm run test-unit-travis || travis_terminate 1
+- npm run test-integration-travis || node scripts/report_integration_test.js || (cat test-output.log && travis_terminate 1)
 - npm run check-packages
 - sh scripts/typedoc/generate_typedoc.sh
 after_success:

package.json

@@ -118,7 +118,7 @@
     "test-unit": "jest --silent --verbose test/unit/",
     "test": "jest --silent --verbose test/",
     "test-unit-travis": "jest --silent --runInBand test/unit/",
-    "test-travis": "jest --silent --runInBand --testNamePattern='^((?!@slow).)*$' test/",
+    "test-integration-travis": "jest --silent --runInBand --no-colors --testNamePattern='^((?!@slow).)*$' --json test/integration > test-output.log",
     "report-coverage": "codecov",
     "watch-doc": "nodemon --watch ./ --ext js,tmpl,json --ignore dist/ --ignore doc/ --ignore test/ --ignore examples/ --exec npm run doc",
     "watch": "npm run test-unit -- --watch",

scripts/report_integration_test.js

@@ -0,0 +1,81 @@
+const fs = require('fs');
+const path = require('path');
+const axios = require('axios');
+const test_output = fs.readFileSync(path.resolve('test-output.log'), { encoding: 'utf8' });
+const test_ouput_json = JSON.parse(test_output);
+const ansi_regex = new RegExp(
+  [
+    '[\\u001B\\u009B][[\\]()#;?]*(?:(?:(?:[a-zA-Z\\d]*(?:;[-a-zA-Z\\d\\/#&.:=?%@~_]*)*)?\\u0007)',
+    '(?:(?:\\d{1,4}(?:;\\d{0,4})*)?[\\dA-PR-TZcf-ntqry=><~]))',
+  ].join('|'),
+  'g'
+);
+
+const failed_suits = test_ouput_json.testResults.filter(suite => suite.status === 'failed');
+
+const errors = {
+  service: [],
+  test: [],
+};
+
+failed_suits.map(suite => {
+  const failed_tests = suite.assertionResults.filter(test => test.status === 'failed');
+  const error_suite = {
+    name: suite.name.split('node-sdk/test')[1],
+    service: [],
+    test: [],
+  };
+
+  failed_tests.map(result => {
+    const message_clean = result.failureMessages.join('\n').replace(ansi_regex, '');
+    error_suite[message_clean.indexOf(/^Received: 5/m) > 0 ? 'service' : 'test'].push(
+      `${result.fullName}\n${message_clean}`
+    );
+  });
+
+  errors.service.push(`${error_suite.name}\n${error_suite.service.join('\n')}`);
+  errors.test.push(`${error_suite.name}\n${error_suite.test.join('\n')}`);
+});
+
+let body = '';
+if (errors.service.length > 0) {
+  body = `${body}## Service Failures\n${errors.service.join('\n')}\n`;
+}
+
+if (errors.test.length > 0) {
+  body = `${body}## Possible Test Failures\n${errors.test.join('\n')}\n`;
+}
+
+if (process.env.TRAVIS_PULL_REQUEST && process.env.TRAVIS_PULL_REQUEST !== 'false') {
+  // Send the result to the pull request if it is a pull request.
+  axios
+    .post(
+      `https://api.github.com/repos/${process.env.TRAVIS_REPO_SLUG}/issues/${
+        process.env.TRAVIS_PULL_REQUEST
+      }/comments`,
+      {
+        body: body,
+      },
+      {
+        headers: {
+          'User-Agent': 'watson-github-bot',
+          Authorization: `token ${process.env.GH_TOKEN}`,
+        },
+      }
+    )
+    .catch(error => {
+      console.error(error); // eslint-disable-line
+    })
+    .then(() => {
+      if (errors.test.length > 0) {
+        process.exit(1); // eslint-disable-line
+      }
+    });
+} else {
+  // Write to stdout
+  console.log(body); // eslint-disable-line
+
+  if (errors.test.length > 0) {
+    process.exit(1); // eslint-disable-line
+  }
+}