separate bot messages from integration and unit tests

sokra · sokra · commit 9ffd97c3c137 · 2017-10-19T17:45:22.000+02:00
diff --git a/open-bot.yaml b/open-bot.yaml
@@ -59,7 +59,7 @@ rules:
 
 
 
-# Report specific error message if mocha fails
+# Report specific error message if mocha for integration tests fails
 - filters:
     ensure:
       value: "{{report_ci}}"
@@ -71,7 +71,7 @@ rules:
       state: "failed"
       allow_failure: false
       config:
-        env: JOB_PART=test
+        env: JOB_PART=integration
     fetch: travis_job.log
     string_cleanup:
       id: logResult
@@ -97,6 +97,46 @@ rules:
 
 
 
+# Report specific error message if mocha for unit tests fails
+- filters:
+    ensure:
+      value: "{{report_ci}}"
+      equals: yep
+    commit: true
+    status:
+      context: "continuous-integration/travis-ci/pr"
+    travis_job:
+      state: "failed"
+      allow_failure: false
+      config:
+        env: JOB_PART=unit
+    fetch: travis_job.log
+    string_cleanup:
+      id: logResult
+      value: "{{{fetch}}}"
+      remove:
+        - "^[\\s\\S]+?\\d+\\s+pending\n+"
+        - "npm ERR!.*\n"
+        - "\n*=============================================================================\n[\\s\\S]*"
+  actions:
+    comment:
+      identifier: "ci-result"
+      message: |-
+        @{{commit.author.login}} Please review the following output log for errors:
+
+        ``` text
+        {{{logResult}}}
+        ```
+
+        Instead of updating these (outdated?) unit tests, you can choose to delete them and add integration tests instead. That would be great.
+
+        See [complete report here]({{status.target_url}}).
+    set:
+      id: report_ci
+      value: nope
+
+
+
 # Report specific error message if linting fails
 - filters:
     ensure:
