-
Notifications
You must be signed in to change notification settings - Fork 44.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor(agent, forge): Move relevant tests from autogpt
to forge
#7247
Conversation
✅ Deploy Preview for auto-gpt-docs canceled.
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #7247 +/- ##
===========================================
+ Coverage 25.53% 54.21% +28.67%
===========================================
Files 80 122 +42
Lines 4664 6875 +2211
Branches 631 881 +250
===========================================
+ Hits 1191 3727 +2536
+ Misses 3406 3015 -391
- Partials 67 133 +66
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly. |
Server CI build for Ubuntu fails for some reason :( |
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
That was me sorry! Updating should fix |
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly. |
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly. |
Background
After refactor #7106, due to coupling tests that should had been moved to
forge
stayed inautogpt
.Changes 🏗️
autogpt/tests/vcr_cassettes
submodule toforge/tests/vcr_cassettes
pyproject.toml
:"requires_openai_api_key", "requires_huggingface_api_key"
Moved relevant tests from
autogpt/tests
to appropiate directories:autogpt/tests/unit/test_web_search.py
→forge/components/web/test_search.py
autogpt/tests/unit/test_git_commands.py
→forge/components/git_operations/test_git_operations.py
autogpt/tests/unit/test_file_operations.py
→forge/components/file_manager/test_file_manager.py
autogpt/tests/integration/test_image_gen.py
→forge/components/image_gen/test_image_gen.py
autogpt/tests/integration/test_web_selenium.py
→forge/components/web/test_selenium.py
autogpt/tests/integration/test_execute_code.py
→forge/components/code_executor/test_code_executor.py
autogpt/tests/unit/test_s3_file_storage.py
→forge/file_storage/test_s3_file_storage.py
autogpt/tests/unit/test_gcs_file_storage.py
→forge/file_storage/test_gcs_file_storage.py
autogpt/tests/unit/test_local_file_storage.py
→forge/file_storage/test_local_file_storage.py
autogpt/tests/unit/test_json.py
→forge/json/test_parsing.py
autogpt/tests/unit/test_logs.py
→forge/logging/test_utils.py
autogpt/tests/unit/test_url_validation.py
→forge/utils/test_url_validator.py
autogpt/tests/unit/test_text_file_parsers.py
→forge/utils/test_file_operations.py
Also:
forge/components/web/test_search.py
PR Quality Scorecard ✨
+2 pts
+5 pts
+5 pts
+5 pts
-4 pts
+4 pts
+5 pts
-5 pts
agbenchmark
to verify that these changes do not regress performance?+10 pts