Incorrect parameter ordering for CancellationToken in generated RequestBuilder code

[johnholliday]

What are you generating using Kiota, clients or plugins?

API Client/SDK

In what context or format are you using Kiota?

Windows executable

Client library/SDK language

Csharp

Describe the bug

The generated API client code fails to compile with the following error:

Argument 3: cannot convert from 'System.Threading.CancellationToken' 
to 'System.Collections.Generic.Dictionary<string, 
Microsoft.Kiota.Abstractions.Serialization.ParsableFactory
<Microsoft.Kiota.Abstractions.Serialization.IParsable>>?'

Expected behavior

The generated code should compile without errors. Specifically, the 'cancellationToken' parameter should be preceded by 'null', as in:

var requestInfo = ToDeleteRequestInformation(requestConfiguration);
return await RequestAdapter.SendAsync<UntypedNode>(requestInfo, default, **null**, cancellationToken).ConfigureAwait(false);

How to reproduce

1. Create a new .NET class library project.
2. Generate the C# Api client.
3. Compile the project.

Open API description file

No response

Kiota Version

1.14.0

Latest Kiota version known to work for scenario above?(Not required)

No response

Known Workarounds

Manually edit the generated code.

Configuration

Windows 11 x64

Debug output

Click to expand log

```

</details>


### Other information

_No response_

[andrueastman]

Thanks for raising this @johnholliday

Is this occurring on all methods or just a specific generated method? Are you able to share a sample of the description that replicates this scenario?

[johnholliday]

@andrueastman Only on DeleteAsync and GetAsync. Sample project attached.
Kiota.Test.zip

[andrueastman]

Authored #4700 to resolve

[johnholliday]

Authored #4700 to resolve

There is still a minor problem that causes CS8625 warnings on code like the following:

#nullable enable
        public async Task<UntypedNode?> DeleteAsync(Action<RequestConfiguration<DefaultQueryParameters>>? requestConfiguration = default, CancellationToken cancellationToken = default)
        {
#nullable restore
#else
        public async Task<UntypedNode> DeleteAsync(Action<RequestConfiguration<DefaultQueryParameters>> requestConfiguration = default, CancellationToken cancellationToken = default)
        {
#endif
            var requestInfo = ToDeleteRequestInformation(requestConfiguration);
            return await RequestAdapter.SendAsync<UntypedNode>(requestInfo, /* CS8625 ==> */default, null, cancellationToken).ConfigureAwait(false);
        }

warning CS8625: Cannot convert null literal to non-nullable reference type

Without delving further to determine if this manual fixup causes other issues, the warning goes away when patched as follows:

return await RequestAdapter.SendAsync<UntypedNode>(requestInfo, default!, null, cancellationToken).ConfigureAwait(false);

However, in my opinion, the generated code should compile without warnings and should not require manual fixups.

[andrueastman]

Thanks for the feedback @johnholliday

following up via #4727