Skip to content

Remove TestClient from documentation - #83

Merged
AndreuCodina merged 4 commits into
mainfrom
feature/remove-testclient-from-documentation
Mar 20, 2026
Merged

Remove TestClient from documentation#83
AndreuCodina merged 4 commits into
mainfrom
feature/remove-testclient-from-documentation

Conversation

@AndreuCodina

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings March 20, 2026 17:51

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the testing documentation to remove reliance on FastAPI’s TestClient and shift examples toward directly building an async ServiceProvider, aligning the docs with async-first testing patterns.

Changes:

  • Replaces TestClient(app)-based FastAPI fixture examples with async with services.build_service_provider().
  • Updates console-application guidance to clarify why ServiceProvider can’t be imported directly from main().
  • Consolidates “globally override services” examples into a single async fixture example.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/pages/testing.md
Comment thread docs/pages/testing.md
@coveralls

coveralls commented Mar 20, 2026

Copy link
Copy Markdown
Collaborator

Coverage Status

coverage: 99.45%. remained the same
when pulling e0f21a2 on feature/remove-testclient-from-documentation
into 24b4eaa on main.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/pages/testing.md
Comment thread docs/pages/testing.md Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/pages/testing.md Outdated
@AndreuCodina
AndreuCodina merged commit c0c2749 into main Mar 20, 2026
22 checks passed
@AndreuCodina
AndreuCodina deleted the feature/remove-testclient-from-documentation branch March 20, 2026 18:08
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