Skip to content

Conversation

@ccoVeille
Copy link
Owner

@ccoVeille ccoVeille commented Oct 29, 2025

I faced an issue when testing with "string" string
Then I realized that using quotes would make it clearer.

I chose to use %#q (so backticks are used when possible) for a better readability in logs, as it would avoid reporting with escaped quotes.

  • chore: add missing test before a refactoring
  • fix: make sure to report the exact string that was provided
  • feat: use quotes when reporting issue with string conversion

@codecov
Copy link

codecov bot commented Oct 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (0baa892) to head (858b8b0).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #125   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          207       207           
=========================================
  Hits           207       207           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

I faced an issue when testing with "string" string

Then I realized that using quotes would make it clearer.

I chose to use %#q (so backticks are used when possible) for a better
readability in logs, as it would avoid reporting with escaped quotes.
@ccoVeille ccoVeille merged commit 28158aa into main Oct 29, 2025
15 checks passed
@ccoVeille ccoVeille deleted the string-reporting branch October 29, 2025 16:25
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.

2 participants