microsoft / vs-streamjsonrpc

The StreamJsonRpc library offers JSON-RPC 2.0 over any .NET Stream, WebSocket, or Pipe. With bonus support for request cancellation, client proxy generation, and more.
Other
730 stars 149 forks source link

Fix System.Text.Json formatter's support for named arguments object #1045

Closed AArnott closed 3 months ago

AArnott commented 3 months ago

Replacing the Assumes.NotNull with a non-throwing null check brings this formatter in line with the other formatters. The added unit test applies to all formatters and demonstrated before the fix that only System.Text.Json failed the test. After the fix all formatters pass.

Fixes #1028