Skip to content

Commit

Permalink
Overiding pytest ini
Browse files Browse the repository at this point in the history
  • Loading branch information
johnjosephhorton committed Dec 31, 2024
1 parent 54ebaba commit 356cf8b
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 19 deletions.
7 changes: 4 additions & 3 deletions .github/workflows/test_suite.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,10 @@ jobs:
shell: bash
run: poetry install --with dev

- name: Run integegation
shell: bash
run: poetry run make test-starter-tutorial

- name: Run edsl tests
shell: bash
run: poetry run make test
Expand All @@ -50,6 +54,3 @@ jobs:
shell: bash
run: poetry run make test-doctests

- name: Run integegation
shell: bash
run: poetry run make test-starter-tutorial
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -213,7 +213,7 @@ test-notebooks:

test-starter-tutorial:
@echo "Testing starter tutorial..."
pytest -v integration/active/test_notebooks.py -k docs/notebooks/hello_world.ipynb
pytest -v integration/active/test_notebooks.py -k docs/notebooks/hello_world.ipynb --override-ini config_file=integration/pytest.ini

# .PHONY: test-notebooks
# test-notebooks: ## Run the notebooks tests
Expand Down
16 changes: 1 addition & 15 deletions integration/pytest.ini
Original file line number Diff line number Diff line change
@@ -1,23 +1,9 @@
[pytest]
env =
EDSL_RUN_MODE=development-testrun
EDSL_DATABASE_PATH=sqlite:///:memory:
EDSL_API_TIMEOUT=60
EDSL_BACKOFF_START_SEC=1
EDSL_BACKOFF_MAX_SEC=60
EDSL_DEFAULT_MODEL=gpt-4o
EDSL_MAX_ATTEMPTS=5
EDSL_FETCH_TOKEN_PRICES=False
EDSL_SERVICE_RPM_BASELINE=250
EDSL_SERVICE_TPM_BASELINE=10000000000
EXPECTED_PARROT_URL=http://localhost:8000
EDSL_MAX_CONCURRENT_TASKS=1000
EDSL_OPEN_EXCEPTION_REPORT_URL=False

# -- required by EDSL
filterwarnings =
ignore::DeprecationWarning

# Notes:
# - Look at conftest.py for the `set_env_vars` fixture.
# This fixture sets env vars for the duration of one test.
# - To run integration tests, you have to have also have a .env with your LLM provider keys stored locally.

0 comments on commit 356cf8b

Please sign in to comment.