mirror of
https://github.com/hwchase17/langchain
synced 2024-10-29 17:07:25 +00:00
c7ca350cd3
In LangChain, all module classes are enumerated in the `__init__.py` file of the correspondent module. But some classes were missed and were not included in the module `__init__.py` This PR: - added the missed classes to the module `__init__.py` files - `__init__.py:__all_` variable value (a list of the class names) was sorted - `langchain.tools.sql_database.tool.QueryCheckerTool` was renamed into the `QuerySQLCheckerTool` because it conflicted with `langchain.tools.spark_sql.tool.QueryCheckerTool` - changes to `pyproject.toml`: - added `pgvector` to `pyproject.toml:extended_testing` - added `pandas` to `pyproject.toml:[tool.poetry.group.test.dependencies]` - commented out the `streamlit` from `collbacks/__init__.py`, It is because now the `streamlit` requires Python >=3.7, !=3.9.7 - fixed duplicate names in `tools` - fixed correspondent ut-s #### Who can review? @hwchase17 @dev2049
92 lines
2.9 KiB
Python
92 lines
2.9 KiB
Python
"""A unit test meant to catch accidental introduction of non-optional dependencies."""
|
|
from pathlib import Path
|
|
from typing import Any, Dict, Mapping
|
|
|
|
import pytest
|
|
import toml
|
|
|
|
HERE = Path(__file__).parent
|
|
|
|
PYPROJECT_TOML = HERE / "../../pyproject.toml"
|
|
|
|
|
|
@pytest.fixture()
|
|
def poetry_conf() -> Dict[str, Any]:
|
|
"""Load the pyproject.toml file."""
|
|
with open(PYPROJECT_TOML) as f:
|
|
return toml.load(f)["tool"]["poetry"]
|
|
|
|
|
|
def test_required_dependencies(poetry_conf: Mapping[str, Any]) -> None:
|
|
"""A test that checks if a new non-optional dependency is being introduced.
|
|
|
|
If this test is triggered, it means that a contributor is trying to introduce a new
|
|
required dependency. This should be avoided in most situations.
|
|
"""
|
|
# Get the dependencies from the [tool.poetry.dependencies] section
|
|
dependencies = poetry_conf["dependencies"]
|
|
|
|
required_dependencies = [
|
|
package_name
|
|
for package_name, requirements in dependencies.items()
|
|
if isinstance(requirements, str) or not requirements.get("optional", False)
|
|
]
|
|
|
|
assert sorted(required_dependencies) == [
|
|
"PyYAML",
|
|
"SQLAlchemy",
|
|
"aiohttp",
|
|
"async-timeout",
|
|
"dataclasses-json",
|
|
"langchainplus-sdk",
|
|
"numexpr",
|
|
"numpy",
|
|
"openapi-schema-pydantic",
|
|
"pydantic",
|
|
"python",
|
|
"requests",
|
|
"tenacity",
|
|
]
|
|
|
|
|
|
def test_test_group_dependencies(poetry_conf: Mapping[str, Any]) -> None:
|
|
"""Check if someone is attempting to add additional test dependencies.
|
|
|
|
Only dependencies associated with test running infrastructure should be added
|
|
to the test group; e.g., pytest, pytest-cov etc.
|
|
|
|
Examples of dependencies that should NOT be included: boto3, azure, postgres, etc.
|
|
"""
|
|
|
|
test_group_deps = sorted(poetry_conf["group"]["test"]["dependencies"])
|
|
|
|
assert test_group_deps == [
|
|
"duckdb-engine",
|
|
"freezegun",
|
|
"lark",
|
|
"pandas",
|
|
"pytest",
|
|
"pytest-asyncio",
|
|
"pytest-cov",
|
|
"pytest-dotenv",
|
|
"pytest-mock",
|
|
"pytest-socket",
|
|
"pytest-watcher",
|
|
"responses",
|
|
"syrupy",
|
|
]
|
|
|
|
|
|
def test_imports() -> None:
|
|
"""Test that you can import all top level things okay."""
|
|
from langchain.chains import LLMChain # noqa: F401
|
|
from langchain.chat_models import ChatOpenAI # noqa: F401
|
|
from langchain.document_loaders import BSHTMLLoader # noqa: F401
|
|
from langchain.embeddings import OpenAIEmbeddings # noqa: F401
|
|
from langchain.llms import OpenAI # noqa: F401
|
|
from langchain.prompts import BasePromptTemplate # noqa: F401
|
|
from langchain.retrievers import VespaRetriever # noqa: F401
|
|
from langchain.tools import DuckDuckGoSearchResults # noqa: F401
|
|
from langchain.utilities import SerpAPIWrapper # noqa: F401
|
|
from langchain.vectorstores import FAISS # noqa: F401
|