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

tests: show compile time and runtime for each _test.v file in separate columns #20520

Conversation

spytheman
Copy link
Member

  • benchmark: support passing an optional preparation time for the step messages
  • pref: support a -skip-running option
  • testing: add compile_begin and compile_end messages to the testing.Reporter interface
  • testing: support the new messages in the output_normal.v Reporter
  • benchmark: add Benchmark.step_restart/0 method
  • tools: fix v test to not skip _test.c.v files by default
  • testing,benchmark: use the new -skip-running and the enhanced Reporter to implement a separate compilation time measurement for each test

@spytheman spytheman changed the title tests show compile times and runtimes separately tests: show compile time and runtime for each _test.v file in separate columns Jan 13, 2024
@spytheman spytheman marked this pull request as draft January 13, 2024 21:05
@spytheman
Copy link
Member Author

The output for v test vlib/builtin/ will look like this:
image

@felipensp
Copy link
Member

Nice improvement.

@spytheman
Copy link
Member Author

Nice improvement.

thank you

@shove70
Copy link
Contributor

shove70 commented Jan 14, 2024

wow, It's beautiful 👍

@spytheman spytheman force-pushed the tests_show_compile_times_and_runtimes_separately branch from 59b4932 to 5556234 Compare January 20, 2024 12:43
@spytheman spytheman marked this pull request as ready for review January 20, 2024 15:38
@spytheman spytheman merged commit 6107e0d into vlang:master Jan 20, 2024
48 checks passed
@spytheman spytheman deleted the tests_show_compile_times_and_runtimes_separately branch January 20, 2024 15:38
kbkpbot pushed a commit to kbkpbot/v that referenced this pull request Jan 21, 2024
@ttytm
Copy link
Member

ttytm commented Feb 17, 2024

@spytheman since this commit the -stats flag won't include prints in the test output.

For example, if you run one of the vpm tests without -d network but with -stats the tests run without information such as the following, which used to be provided.

> skipping /Users/t/Nextcloud/Dev/vlang/v/cmd/tools/vpm/remove_test.v, when `-d network` is missing

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.

4 participants