Skip to content
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.

Sign up for GitHub

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

Jump to bottom

added fixture and skipifs to the community tests #546

Merged
merged 1 commit into from
Sep 26, 2024
Merged

added fixture and skipifs to the community tests #546

merged 1 commit into from
Sep 26, 2024

Conversation

3rd-Son
Copy link
Contributor

@3rd-Son 3rd-Son commented Sep 26, 2024

I added fixture and skipifs to the necessary test cases in swarmauri_community
I also updated the setup.py in the swarmauri_community

currently, below is the log summary of the test

===================================================================================================== short test summary info =====================================================================================================
FAILED tests/unit/tools/EntityRecognitionTool_test.py::test_call - RuntimeError: Failed to import transformers.models.bert.modeling_tf_bert because of the following error (look up to see its traceback):
FAILED tests/unit/tools/PsutilTool_test.py::test_call[network-expected_keys3-False] - psutil.AccessDenied: (pid=11947)
FAILED tests/unit/tools/PsutilTool_test.py::test_call[sensors-expected_keys4-False] - AttributeError: module 'psutil' has no attribute 'sensors_temperatures'
FAILED tests/unit/tools/SentimentAnalysisTool_test.py::test_call[I love this product!-expected_labels0] - RuntimeError: Sentiment analysis failed: Failed to import transformers.models.distilbert.modeling_tf_distilbert because of the following error (look up to see its traceback):
FAILED tests/unit/tools/SentimentAnalysisTool_test.py::test_call[I hate this product!-expected_labels1] - RuntimeError: Sentiment analysis failed: Failed to import transformers.models.distilbert.modeling_tf_distilbert because of the following error (look up to see its traceback):
FAILED tests/unit/tools/SentimentAnalysisTool_test.py::test_call[This product is okay.-expected_labels2] - RuntimeError: Sentiment analysis failed: Failed to import transformers.models.distilbert.modeling_tf_distilbert because of the following error (look up to see its traceback):
ERROR tests/unit/vector_stores/Neo4jVectorStore_test.py::test_ubc_type - neo4j.exceptions.ServiceUnavailable: Couldn't connect to localhost:7687 (resolved to ()):
ERROR tests/unit/vector_stores/Neo4jVectorStore_test.py::test_add_document - neo4j.exceptions.ServiceUnavailable: Couldn't connect to localhost:7687 (resolved to ()):
ERROR tests/unit/vector_stores/Neo4jVectorStore_test.py::test_add_documents - neo4j.exceptions.ServiceUnavailable: Couldn't connect to localhost:7687 (resolved to ()):
================================================================================ 6 failed, 113 passed, 145 skipped, 6 warnings, 3 errors in 40.46s ================================================================================

What do you have to say about the errors here

@3rd-Son 3rd-Son requested a review from cobycloud as a code owner September 26, 2024 11:23
MichaelDecent
MichaelDecent reviewed Sep 26, 2024
View reviewed changes
pkgs/community/tests/unit/vector_stores/Neo4jVectorStore_test.py
Copy link
Collaborator

@MichaelDecent MichaelDecent Sep 26, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we remove the defaults values when fetching the env variables, create the actual env variables, and try using them, to see if we will reproduce same error. NEO4J_PASSWORD and
NEO4J_URI

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i will DM mine to test with

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay
Waiting for the DM

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sent already

@cobycloud cobycloud merged commit fc9a7df into swarmauri:0.4.5.dev1 Sep 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MichaelDecent MichaelDecent MichaelDecent left review comments

@cobycloud cobycloud cobycloud approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@3rd-Son @cobycloud @MichaelDecent