[Repo Assist] test: extend OperationCompiler unit tests (+12 tests)

[github-actions]

🤖 This PR was created by Repo Assist, an automated AI assistant.

Summary

Extends Schema.OperationCompilationTests.fs with 12 new unit tests covering three previously untested code paths in OperationCompiler.

Test count: 268 → 280

New Coverage

asAsync=true mode (4 tests)

The OperationCompiler has a asAsync: bool constructor parameter that switches return types from Task<T> to Async<T>. Before this PR there were zero unit tests exercising this path.

• GET returning JSON string → Async<string> not Task<string>
• POST with no body → Async<unit> not Task<unit>
• Method name generation still works in async mode
• CancellationToken is still injected last in async mode

DELETE / PUT operations (6 tests)

The existing tests only covered GET and POST. Added coverage for:

• DELETE endpoint — generates method, returns Task<unit> on 204, path param ordering
• PUT endpoint — generates method, path + body + CT ordering, Task<T> return when JSON response

Header parameters (2 tests)

• Required header params are included as method parameters
• Header parameter names are camelCased (X-Api-Version → xApiVersion)
• Required header params are non-optional (IsOptional = false)

Test Status

dotnet build tests/SwaggerProvider.Tests/SwaggerProvider.Tests.fsproj -c Release  ✅
dotnet tests/SwaggerProvider.Tests/bin/Release/net10.0/SwaggerProvider.Tests.dll  ✅
Total: 280, Errors: 0, Failed: 0, Skipped: 1 (pre-existing)

Formatting validated with Fantomas.

Generated by 🌈 Repo Assist, see workflow run. Learn more.

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@97143ac59cb3a13ef2a77581f929f06719c7402a

[Copilot]

Pull request overview

Extends Schema.OperationCompilationTests.fs with additional unit tests to cover previously untested OperationCompiler behaviors, including async (asAsync=true) return types, additional HTTP methods, and header-parameter handling.

Changes:

• Add helper to compile schemas with asAsync=true and new assertions for Async<'T> return types.
• Add DELETE and PUT operation compilation tests (method generation, parameter ordering, return type resolution).
• Add header-parameter compilation tests (inclusion, camelCasing, required vs optional).

---

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

[sergey-tihon]

@copilot format, fix build and address inline commnets

[Copilot]

@copilot format, fix build and address inline commnets

Done in e0bbd63. I fixed the inline review issue by updating the section header to match actual coverage (DELETE / PUT operations), then re-ran format/build/tests locally (CheckFormat, dotnet build tests/SwaggerProvider.Tests/..., and the test DLL), all passing.