From 163cf1befda9bebc9a86b4871ab92e9b00b9e53b Mon Sep 17 00:00:00 2001
From: Andrey Lushnikov <aslushnikov@gmail.com>
Date: Mon, 26 Oct 2020 17:46:13 -0700
Subject: [PATCH] devops: collect test reports for all bots (#4246)

Drive-by: make sure to upload flakiness even if tests fail.
---
 .github/workflows/tests.yml | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index db503b0df9..e1543f2fa2 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -13,6 +13,7 @@ on:
 env:
   # Force terminal colors. @see https://www.npmjs.com/package/colors
   FORCE_COLOR: 1
+  FLAKINESS_CONNECTION_STRING: ${{ secrets.FLAKINESS_CONNECTION_STRING }}
 
 jobs:
   test_linux:
@@ -42,9 +43,7 @@ jobs:
         BROWSER: ${{ matrix.browser }}
         FOLIO_JSON_OUTPUT_NAME: "test-results/report.json"
     - run: ./utils/upload_flakiness_dashboard.sh ./test-results/report.json
-      if: github.ref == 'refs/heads/master'
-      env:
-        FLAKINESS_CONNECTION_STRING: ${{ secrets.FLAKINESS_CONNECTION_STRING }}
+      if: always() && github.ref == 'refs/heads/master'
     - uses: actions/upload-artifact@v1
       if: always()
       with:
@@ -72,6 +71,8 @@ jobs:
       env:
         BROWSER: ${{ matrix.browser }}
         FOLIO_JSON_OUTPUT_NAME: "test-results/report.json"
+    - run: ./utils/upload_flakiness_dashboard.sh ./test-results/report.json
+      if: always() && github.ref == 'refs/heads/master'
     - uses: actions/upload-artifact@v1
       if: ${{ always() }}
       with:
@@ -101,6 +102,8 @@ jobs:
       env:
         BROWSER: ${{ matrix.browser }}
         FOLIO_JSON_OUTPUT_NAME: "test-results/report.json"
+    - run: ./utils/upload_flakiness_dashboard.sh ./test-results/report.json
+      if: always() && github.ref == 'refs/heads/master'
     - uses: actions/upload-artifact@v1
       if: ${{ always() }}
       with:
@@ -153,6 +156,8 @@ jobs:
         BROWSER: ${{ matrix.browser }}
         HEADLESS: "false"
         FOLIO_JSON_OUTPUT_NAME: "test-results/report.json"
+    - run: ./utils/upload_flakiness_dashboard.sh ./test-results/report.json
+      if: always() && github.ref == 'refs/heads/master'
     - uses: actions/upload-artifact@v1
       if: ${{ always() }}
       with:
@@ -185,6 +190,8 @@ jobs:
         BROWSER: ${{ matrix.browser }}
         PWWIRE: true
         FOLIO_JSON_OUTPUT_NAME: "test-results/report.json"
+    - run: ./utils/upload_flakiness_dashboard.sh ./test-results/report.json
+      if: always() && github.ref == 'refs/heads/master'
     - uses: actions/upload-artifact@v1
       if: ${{ always() }}
       with:
@@ -216,6 +223,8 @@ jobs:
       env:
         BROWSER: ${{ matrix.browser }}
         FOLIO_JSON_OUTPUT_NAME: "test-results/report.json"
+    - run: ./utils/upload_flakiness_dashboard.sh ./test-results/report.json
+      if: always() && github.ref == 'refs/heads/master'
     - uses: actions/upload-artifact@v1
       if: ${{ always() }}
       with: