Skip to content

Update README.md and swift-testing unit tests for clarity of usage#36

Merged
mlink merged 1 commit into
mainfrom
update-readme-and-swift-testing-for-clarity
May 13, 2026
Merged

Update README.md and swift-testing unit tests for clarity of usage#36
mlink merged 1 commit into
mainfrom
update-readme-and-swift-testing-for-clarity

Conversation

@mlink
Copy link
Copy Markdown
Contributor

@mlink mlink commented May 12, 2026

This pull request updates the subprocess testing infrastructure and test suite organization to improve clarity, consistency, and reliability of parallel and subprocess-related tests. The changes standardize the use of .subprocess and .serialized annotations, clarify test naming, and introduce new test coverage for error scenarios.

Test infrastructure and annotation updates:

  • Updated all test and suite annotations to use .subprocess instead of the deprecated .subprocessTesting, and added .serialized where necessary to prevent race conditions with shared mock storage. [1] [2] [3] [4] [5]

Test naming and organization improvements:

  • Renamed several test functions for clarity and consistency, such as changing testEcho to echo, testSoftwareVersion to software version, and testMissingFile to missing file. [1] [2] [3] [4]

New and enhanced test coverage:

  • Added a new test grep returns an error to specifically verify error handling when grep fails, using isolated mocks for this scenario.

Documentation improvements:

  • Updated documentation in README.md to clarify the need for .serialized when using .subprocess at the suite level, to ensure proper test isolation.

These changes help ensure that subprocess tests are more robust, easier to understand, and less prone to issues with concurrent execution.

@mlink mlink merged commit abf9586 into main May 13, 2026
4 checks passed
@mlink mlink deleted the update-readme-and-swift-testing-for-clarity branch May 13, 2026 13:57
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.

3 participants