Format OData cast operationIds

[peombwa]

Overview

This PR fixes #1228 by ensuring OData cast operationIds are resolved to format that can be interpreted by AutoREST. This is done by merging [ACTION] and [AS_CAST_TYPE] segments on an OData cast operationId. For example:

• directory.ListDeletedItems.AsApplication -> directory_ListDeletedItemsAsApplication -> Get-MgDirectoryDeletedItemAsApplication.

This PR is a continuation of microsoftgraph/msgraph-metadata#317.

Testing Instructions

• See unit tests.

[irvinesunday]

Other than the nitpick typo, this LGTM. However, we could accomplish the same by pushing a small change in the conversion lib. See: https://github.com/microsoft/OpenAPI.NET.OData/blob/ef336182076c8a7372af815df71d25d1be04de81/src/Microsoft.OpenApi.OData.Reader/Operation/ODataTypeCastGetOperationHandler.cs#L179

[peombwa]

Other than the nitpick typo, this LGTM. However, we could accomplish the same by pushing a small change in the conversion lib. See: https://github.com/microsoft/OpenAPI.NET.OData/blob/ef336182076c8a7372af815df71d25d1be04de81/src/Microsoft.OpenApi.OData.Reader/Operation/ODataTypeCastGetOperationHandler.cs#L179

Yes, let's tackle that as part of microsoft/OpenAPI.NET.OData#361. All operationIds should be standardized in the conversion lib.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

100.0% Coverage
0.0% Duplication