From feacf89548c487ef98e1ceeac6997c91c9f6bcfa Mon Sep 17 00:00:00 2001 From: Reid <61492567+reidliu41@users.noreply.github.com> Date: Tue, 11 Mar 2025 06:50:46 +0800 Subject: [PATCH] docs: improve integration test doc (#1502) # What does this PR do? [Provide a short summary of what this PR does and why. Link to relevant issues if applicable.] It should use `export` for env var for api key. [//]: # (If resolving an issue, uncomment and update the line below) [//]: # (Closes #[issue-number]) ## Test Plan [Describe the tests you ran to verify your changes with result summaries. *Provide clear instructions so the plan can be easily re-executed.*] [//]: # (## Documentation) Signed-off-by: reidliu Co-authored-by: reidliu --- tests/integration/README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/integration/README.md b/tests/integration/README.md index c7a8b4722..beb234740 100644 --- a/tests/integration/README.md +++ b/tests/integration/README.md @@ -55,7 +55,7 @@ Running all inference tests for a number of models: TEXT_MODELS=meta-llama/Llama-3.1-8B-Instruct,meta-llama/Llama-3.1-70B-Instruct VISION_MODELS=meta-llama/Llama-3.2-11B-Vision-Instruct EMBEDDING_MODELS=all-MiniLM-L6-v2 -TOGETHER_API_KEY=... +export TOGETHER_API_KEY= pytest -s -v tests/api/inference/ \ --stack-config=together \ @@ -67,7 +67,7 @@ pytest -s -v tests/api/inference/ \ Same thing but instead of using the distribution, use an adhoc stack with just one provider (`fireworks` for inference): ```bash -FIREWORKS_API_KEY=... +export FIREWORKS_API_KEY= pytest -s -v tests/api/inference/ \ --stack-config=inference=fireworks \