Merge pull request #240 from HarperFast/integration-apitest-upd

simplify apiTests "upload logs on failure" step condition and upload available output and logs on failure

Author: heskew

.github/workflows/integration-tests.yml

@@ -112,9 +112,10 @@ jobs:
           THREADS_DEBUG: false
           NODE_HOSTNAME: 'localhost'
         run: |
-          node ./dist/bin/harper.js install
+          mkdir -p /tmp/hdb/log
+          node ./dist/bin/harper.js install > /tmp/hdb/log/install-stdout.log 2> /tmp/hdb/log/install-stderr.log
           sleep 10
-          node ./dist/bin/harper.js start
+          node ./dist/bin/harper.js start > /tmp/hdb/log/start-stdout.log 2> /tmp/hdb/log/start-stderr.log &
           sleep 10
 
       - name: Run API Tests
@@ -125,12 +126,13 @@ jobs:
         run: npm run test:integration:api-tests
 
       - name: Upload Harper logs
-        if: steps.run-api-tests.outcome == 'failure'
+        if: failure()
         uses: actions/upload-artifact@v7
         with:
           name: harper-integration-api-test-logs-node-${{ matrix.node-version }}
-          path: /tmp/hdb/log/hdb.log
-          retention-days: 2
+          path: /tmp/hdb/log/
+          retention-days: 3
+          if-no-files-found: ignore
 
   run-integration-tests:
     name: Integration Tests ${{matrix.shard}}/4 (Node.js v${{ matrix.node-version }})