Skip to content

Conversation

@Greenheart
Copy link

Main changes

Describe how to create temporary directories, and how to execute the CLI test with a specific current working directory (CWD).

This expands on the basic examples and ensures the common needs for CLI app testing are covered. Hopefully, this will make testing more accessible to people reading the book.

@epage
Copy link
Contributor

epage commented Jan 19, 2026

  1. Please clean up your commits for how they should be reviewed and merged
  2. Please separate concerns to separate commits


</aside>

## Executing tests in a temporary directory
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is all focusing on teaching an API and is not fitting it within the narrative of the book

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will rewrite the section intro to fit better with the style of the section before and after.

For example, by connecting the newly introduced APIs to what the book has explored so far.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It also needs to be tied into the application we are testing

@Greenheart
Copy link
Author

Greenheart commented Jan 19, 2026

Thanks for the review.

Update: I split the unrelated changes into a separate commit, and combined the main changes into a single commit.

Will think about how to get the example to fit better with the book's narrative.

@Greenheart Greenheart force-pushed the feature/add-cli-testing-with-tmp-directory-and-cwd branch 2 times, most recently from 3fe28f9 to 26b2faf Compare January 19, 2026 14:45
test with a specific current working directory (CWD).

This expands on the basic examples and ensures the common needs for CLI
app testing are covered. Hopefully, this will make testing more
accessible to people reading the book.
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.

2 participants