Skip to content

Commit

Permalink
fix: use spawn to stream larger output rather than exec which buffers it
Browse files Browse the repository at this point in the history
  • Loading branch information
nick-fields committed Aug 4, 2022
1 parent e75d568 commit 7aea092
Showing 1 changed file with 4 additions and 16 deletions.
20 changes: 4 additions & 16 deletions .github/workflows/ci_cd.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 7aea092

Please sign in to comment.