Remove test-cov

Author: tylerhutcherson

.github/workflows/run_tests.yml

@@ -75,12 +75,12 @@ jobs:
         AWS_ACCESS_KEY_ID: ${{ secrets.AWS_ACCESS_KEY_ID }}
         AWS_SECRET_ACCESS_KEY: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
       run: |
-        poetry run test-cov
+        poetry run test-verbose
 
     - name: Run tests
       if: matrix.connection != 'plain' || matrix.redis-stack-version != 'latest'
       run: |
-        SKIP_VECTORIZERS=True SKIP_RERANKERS=True poetry run test-cov
+        SKIP_VECTORIZERS=True SKIP_RERANKERS=True poetry run test-verbose
 
     - name: Run notebooks
       if: matrix.connection == 'plain' && matrix.redis-stack-version == 'latest'

CONTRIBUTING.md

@@ -78,17 +78,17 @@ To run Testcontainers-based tests you need a local Docker installation such as:
 
 Tests w/ vectorizers:
 ```bash
-poetry run test-cov
+poetry run test-verbose
 ```
 
 Tests w/out vectorizers:
 ```bash
-SKIP_VECTORIZERS=true poetry run test-cov
+SKIP_VECTORIZERS=true poetry run test-verbose
 ```
 
 Tests w/out rerankers:
 ```bash
-SKIP_RERANKERS=true poetry run test-cov
+SKIP_RERANKERS=true poetry run test-verbose
 ```
 
 ### Documentation

Makefile

@@ -19,10 +19,10 @@ check-types:
 lint: format check-types
 
 test:
-	SKIP_RERANKERS=true SKIP_VECTORIZERS=true poetry run test-cov
+	SKIP_RERANKERS=true SKIP_VECTORIZERS=true poetry run test-verbose
 
 test-all:
-	poetry run test-cov
+	poetry run test-verbose
 
 check: lint test
 

poetry.lock

@@ -51,7 +51,6 @@ black = ">=20.8b1"
 isort = ">=5.6.4"
 pylint = "3.1.0"
 pytest = "8.1.1"
-pytest-cov = "5.0.0"
 pytest-asyncio = "0.23.6"
 mypy = "1.9.0"
 types-redis = "*"
@@ -82,7 +81,6 @@ check-lint = "scripts:check_lint"
 check-mypy = "scripts:check_mypy"
 test = "scripts:test"
 test-verbose = "scripts:test_verbose"
-test-cov = "scripts:test_cov"
 cov = "scripts:cov"
 test-notebooks = "scripts:test_notebooks"
 build-docs = "scripts:build_docs"

pyproject.toml

@@ -235,6 +235,7 @@ def as_redis_field(self) -> RedisField:
 
 class FlatVectorField(BaseField):
     "Vector field with a FLAT index (brute force nearest neighbors search)"
+
     type: str = Field(default="vector", const=True)
     attrs: FlatVectorFieldAttributes
 

redisvl/schema/fields.py

@@ -1,8 +1,8 @@
 import asyncio
 import os
+import warnings
 from collections import namedtuple
 from time import sleep, time
-import warnings
 
 import pytest
 from pydantic.v1 import ValidationError
@@ -918,11 +918,19 @@ def test_invalid_vectorizer(redis_url):
 def test_passes_through_dtype_to_default_vectorizer(redis_url):
     # The default is float32, so we should see float64 if we pass it in.
     cache = SemanticCache(
-        name="test_pass_through_dtype", dtype="float64", redis_url=redis_url, overwrite=True
+        name="test_pass_through_dtype",
+        dtype="float64",
+        redis_url=redis_url,
+        overwrite=True,
     )
     assert cache._vectorizer.dtype == "float64"
 
 
 def test_deprecated_dtype_argument(redis_url):
     with pytest.warns(DeprecationWarning):
-        SemanticCache(name="test_deprecated_dtype", dtype="float32", redis_url=redis_url, overwrite=True)
+        SemanticCache(
+            name="test_deprecated_dtype",
+            dtype="float32",
+            redis_url=redis_url,
+            overwrite=True,
+        )

tests/integration/test_llmcache.py

@@ -613,11 +613,16 @@ def test_invalid_vectorizer(redis_url):
 def test_passes_through_dtype_to_default_vectorizer(redis_url):
     # The default is float32, so we should see float64 if we pass it in.
     cache = SemanticSessionManager(
-        name="test_pass_through_dtype", dtype="float64", redis_url=redis_url, overwrite=True
+        name="test_pass_through_dtype",
+        dtype="float64",
+        redis_url=redis_url,
+        overwrite=True,
     )
     assert cache._vectorizer.dtype == "float64"
 
 
 def test_deprecated_dtype_argument(redis_url):
     with pytest.warns(DeprecationWarning):
-        SemanticSessionManager(name="float64 session", dtype="float64", redis_url=redis_url, overwrite=True)
+        SemanticSessionManager(
+            name="float64 session", dtype="float64", redis_url=redis_url, overwrite=True
+        )