Skip to content

Stabilize timeout partial output test#19

Merged
maniramezan merged 1 commit into
mainfrom
fix/ci-timeout-partial-output
May 22, 2026
Merged

Stabilize timeout partial output test#19
maniramezan merged 1 commit into
mainfrom
fix/ci-timeout-partial-output

Conversation

@maniramezan
Copy link
Copy Markdown
Owner

Summary

  • make timeoutPreservesPartialOutput wait for a marker after stdout is emitted
  • assert the deterministic partial stdout value instead of racing timeout capture

Validation

  • swift-format lint --strict Tests/SwiftyShellTests/Core/CommandTests.swift
  • swift test --filter timeoutPreservesPartialOutput -Xswiftc -warnings-as-errors
  • swift test -Xswiftc -warnings-as-errors
  • swift -warnings-as-errors Scripts/validate-traits.swift
  • swift -warnings-as-errors Scripts/validate-docc-coverage.swift
  • swift package -Xswiftc -warnings-as-errors --allow-writing-to-directory docs generate-documentation --target SwiftyShell --output-path docs --transform-for-static-hosting --hosting-base-path SwiftyShell
  • swift test --enable-all-traits --enable-code-coverage -Xswiftc -warnings-as-errors
  • swift -warnings-as-errors Scripts/validate-code-coverage.swift --input .build/arm64-apple-macosx/debug/codecov/SwiftyShell.json --minimum-line-coverage 84

@maniramezan maniramezan merged commit fa4b98c into main May 22, 2026
9 checks passed
@maniramezan maniramezan deleted the fix/ci-timeout-partial-output branch May 22, 2026 02:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant