Generate Multiple interfaces based on first Tag

[kirides]

This PR adds an additional mode for --multiple-interfaces called ByTag which uses the first Tag of the operation to generate the interface name.

This PR also adds a BREAKING change as --multiple-interfaces now takes in an enumeration of Unset, ByEndpoint and ByTag.
The source generator .refitter file has the same breaking change, as it now takes its value as string like this:

{
  "multipleInterfaces": "ByTag" // or ByEndpoint
}

Generated sample:

namespace GeneratedCode
{
    // No/Empty first Tag 
    public interface IDocumentV1Api
    {
        [Get("/api/greet-unsafe")]
        Task GreetUnsafe([Query] string name);
    }

    // tag "Greeting"
    public interface IGreetingApi
    {
        [Get("/api/greet")]
        Task Greet([Query] string name);
    }

    // tag "OpenApi"
    public interface IOpenApiApi
    {
        [Get("/api/custom-docs")]
        Task<ICollection<Item>> CustomDocs([Query] string name);

        [Get("/api/some-other-thing")]
        Task<ICollection<Item>> SomeOtherThing([Query] string name);
    }


}

spec:

openapi: 3.0.1
# ...
info:
  title: Document V1
paths:
  /api/greet-unsafe:
    get:
      tags:
        - ''
      parameters:...
      responses:
        '200':...
  /api/greet:
    get:
      tags:
        - Greeting
      parameters:...
      responses:...
  /api/custom-docs:
    get:
      tags:
        - OpenApi
      parameters:...
      responses:
        '200':...
  /api/some-other-thing:
    get:
      tags:
        - OpenApi
      parameters:...
      responses:
        '200':...

[christianhelle]

@kirides this is some very good stuff! I don't mind the breaking change since as far as I know, I'm the only one using --multiple-interfaces but we should update the source generator README

[christianhelle]

@all-contributors please add @kirides for code

[allcontributors]

@christianhelle

I've put up a pull request to add @kirides! 🎉

[christianhelle]

@kirides The builds are failing because --multiple-interfaces is now expecting an extra argument

Can I ask you to update line 60 in the file .github\workflows\template.yml to

${{ inputs.command }} ./openapi.${{ inputs.format }} --namespace "$namespace.MultipleInterfaces" --output "MultipleInterfaces$outputPath" --multiple-interfaces ByEndpoint --no-logging

and line 116-119 in the file test\smoke-tests.ps1 to:

Write-Host "dotnet run --project ../src/Refitter/Refitter.csproj ./openapi.$format --namespace $namespace.MultipleInterfaces --output ./GeneratedCode/MultipleInterfaces$outputPath --multiple-interfaces ByEndpoint --no-logging"
$process = Start-Process "dotnet" `
    -Args "run --project ../src/Refitter/Refitter.csproj ./openapi.$format --namespace $namespace.MultipleInterfaces --output ./GeneratedCode/MultipleInterfaces$outputPath --multiple-interfaces ByEndpoint --no-logging" `
    -NoNewWindow `
    -PassThru

[christianhelle]

@kirides if its not too much trouble, please update src\Refitter.SourceGenerator\README.md so the example .refitter file and explanations match the proposed changes

[kirides]

Sure will do those things.
Just gotta get on to a PC :)

[christianhelle]

@kirides thanks!

Remember to enjoy the weekend first 😄

[christianhelle]

This feature is now released to NuGet as v0.7.2