Fix missing Content-Type [Headers]

[christianhelle]

This resolves #617

This pull request includes several changes to enhance the header generation functionality in the Refitter.Core project. The key changes involve renaming methods to reflect their purpose better, adding support for generating Content-Type headers, and updating tests to verify the new functionality.

With the changes in the pull request, the following OpenAPI document:

openapi: 3.0.1
paths:
  /foo:
    post:
      requestBody:
        content:
          application/json:
            schema:
              type: string
        required: true
      responses:
        '200':
          content:
            application/json:
              schema:
                type: string

Generate the following Refit interface:

public partial interface IApiClient
{
    [Headers("Accept: application/json", "Content-Type: application/json")]
    [Post("/foo")]
    Task<string> Foo([Body] string body);
}

Header Generation Enhancements:

• Renamed the GenerateAcceptHeaders method to GenerateHeaders and added logic to generate Content-Type headers if specified in the settings. [1] [2] [3] [4] [5] [6] [7] [8]

• Added a new property AddContentTypeHeaders to the RefitGeneratorSettings class to control the generation of Content-Type headers.

Test Updates:

• Updated existing tests in src/Refitter.SourceGenerator.Tests/CustomGenerated/CustomGenerated.cs to include Content-Type headers in the Headers attribute. [1] [2] [3] [4] [5] [6]

• Added new test classes RequestResponseHeadersTests and ResponseHeadersTests to verify the generation of Accept and Content-Type headers. [1] [2]

• Modified tests in src/Refitter.Tests/SwaggerPetstoreApizrTests.cs and src/Refitter.Tests/SwaggerPetstoreTests.cs to validate the presence or absence of Accept headers based on the OpenAPI specification version. [1] [2]

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.60%. Comparing base (125b398) to head (fd62613).
Report is 19 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main     #619       +/-   ##
===========================================
- Coverage   98.97%   85.60%   -13.38%     
===========================================
  Files          55       56        +1     
  Lines        2639     3195      +556     
===========================================
+ Hits         2612     2735      +123     
- Misses          4      419      +415     
- Partials       23       41       +18     

Flag	Coverage Δ
unittests	85.60% <100.00%> (-13.38%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[christianhelle]

@all-contributors please add @Metziell for bugs

[allcontributors]

@christianhelle

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

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud