Fix a few bugs#205
Merged
Merged
Conversation
Fixes dylan-lang#83 I'm claiming that this fixes dylan-lang#83 even though it doesn't connect the output with the test result objects. This provides a good base that makes that easy to accomplish if we want to do that later. The main thing is that the output no longer spams the console and yet it is available for later debugging if needed.
Adds a `.1`, `.2`, etc uniquifier to the end of the test run directory name so that if two test runs are executed within the same second there won't be a clash. The unique directory for the entire test run is now stored in the runner. Previously it was possible to end up with one test's temp files in `_test/cgay-20250926-101601` and the next test's temp files in `_test/cgay-20250926-101602`. Fixes dylan-lang#149 Fixes dylan-lang#199
Fixes dylan-lang#157 (Well not really, but it verifies that dylan-lang#157 was fixed sometime in the past.)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.