Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
While working on #397 I was updating some tests and when I ran
pytest
some tests were failing. I found that our CI only runspytest -m unit
which is a small subset of our total tests. I opened #400 to handle adding the correct decorators to all our tests, but addressed the failing tests here.Changes:
errors_test.py
which were failing since feat: retryable implementation #363 was mergedworkflows/multi_agents.py
toworkflows/test_multi_agents.py
sincepytest
only looks for files namedtest_*.py
or*_test.py
, once run these tests were failing due to an invalid ollama model.llama3.1
instead ofllama3.1:8b
. I havellama3.1:latest
installed locally and notllama3.1:8b
so this e2e test was erroring. By updating the default to remove the tag it addresses the issue and matches will all other use cases in the current code.Checklist