Skip to content

Update testing documentation to include withApp(configure:) method #1086

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 10 commits into from
Aug 18, 2025

Conversation

maartene
Copy link
Contributor

As per Tim's instructions in this forum thread https://forums.swift.org/t/vapor-testing-cant-find-swift-openapi-generator-created-endpoints/81224/13, I'm clarifying the Testing documentation about how to use the withApp configure method.

I.e., instead of creating your own withApp method, you should use the withApp(configure:) method instead.

@maartene maartene requested review from 0xTim and gwynne as code owners July 23, 2025 15:13
@maartene
Copy link
Contributor Author

I've only updated the English version for now. Once you feel this is of sufficient quality, I'll look at other languages.


```swift
private func withApp(_ test: (Application) async throws -> ()) async throws {
private func withAppIncludingDB(_ test: (Application) async throws -> ()) async throws {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If @0xTim and @gwynne are fine with the name withAppIncludingDB, I'm fine too

@fpseverino fpseverino merged commit 0071b73 into vapor:main Aug 18, 2025
1 check passed
@penny-for-vapor penny-for-vapor bot mentioned this pull request Aug 18, 2025
10 tasks
ptoffy added a commit that referenced this pull request Aug 21, 2025
ptoffy added a commit that referenced this pull request Aug 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants