Skip to content

docs: update test line references in MessagePack section#4580

Open
WolfMTK wants to merge 1 commit intolitestar-org:mainfrom
WolfMTK:fix-docs-test
Open

docs: update test line references in MessagePack section#4580
WolfMTK wants to merge 1 commit intolitestar-org:mainfrom
WolfMTK:fix-docs-test

Conversation

@WolfMTK
Copy link
Copy Markdown

@WolfMTK WolfMTK commented Feb 5, 2026

Description

Fixed a typo has been corrected in the strings pertaining to request testing with MessagePack.

Closes

@WolfMTK WolfMTK requested review from a team as code owners February 5, 2026 12:16
@github-actions github-actions bot added area/docs This PR involves changes to the documentation size: small type/docs pr/external Triage Required 🏥 This requires triage labels Feb 5, 2026
Copy link
Copy Markdown
Member

@sobolevn sobolevn left a comment

Choose a reason for hiding this comment

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

For future PRs: please attach links to changed file numbers / screenshots of the changed docs.

Thank you!

The change seems correct:

def test_msgpack_app() -> None:
test_data = {"name": "Moishe Zuchmir", "age": 30, "programmer": True}
with TestClient(app=msgpack_app) as client:
response = client.post("/", content=encode_msgpack(test_data))
assert response.json() == test_data

@sobolevn sobolevn enabled auto-merge (squash) February 5, 2026 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/docs This PR involves changes to the documentation pr/external size: small Triage Required 🏥 This requires triage type/docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants