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.

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

Integration tests failing #2426

Closed
letmerecall opened this issue Apr 5, 2023 · 1 comment · Fixed by #2560
Closed

Integration tests failing #2426

letmerecall opened this issue Apr 5, 2023 · 1 comment · Fixed by #2560

Comments

@letmerecall
Copy link
Contributor

Describe the issue

make integration_tests failing with the following error.

Screenshot 2023-04-05 at 1 57 28 PM

How to reproduce it?

  1. Install poetry requirements poetry install -E all
  2. Run integration tests make integration_tests

Related issues/PRs

  • It might be a result of rm pandas dependency #2102 where we removed pandas from dependency, but we still import it in tests/integration_tests/document_loaders/test_dataframe.py
@sergerdn
Copy link
Contributor

sergerdn commented Apr 5, 2023

Please note that it is not possible to run many integration tests since they require the execution of certain processes, such as Redis, Elastic Search, and so on. Currently, we do not have any automation in place to initiate all the required processes before running the tests.

As a result, we do not have the necessary feature to run functional tests in the GitHub flow. We are currently only running unit tests.

I have implemented some new feature in #2402 that have dependencies for integration tests:

Include an optional group test_integration in the pyproject.toml file. This group should contain dependencies for integration tests and can be installed using the command poetry install --with test_integration.
Any new dependencies should be added by running poetry add some_new_deps --group "test_integration"

[tool.poetry.group.test_integration]
optional = true

[tool.poetry.group.test_integration.dependencies]
openai = "^0.27.4"
elasticsearch = {extras = ["async"], version = "^8.6.2"}
## add here anything you need to make integration/functional tests running

If my PR will merged, feel free to add any dependencies necessary to run your tests.

hwchase17 pushed a commit that referenced this issue Apr 8, 2023
Almost all integration tests have failed, but we haven't encountered any
import errors yet. Some tests failed due to lazy import issues. It
doesn't seem like a problem to resolve some of these errors in the next
PR.
I have a headache from resolving conflicts with `deeplake` and `boto3`,
so I will temporarily comment out `boto3`.


fix #2426
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants