Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.53%. Comparing base(524b37c) to head(2828e06).
Report is 8 commits behind head on main.

@@            Coverage Diff             @@##             main    #3205      +/-   ##==========================================- Coverage   96.54%   96.53%   -0.01%==========================================  Files          53       53                Lines        8935     8930       -5       Branches     1642     1642              ==========================================- Hits         8626     8621       -5  Misses        186      186                Partials      123      123

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

I have updated these without using therepr version of f-string? Will update...

We should be able to get rid of the strings for both of the files using the representation completely as the message does not seem to really provide any benefit withpytest and its assertion rewriting. Or am I am missing something?

Have amended so that this PR recreates completely what the existing code does (hopefully).

I think you are saying that pytest gives us the same information implicitly, so the strings can be removed completely?

Could keep this PR self-contained and then do other changes subsequently?

I think you are saying that pytest gives us the same information implicitly, so the strings can be removed completely?

Yes.

I think you are saying that pytest gives us the same information implicitly, so the strings can be removed completely?

Yes.

Implemented, and the PR is still self-contained.