mikros/rust
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Auto merge of #11325 - oli-obk:SPEEDTEST, r=flip1995

Browse Source 
Fix SPEEDTEST instructions and output

* `--nocapture` hasn't been needed anymore since forever (even before `ui_test`)
* the result was dividing by 1000 instead of the number of test runs, giving bogus (but still useful for the purpose) timing results.

changelog: fix SPEEDTEST instructions and output
This commit is contained in:
bors 2023-08-11 15:30:38 +00:00
commit 75370e0671
2 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
book/src/development/speedtest.md
View File

@ -9,16 +9,12 @@ accessed by the `SPEEDTEST` (and `SPEEDTEST_*`) environment variables.
To do a simple speed test of a lint (e.g. `allow_attributes`), use this command. To do a simple speed test of a lint (e.g. `allow_attributes`), use this command.
```sh ```sh
$ SPEEDTEST=ui TESTNAME="allow_attributes" cargo uitest -- --nocapture $ SPEEDTEST=ui TESTNAME="allow_attributes" cargo uitest
``` ```
This will test all `ui` tests (`SPEEDTEST=ui`) whose names start with `allow_attributes`. By default, `SPEEDTEST` will This will test all `ui` tests (`SPEEDTEST=ui`) whose names start with `allow_attributes`. By default, `SPEEDTEST` will
iterate your test 1000 times. But you can change this with `SPEEDTEST_ITERATIONS`. iterate your test 1000 times. But you can change this with `SPEEDTEST_ITERATIONS`.
```sh ```sh
$ SPEEDTEST=toml SPEEDTEST_ITERATIONS=100 TESTNAME="semicolon_block" cargo uitest -- --nocapture $ SPEEDTEST=toml SPEEDTEST_ITERATIONS=100 TESTNAME="semicolon_block" cargo uitest
``` ```
> **WARNING**: Be sure to use `-- --nocapture` at the end of the command to see the average test time. If you don't
> use `-- --nocapture` (e.g. `SPEEDTEST=ui` `TESTNAME="let_underscore_untyped" cargo uitest -- --nocapture`), this
> will not show up.

6
tests/compile-test.rs
View File

@ -377,7 +377,11 @@ fn main() {
f(); f();
sum += start.elapsed().as_millis(); sum += start.elapsed().as_millis();
} }
println!("average {} time: {} millis.", speedtest.to_uppercase(), sum / 1000); println!(
"average {} time: {} millis.",
speedtest.to_uppercase(),
sum / u128::from(iterations)
);
} else { } else {
run_ui(); run_ui();
run_ui_toml(); run_ui_toml();