infra: remove use of --vcr-record=none (#31452)

This option is specific to `pytest-vcr`. `pytest-recording` runs in this
mode by default.
This commit is contained in:
ccurme 2025-06-01 10:49:59 -04:00 committed by GitHub
parent 3db1aa0ba6
commit d3be4a0c56
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
4 changed files with 2 additions and 15 deletions

View File

@ -14,7 +14,7 @@ test tests:
uv run --group test pytest -vvv --disable-socket --allow-unix-socket $(TEST_FILE) uv run --group test pytest -vvv --disable-socket --allow-unix-socket $(TEST_FILE)
integration_test integration_tests: integration_test integration_tests:
uv run --group test --group test_integration pytest -vvv --timeout 30 $(TEST_FILE) --vcr-record=none uv run --group test --group test_integration pytest -vvv --timeout 30 $(TEST_FILE)
test_watch: test_watch:
uv run --group test ptw --snapshot-update --now . -- -vv $(TEST_FILE) uv run --group test ptw --snapshot-update --now . -- -vv $(TEST_FILE)

View File

@ -24,7 +24,7 @@ test tests:
TIKTOKEN_CACHE_DIR=tiktoken_cache uv run --group test pytest --disable-socket --allow-unix-socket $(TEST_FILE) TIKTOKEN_CACHE_DIR=tiktoken_cache uv run --group test pytest --disable-socket --allow-unix-socket $(TEST_FILE)
integration_test integration_tests: integration_test integration_tests:
uv run --group test --group test_integration pytest -n auto $(TEST_FILE) --vcr-record=none uv run --group test --group test_integration pytest -n auto $(TEST_FILE)
test_watch: test_watch:
uv run --group test ptw --snapshot-update --now . -- -vv $(TEST_FILE) uv run --group test ptw --snapshot-update --now . -- -vv $(TEST_FILE)

View File

@ -661,12 +661,6 @@ class ChatModelIntegrationTests(ChatModelTests):
You can then commit the cassette to your repository. Subsequent test runs You can then commit the cassette to your repository. Subsequent test runs
will use the cassette instead of making HTTP calls. will use the cassette instead of making HTTP calls.
.. tip::
Adding ``--vcr-record=none`` to the pytest command will ensure that
no new cassettes are recorded, and only existing cassettes are used.
Consider adding this to your CI configuration (e.g., modify relevant
Makefile commands).
""" # noqa: E501 """ # noqa: E501
@property @property

View File

@ -763,13 +763,6 @@ class ChatModelUnitTests(ChatModelTests):
You can then commit the cassette to your repository. Subsequent test runs You can then commit the cassette to your repository. Subsequent test runs
will use the cassette instead of making HTTP calls. will use the cassette instead of making HTTP calls.
.. tip::
Adding ``--vcr-record=none`` to the pytest command will ensure that
no new cassettes are recorded, and only existing cassettes are used.
Consider adding this to your CI configuration (e.g., modify relevant
Makefile commands).
Testing initialization from environment variables Testing initialization from environment variables
Some unit tests may require testing initialization from environment variables. Some unit tests may require testing initialization from environment variables.
These tests can be enabled by overriding the ``init_from_env_params`` These tests can be enabled by overriding the ``init_from_env_params``