Skip to content

Conversation

ipince
Copy link
Contributor

@ipince ipince commented Jul 24, 2023

Summary

We were abusing the "example test" term to really mean "run devbox run run_test on any devbox.json". Updated code and docs to reflect reality, since it was confusing while I was debugging a failed test.

How was it tested?

DEVBOX_RUN_DEVBOX_JSON_TESTS=1 go test testscripts [-run TestExamples/flake_remote_run_test

@ipince ipince requested review from mikeland73 and savil July 24, 2023 21:05
@ipince
Copy link
Contributor Author

ipince commented Jul 24, 2023

Note: the failing test is unrelated to this change (due to broken process-compose flake)

Copy link
Collaborator

@savil savil left a comment

Choose a reason for hiding this comment

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

ah, yeah, I guess this got generalized when testscripts/shellenv was added but the names were never updated properly...

@ipince ipince merged commit aab7f8c into main Jul 24, 2023
@ipince ipince deleted the rodrigo/better-test-docs branch July 24, 2023 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants