Distinct same types

[xDDDx]

This solves the issue #424

The assemblies are loaded twice and so the types

[ghost]

All CLA requirements met.

[justinyoo]

@xDDDx Thanks for the PR! As long as I understand correctly, it needs to be merged after #424, right?

Would you be able to add some test cases to support your changes?

[xDDDx]

@justinyoo I added some DistinctBy tests. The problem with DistinctBy is, that it is part of .net6 in System.Linq but not in net5 and below. I`m not sure what is the right approach to use it with multiple frameworks

I'm not sure about #424 This can be a wrong fix. What if I'm realy defined two different endpoints and gived them same operation id? Or is it also a problem for functions like this with getand poste.g.?

[OpenApiOperation(operationId: "health-check")]
public async Task<HttpResponseData> Run([HttpTrigger(AuthorizationLevel.Function, "get", "post", Route = "check")] HttpRequestData req)

[justinyoo]

@justinyoo I added some DistinctBy tests. The problem with DistinctBy is, that it is part of .net6 in System.Linq but not in net5 and below. I`m not sure what is the right approach to use it with multiple frameworks

I'm not sure about #424 This can be a wrong fix. What if I'm realy defined two different endpoints and gived them same operation id? Or is it also a problem for functions like this with getand poste.g.?

[OpenApiOperation(operationId: "health-check")]
public async Task<HttpResponseData> Run([HttpTrigger(AuthorizationLevel.Function, "get", "post", Route = "check")] HttpRequestData req)

1. As we support .NET Core 3.1, .NET 5 and .NET 6, our change must be applicable to all those three versions. I saw you've changed it from DistinctBy to DistinctByProperty, which should be fine.
2. At the beginning, the message body tagged Only add unique openApiReferenceIds to the exported types #424 That's why I asked whether it's relevant or not.
3. According to the OpenAPI spec, there must be only one unique operation ID in one OpenAPI document.
4. Although AzFunc support multiple verbs, OpenAPI extension assumes each method only takes one verb. If there are more than one verbs declared in one method, only the first one is taken.

[justinyoo]

LGTM!