From 7aea0927da65a7b007199cb0004b4d9e38add63c Mon Sep 17 00:00:00 2001 From: Nick Fields <46869826+nick-fields@users.noreply.github.com> Date: Wed, 3 Aug 2022 21:47:27 -0400 Subject: [PATCH] fix: use spawn to stream larger output rather than exec which buffers it --- .github/workflows/ci_cd.yml | 20 ++++---------------- 1 file changed, 4 insertions(+), 16 deletions(-) diff --git a/.github/workflows/ci_cd.yml b/.github/workflows/ci_cd.yml index 602a745..07d7b50 100644 --- a/.github/workflows/ci_cd.yml +++ b/.github/workflows/ci_cd.yml @@ -61,30 +61,18 @@ jobs: node-version: 16 - name: Install dependencies run: npm ci - - name: 1MiB - id: sigpipe-a + - name: 100MiB + id: 100MiB continue-on-error: true uses: ./ with: max_attempts: 1 timeout_minutes: 5 - command: 'make bytes-1024' - - name: 1⅛MiB - id: sigpipe-b - continue-on-error: true - uses: ./ - with: - max_attempts: 1 - timeout_minutes: 5 - command: 'make bytes-1152' - - uses: nick-invision/assert-action@v1 - with: - expected: failure - actual: ${{ steps.sigpipe-a.outcome }} + command: 'make bytes-102400' - uses: nick-invision/assert-action@v1 with: expected: failure - actual: ${{ steps.sigpipe-b.outcome }} + actual: ${{ steps.100MiB.outcome }} ci_integration: name: Run Integration Tests