Skip to content

Commit 878949b

Browse files
fix: resolve import path issues in moved test files
- Fix remaining src.contextforge_memory imports in test files - Add contextforge_memory import to fix module reload issues - Update all import paths to use contextforge_memory instead of src.contextforge_memory - Resolves CI failures with KeyError: 'contextforge_memory' and ModuleNotFoundError
1 parent 5815d2b commit 878949b

File tree

3 files changed

+13
-12
lines changed

3 files changed

+13
-12
lines changed

tests/test_simple_imports.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,10 @@ class TestSimpleImports:
1414

1515
def test_main_module_import(self):
1616
"""Test that the main module can be imported."""
17-
import src.contextforge_memory
17+
import contextforge_memory
1818

19-
assert hasattr(src.contextforge_memory, "__version__")
20-
assert hasattr(src.contextforge_memory, "InMemoryCosineIndex")
19+
assert hasattr(contextforge_memory, "__version__")
20+
assert hasattr(contextforge_memory, "InMemoryCosineIndex")
2121

2222
def test_embeddings_module_imports(self):
2323
"""Test that embeddings modules can be imported."""
@@ -53,7 +53,7 @@ def test_vector_index_module_imports(self):
5353

5454
def test_import_with_reload(self):
5555
"""Test that modules can be reloaded without issues."""
56-
import src.contextforge_memory.embeddings.base as base_module
56+
import contextforge_memory.embeddings.base as base_module
5757

5858
# Reload the module
5959
importlib.reload(base_module)

tests/test_strict_provider_detailed.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414

1515
import pytest
1616

17+
import contextforge_memory # noqa: F401 # Required for module reloading
1718
from contextforge_memory.embeddings.openai import OpenAIEmbeddingsProvider
1819
from contextforge_memory.embeddings.sentencetransformers import (
1920
SentenceTransformersProvider,

tests/test_threadpool_config.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -18,14 +18,14 @@ class TestThreadPoolConfiguration:
1818

1919
def test_default_pool_size(self):
2020
"""Test that default pool size is 4 when no env var is set."""
21-
from src.contextforge_memory.main import ThreadPoolConfig
21+
from contextforge_memory.main import ThreadPoolConfig
2222

2323
config = ThreadPoolConfig()
2424
assert config.pool_size == 4
2525

2626
def test_explicit_pool_size(self):
2727
"""Test that explicit pool sizes are accepted."""
28-
from src.contextforge_memory.main import ThreadPoolConfig
28+
from contextforge_memory.main import ThreadPoolConfig
2929

3030
test_cases = [1, 4, 8, 16, 32]
3131

@@ -35,7 +35,7 @@ def test_explicit_pool_size(self):
3535

3636
def test_from_env_with_valid_values(self):
3737
"""Test that from_env() works with valid environment values."""
38-
from src.contextforge_memory.main import ThreadPoolConfig
38+
from contextforge_memory.main import ThreadPoolConfig
3939

4040
test_cases = [1, 4, 8, 16, 32]
4141

@@ -46,7 +46,7 @@ def test_from_env_with_valid_values(self):
4646

4747
def test_from_env_with_invalid_values(self):
4848
"""Test that from_env() handles invalid environment values gracefully."""
49-
from src.contextforge_memory.main import ThreadPoolConfig
49+
from contextforge_memory.main import ThreadPoolConfig
5050

5151
invalid_values = ["invalid", "-1", "0", "not_a_number"]
5252

@@ -58,15 +58,15 @@ def test_from_env_with_invalid_values(self):
5858

5959
def test_from_env_with_missing_var(self):
6060
"""Test that from_env() uses default when env var is missing."""
61-
from src.contextforge_memory.main import ThreadPoolConfig
61+
from contextforge_memory.main import ThreadPoolConfig
6262

6363
with patch.dict(os.environ, {}, clear=True):
6464
config = ThreadPoolConfig.from_env()
6565
assert config.pool_size == 4
6666

6767
def test_pool_creation(self):
6868
"""Test that create_pool() returns a working ThreadPoolExecutor."""
69-
from src.contextforge_memory.main import ThreadPoolConfig
69+
from contextforge_memory.main import ThreadPoolConfig
7070

7171
config = ThreadPoolConfig(pool_size=2)
7272
pool = config.create_pool()
@@ -86,7 +86,7 @@ def simple_task(x):
8686

8787
def test_pool_with_concurrent_tasks(self):
8888
"""Test that the pool can handle concurrent tasks efficiently."""
89-
from src.contextforge_memory.main import ThreadPoolConfig
89+
from contextforge_memory.main import ThreadPoolConfig
9090

9191
config = ThreadPoolConfig(pool_size=3)
9292
pool = config.create_pool()
@@ -150,7 +150,7 @@ def test_pool_with_real_app_integration(self):
150150

151151
def test_environment_parsing_edge_cases(self):
152152
"""Test edge cases in environment variable parsing."""
153-
from src.contextforge_memory.main import ThreadPoolConfig
153+
from contextforge_memory.main import ThreadPoolConfig
154154

155155
# Test with very large numbers
156156
with patch.dict(os.environ, {"CONTEXTFORGE_IO_POOL_SIZE": "1000"}):

0 commit comments

Comments
 (0)