NXC-301: Update ToQueryString and usage

[PJvGrol]

This pull request refactors the way query strings are generated and appended to API URLs throughout the SDK, replacing the previous CreateUriQuery method with a new, more robust ToQueryString implementation. It also updates all usages to ensure proper URL encoding of query parameters, which improves reliability and correctness when dealing with special characters. Unit tests are updated accordingly to match the new encoding behavior.

Core refactoring and improvements:

• Replaced the CreateUriQuery method with a new ToQueryString method in BaseService, supporting both Dictionary<string, string> and object inputs, and ensuring all query parameter keys are camelCased and values are properly URL-encoded.
• Updated all service methods to use ToQueryString instead of CreateUriQuery for building query strings in API requests, affecting files such as NexusAPIService.cs, CustomerPersonService.cs, CustomerService.cs, DocumentStoreRecordService.cs, and DocumentStoreTypeService.cs. [1] [2] [3] [4] [5] [6] [7]

Testing and encoding fixes:

• Updated test cases to expect URL-encoded query parameters (e.g., : encoded as %3A), ensuring tests reflect the new query string behavior and encoding. [1] [2] [3] [4]

Minor codebase updates:

• Added missing using System.Globalization; and using System.Reflection; directives required by the new ToQueryString implementation in BaseService.cs.

[Copilot]

Pull request overview

This PR refactors how query strings are constructed across the SDK by replacing the prior CreateUriQuery approach with a new ToQueryString implementation, and updates affected service calls and tests so query parameters are properly URL-encoded (notably for characters like : and |).

Changes:

• Introduced BaseService.ToQueryString(...) and updated multiple service methods to use it when building request URLs.
• Updated test expectations to match URL-encoded query parameter behavior.
• Added required using directives for the new reflection/casing logic in BaseService.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
tests/Nexus.Crypto.SDK.Tests/TransactionControllerTests.cs	Updates mocked request URI to expect encoded query parameters.
tests/Nexus.Crypto.SDK.Tests/NexusLabelApiSdkTests.cs	Updates mocked request URIs (and a Dictionary type usage) to align with encoded query strings.
src/Nexus.Crypto.SDK/Services/BaseService.cs	Replaces CreateUriQuery with ToQueryString (dictionary + object overload) and adds camel-casing + URL encoding.
src/Nexus.Crypto.SDK/NexusAPIService.cs	Switches several endpoints to use the new ToQueryString for dictionary-based query parameters.
src/Nexus.Crypto.SDK/Services/DocumentStoreTypeService.cs	Updates URL construction to use BaseService.ToQueryString.
src/Nexus.Crypto.SDK/Services/DocumentStoreRecordService.cs	Updates URL construction to use BaseService.ToQueryString.
src/Nexus.Crypto.SDK/Services/CustomerService.cs	Updates URL construction to use BaseService.ToQueryString.
src/Nexus.Crypto.SDK/Services/CustomerPersonService.cs	Updates URL construction to use BaseService.ToQueryString.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.