git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] perf-lib: fix missing test titles in output

From
Derrick Stolee <derrickstolee@github.com>
Date
Jun 16, 2022, 13:03 UTC
Message-ID
<6671fd2c-ffcb-6278-494f-2d3cfe4c36d1@github.com>
In-Reply-To
<YqrXLO5oMYeOr2PB@coredump.intra.peff.net>
On 6/16/2022 3:09 AM, Jeff King wrote:
Show 20 quoted lines
> Commit 5dccd9155f (t/perf: add iteration setup mechanism to perf-lib,
> 2022-04-04) modified the parameter parsing of test_wrapper() such that
> the test title was no longer in $1, and is instead in $test_title_.
> 
> We correctly pass the new variable to the code which outputs the title
> to the log, but missed the spot in test_wrapper() where the title is
> written to the ".descr" file which is used to produce the final output
> table. As a result, all of the titles are missing from that table (or
> worse, using whatever was left in $1):
> 
>   $ ./p0000-perf-lib-sanity.sh
>   [...]
>   Test           this tree
>   ------------------------------
>   0000.1:        0.01(0.01+0.00)
>   0000.2:        0.01(0.00+0.01)
>   0000.4:        0.00(0.00+0.00)
>   0000.5: true   0.00(0.00+0.00)
>   0000.7:        0.00(0.00+0.00)
>   0000.8:        0.00(0.00+0.00)
Yikes!
 
Show 10 quoted lines
> After this patch, we get the pre-5dccd9155f output:
> 
>   Test                                                       this tree
>   --------------------------------------------------------------------------
>   0000.1: test_perf_default_repo works                       0.00(0.00+0.00)
>   0000.2: test_checkout_worktree works                       0.01(0.00+0.01)
>   0000.4: export a weird var                                 0.00(0.00+0.00)
>   0000.5: éḿíẗ ńöń-ÁŚĆÍÍ ćḧáŕáćẗéŕś                          0.00(0.00+0.00)
>   0000.7: important variables available in subshells         0.00(0.00+0.00)
>   0000.8: test-lib-functions correctly loaded in subshells   0.00(0.00+0.00)
Good catch. LGTM.
-Stolee
Previous: Jeff KingNext: Neeraj Singh (WINDOWS-SFS)
Message 2 of 3 in “perf-lib: fix missing test titles in output”
  1. perf-lib: fix missing test titles in outputJeff King, Jun 16, 2022
  2. Derrick StoleeJun 16, 2022
  3. Neeraj Singh (WINDOWS-SFS)Jun 16, 2022

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.