Skip to content

Add swift-http-server code for testing#76

Merged
FranzBusch merged 4 commits intoapple:mainfrom
xbhatnag:server-test-code
Feb 2, 2026
Merged

Add swift-http-server code for testing#76
FranzBusch merged 4 commits intoapple:mainfrom
xbhatnag:server-test-code

Conversation

@xbhatnag
Copy link
Collaborator

Take HTTPServer source from swift-http-server repo and add it to this repo so that it can be used for testing the client in future commits.

This is needed because adding the swift-http-server repo as a Swift dependency causes conflicts with the AsyncStreaming target that is common to both.

@xbhatnag xbhatnag added the 🔨 semver/patch No public API change. label Jan 30, 2026
@xbhatnag xbhatnag self-assigned this Jan 30, 2026
@xbhatnag xbhatnag linked an issue Jan 30, 2026 that may be closed by this pull request
@xbhatnag
Copy link
Collaborator Author

Bleh, there are API breakages because we've changed HTTPServer from what it used to be. How do we reconcile the differences? Or should we just call the swift-http-server one something different and let there be two different APIs?

@guoye-zhang
Copy link
Contributor

API breakage is fine since we aren't 1.0 yet

Take HTTPServer source from `swift-http-server` repo
and add it here so that it can be used for testing the
client in future commits.

This is needed because adding the `swift-http-server`
repo as a Swift dependency causes conflicts with the
`AsyncStreaming` target that is common to both.
@xbhatnag
Copy link
Collaborator Author

Was able to avoid any API breakage by creating a separate target meant only for testing.

@FranzBusch FranzBusch enabled auto-merge (squash) February 2, 2026 16:35
@FranzBusch FranzBusch merged commit 6de0c8c into apple:main Feb 2, 2026
21 checks passed
@xbhatnag xbhatnag deleted the server-test-code branch February 2, 2026 16:49
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.

Create a compliance test suite

3 participants