Skip to content

Conversation

jakepetroules
Copy link
Member

Disable tests relying on the executable since those aren't yet supported by SwiftPM, and tweak a couple platform-conditional pieces of code to support the Windows path.

@czechboy0
Copy link
Contributor

Could you also enable Windows CI like on the runtime repo?

@czechboy0
Copy link
Contributor

C:\swift-openapi-generator\Tests\OpenAPIGeneratorReferenceTests\FileBasedReferenceTests.swift:102: error: FileBasedReferenceTests.testPetstore : failed - 

Directory contents not equal:

  File under test: Types.swift

  Reference file: ReferenceSources/Petstore/Types.swift

  Diff output: (see below)

--- begin diff output --------------------------------------

failed: Error Domain=NSCocoaErrorDomain Code=260 "The file doesn’t exist."

--- end diff output ----------------------------------------

Seems the integration test isn't passing yet - could you take a look, @jakepetroules?

Disable tests relying on the executable since those aren't yet supported by SwiftPM, and tweak a couple platform-conditional pieces of code to support the Windows path.
@czechboy0 czechboy0 added the 🔨 semver/patch No public API change. label Apr 4, 2025
@czechboy0 czechboy0 enabled auto-merge (squash) April 4, 2025 19:26
@czechboy0 czechboy0 merged commit 02cab48 into apple:main Apr 4, 2025
40 of 41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 semver/patch No public API change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants