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

Fix catch2 adapter mulit-line benchmark name parsing #351

Closed
epompeii opened this issue Mar 22, 2024 · 1 comment
Closed

Fix catch2 adapter mulit-line benchmark name parsing #351

epompeii opened this issue Mar 22, 2024 · 1 comment

Comments

@epompeii
Copy link
Member

When a benchmark name is too long catch2 wraps the benchmark name:
benchmark-action/github-action-benchmark#55

Add this test file: https://github.com/benchmark-action/github-action-benchmark/blob/7bec15aacc831ab865c072311324173b9a61a3dc/test/data/extract/catch2_output.txt

For anyone here in the meantime, use this workaround:

As a workaround we can add #define CATCH_CONFIG_CONSOLE_WIDTH 200 (or any width needed) in the benchmark source to overwrite catch2's default output width of 80 characters to prevent long benchmark names wrapping.

benchmark-action/github-action-benchmark#46 (comment)

@epompeii
Copy link
Member Author

This bug as been fixed and the above should now work: https://bencher.dev/docs/reference/changelog/#v0423

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

No branches or pull requests

1 participant