Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

exercise util: fix update cmd test file layout #686

Closed
coriolinus opened this issue Oct 20, 2018 · 1 comment
Closed

exercise util: fix update cmd test file layout #686

coriolinus opened this issue Oct 20, 2018 · 1 comment
Labels
track meta/exercise crate Issues dealing with the exercise crate

Comments

@coriolinus
Copy link
Member

The test file is easiest for a human to read and understand if it contains the following sections in sequence:

  • test module doccomment
  • imports
  • process_X_case implementations
  • tests

The exercise update command should ensure that the test output file conforms to that sequencing.

From the exercise tool's point of view, #685 requires parsing the test file to determine which tests exist, and what bytes they contain, anyway. Given that information, it should not be difficult to insert new process_X_case implementations at the end of the appropriate section.

@senekor
Copy link
Contributor

senekor commented Sep 11, 2023

exercise util has been deleted since, see this thread for more context.

@senekor senekor closed this as not planned Won't fix, can't repro, duplicate, stale Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
track meta/exercise crate Issues dealing with the exercise crate
Projects
None yet
Development

No branches or pull requests

2 participants