Bumps OpenAPIKit to version 3.0.0-alpha.7 fixing an issue with trailing slashes.

[miguel-arrf]

Motivation

As discussed in #51, there was an issue with paths containing trailing slashes. The problem was identified in the OpenAPIKit dependency. It was fixed in the PR mattpolzin/OpenAPIKit#274.

Modifications

This PR updates the OpenAPIKit dependency version from 3.0.0-alpha.6 to 3.0.0-alpha.7 (available here), which includes the mentioned PR.

Result

If a path ends with a trailing slash, such as foo/bar/, the generated functions will now correctly call the path foo/bar/ instead of (as previously) foo/bar.

Test Plan

Replaced an endpoint in the reference test to use a path with a trailing slash.

Resolves

• Resolves Issue with paths ending with a slash #51

[simonjbeaumont]

@miguel-arrf Thanks for driving this! 🙏

⚠️ No steps have been taken yet to test this PR. The existing tests should be updated to include paths with trailing slashes.

Is this something you intend to do in this PR?

[simonjbeaumont]

@swift-server-bot add to allowlist

[czechboy0]

You could just change one of the paths in the integration test to end with a slash instead of adding another path+operation. But it'd be good for our tests to include this case.

[simonjbeaumont]

Thanks for the fix. Based on the discussion here and on the issue, I'll mark this as request changes, until we get the test changed.

[czechboy0]

Thanks @miguel-arrf for updating the tests, but I don't think we need a new operation for it, as those are pretty expensive to maintain (as you just saw when adding a new one), so we add new ones sparingly, only when we can't extend existing ones. An easier alternative is to modify the path of one of the existing operations, and update the corresponding test.

[miguel-arrf]

@czechboy0 Ah, I see! Thank you, I'll update it accordingly :)!

[miguel-arrf]

(I still have some more tests to change, because of the Server!)

[czechboy0]

Looks great, thank you, @miguel-arrf! 🙏

[simonjbeaumont]

@miguel-arrf Looks great!

Would you like to update the PR description given you've gone to the effort of sorting the testing out?

⚠️ No steps have been taken yet to test this PR. The existing tests should be updated to include paths with trailing slashes.

EDIT: Actually, don't worry—I'll do it.

Thanks for the contribution!

[miguel-arrf]

Thanks @simonjbeaumont ! ☺️