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

Add unit tests #38

Closed
jerryjliu opened this issue Nov 23, 2022 · 1 comment
Closed

Add unit tests #38

jerryjliu opened this issue Nov 23, 2022 · 1 comment
Milestone

Comments

@jerryjliu
Copy link
Collaborator

We need a proper suite of unit tests with tests

@jerryjliu
Copy link
Collaborator Author

Sufficiently closed in #43, #44, #45

@jerryjliu jerryjliu modified the milestones: 43, 44 Nov 28, 2022
viveksilimkhan1 pushed a commit to viveksilimkhan1/llama_index that referenced this issue Oct 30, 2023
* Update build matrix and job run conditions.

* Limit concurrency.

* Fix build matrix Python version error. Add missing external markers to REST tests.

* Test on installed package.

* Fix Python version typo.

* Test commit.

* Test commit.

* Add env block.

* Install dev requirements for tests.

* Remove dependency check from query functions.

* Use python -m in install from sdist step.

* Test uninstalling dotenv before reinstalling it.

* Add import guard for minichain and langchain.

* Add Pytest skip markers for LangChain and MiniChain tests.

* Add Pytest skip markers for NER MiniChain tests.

* Change backend to REST from minichain.

* Set max-parallel to 2.

* Add Literal to compat.py.

* Remove max-parallel.

* Add version guards for minichain and langchain in requirements-dev.txt.

* Fix env markers in requirements-dev.txt.

* Remove inheritance in NoopTask_Incorrect.

* Disable fail-fast for test purposes.

* Upgrade pip.

* Narrow down Python range for MiniChain.

* Add pytest config in pyproject.toml.

* Add external marker to pytest config.

* Drop mypy validation step due to fix of pre-commit config in other PR.

* Update spacy_llm/tests/backends/test_minichain.py

Co-authored-by: Adriane Boyd <adrianeboyd@gmail.com>

* Update spacy_llm/tests/backends/test_langchain.py

Co-authored-by: Adriane Boyd <adrianeboyd@gmail.com>

* Use :: instead of : for filterwarnings config.

* Reverse package/warning order in filterwarnings config.

* Use : instead of :: for filterwarnings config.

* Use deprecation warning marker in tests instead of in pytest config.

* Reverse filterwarnings order.

* Use : instead of :: for filterwarnings.

* Move filterwarning to pyproject.toml.

* Remove upper Python version bounds for langchain and minichain dependencies. Bumped minimal version for minichain to 3.8.

* Limit upper Python version bound for minichain.

* Move filterwarnings to new pytest config in setup.cfg.

* Update requirements-dev.txt

Co-authored-by: Adriane Boyd <adrianeboyd@gmail.com>

* Attempt to fix file path for Jinja NER tests.

* Add MANIFEST.in.

* Add .json and .jsonl files.

* Add test output.

* Add test output.

* Fix paths in test_ner.py.

* Fix EXAMPLES_DIR.

* Add pytest mark for filterwarnings in compat.py.

* Guard pytest import.

* Move pytest settings to pyproject.toml.

* Add testpaths.

* Remove debugging step. Limit build matrix temporarily.

* Remove regex in filter warning.

* Add workaround for depreciation warning originating from minichain.

* Add workaround for depreciation warning originating from minichain.

* Add ignore in pytest call.

* Remove .catch_warnings().

* Move pytest config entirely into pyproject.toml.

* Ignore all DepreciationWarnings.

* Fix paths in text_textcat.py.

* Fix paths in text_textcat.py.

* Fix paths in text_textcat.py.

* Restore all platforms.

---------

Co-authored-by: Adriane Boyd <adrianeboyd@gmail.com>
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

No branches or pull requests

1 participant