Closes #1732
Fixes a bug where existing query parameters would have their positions reversed #1732

• Extracted logic into a method
• If the existing Query is empty we early return
  • This early return prevents the allocation of an empty 300 byte NameValueCollection
• Otherwise the query is parsed with each key pair added
  • There might be room for improvement here by using a linq Select method. Select would know the length of the collection and InsertRange would be able to bulk copy the values. This could only be done if we know for certain that the key can't be empty and Where is not required.

Before

Query changes

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.63%. Comparing base (6ebeda5) to head (7291cca).
Report is 47 commits behind head on main.

❗ Current head 7291cca differs from pull request most recent head 713e067

Please upload reports for the commit 713e067 to get more accurate results.

@@            Coverage Diff             @@
##             main    #1737      +/-   ##
==========================================
- Coverage   87.73%   84.63%   -3.11%     
==========================================
  Files          33       36       +3     
  Lines        2348     2512     +164     
  Branches      294      327      +33     
==========================================
+ Hits         2060     2126      +66     
- Misses        208      304      +96     
- Partials       80       82       +2     

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

Nice, I really want to make some kind of live test, for example two console apps that can execute to test the functionality more fully. There's many parts of the code which are either untestable or have no existing tests.
If you have any ideas how we can achieve this, please shout.

I really want to make some kind of live test, for example two console apps that can execute to test the functionality more fully.

Like a sandbox project? It could be a single file with an interface and dummy http request handler.

There's many parts of the code which are either untestable or have no existing tests.

Which parts did you have in mind? When experimenting with the source generator I have added snapshot testing. The current system is okay because the generator is basic, but any future improvements will need a better test setup. See mapperly as a good example (not biased)😄

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.