Skip to content

test: fix: correct assertion#8732

Open
whoisj wants to merge 4 commits intomainfrom
jwyman/tri-929-2604-l0_cuda_shared_memory-base
Open

test: fix: correct assertion#8732
whoisj wants to merge 4 commits intomainfrom
jwyman/tri-929-2604-l0_cuda_shared_memory-base

Conversation

@whoisj
Copy link
Copy Markdown
Contributor

@whoisj whoisj commented Apr 10, 2026

This change adjusts the capitalization used in assertions to validate specific error messages.

'Request JSON size of' -> 'request JSON size of'.

CI Pipeline ID: 48225152

@whoisj whoisj requested a review from yinggeh April 10, 2026 16:51
@whoisj whoisj added PR: ci Changes to our CI configuration files and scripts PR: fix A bug fix PR: test Adding missing tests or correcting existing test labels Apr 10, 2026
@yinggeh
Copy link
Copy Markdown
Contributor

yinggeh commented Apr 10, 2026

Also in server/qa/L0_cuda_shared_memory/cuda_shared_memory_test.py and server/qa/L0_http/http_test.py

@whoisj
Copy link
Copy Markdown
Contributor Author

whoisj commented Apr 10, 2026

Also in server/qa/L0_cuda_shared_memory/cuda_shared_memory_test.py and server/qa/L0_http/http_test.py

Good catch. Fixed up.

whoisj added 4 commits April 17, 2026 11:52
This change adjusts the capitalization used in assertions to validate specific error messages.

'Request JSON size of' -> 'request JSON size of'.
add --allow-client-shm=true to server start options

add .gitignore to L0_http
@whoisj whoisj force-pushed the jwyman/tri-929-2604-l0_cuda_shared_memory-base branch from 37fcba4 to 4328eb8 Compare April 17, 2026 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: ci Changes to our CI configuration files and scripts PR: fix A bug fix PR: test Adding missing tests or correcting existing test

Development

Successfully merging this pull request may close these issues.

2 participants