c889a57b6b
Test Suites / Build CI Environment (push) Has been cancelled
Test Suites / Basic Tests (push) Has been cancelled
Test Suites / End-to-End Tests (push) Has been cancelled
Test Suites / CLI Tests (push) Has been cancelled
Test Suites / Slow End-to-End Tests (push) Has been cancelled
Test Suites / Graph Database Tests (push) Has been cancelled
Test Suites / Vector DB Tests (push) Has been cancelled
Test Suites / Temporal Graph Test (push) Has been cancelled
Test Suites / Search Test on Different DBs (push) Has been cancelled
Test Suites / Example Tests (push) Has been cancelled
Test Suites / Notebook Tests (push) Has been cancelled
Test Suites / OS and Python Tests Ubuntu (push) Has been cancelled
Test Suites / OS and Python Tests Extended (push) Has been cancelled
Test Suites / LLM Test Suite (push) Has been cancelled
Test Suites / S3 File Storage Test (push) Has been cancelled
Test Suites / Run Integration Tests (push) Has been cancelled
Test Suites / MCP Tests (push) Has been cancelled
Test Suites / Docker Compose Test (push) Has been cancelled
Test Suites / Docker CI test (push) Has been cancelled
Test Suites / Relational DB Migration Tests (push) Has been cancelled
Test Suites / Distributed Cognee Test (push) Has been cancelled
Test Suites / DB Examples Tests (push) Has been cancelled
Test Suites / Test Completion Status (push) Has been cancelled
Test Suites / Claude Code Review (push) Has been cancelled
Test Suites / basic checks (push) Has been cancelled
build | Build and Push Cognee MCP Docker Image to dockerhub / docker-build-and-push (push) Has been cancelled
Scorecard supply-chain security / Scorecard analysis (push) Has been cancelled
build | Build and Push Docker Image to dockerhub / docker-build-and-push (push) Has been cancelled
Weighted Edges Tests / Test Weighted Edges Core Functionality (3.11) (push) Has been cancelled
Weighted Edges Tests / Test Weighted Edges Core Functionality (3.12) (push) Has been cancelled
Weighted Edges Tests / Test Weighted Edges with Different Graph Databases (kuzu, kuzu) (push) Has been cancelled
Weighted Edges Tests / Test Weighted Edges with Different Graph Databases (neo4j, neo4j) (push) Has been cancelled
Weighted Edges Tests / Test Weighted Edges Examples (push) Has been cancelled
Weighted Edges Tests / Code Quality for Weighted Edges (push) Has been cancelled
652 lines
22 KiB
Python
652 lines
22 KiB
Python
"""
|
|
Unit Tests: Graph Methods
|
|
|
|
Tests for core graph deletion methods with specific inputs and outputs.
|
|
|
|
Test Coverage:
|
|
- test_get_data_related_nodes_excludes_shared: Verify shared nodes are excluded
|
|
- test_delete_data_nodes_and_edges_removes_from_all_systems: Verify complete cleanup
|
|
"""
|
|
|
|
import os
|
|
import pathlib
|
|
import pytest
|
|
import pytest_asyncio
|
|
from uuid import UUID, uuid4, uuid5, NAMESPACE_OID
|
|
|
|
import cognee
|
|
from cognee.context_global_variables import set_database_global_context_variables
|
|
from cognee.infrastructure.databases.graph import get_graph_engine
|
|
from cognee.infrastructure.databases.relational import get_relational_engine
|
|
from cognee.infrastructure.databases.relational.create_relational_engine import (
|
|
create_relational_engine,
|
|
)
|
|
from cognee.modules.data.methods import create_dataset, create_authorized_dataset
|
|
from cognee.modules.engine.operations.setup import setup
|
|
from cognee.modules.graph.methods import (
|
|
delete_data_nodes_and_edges,
|
|
get_data_related_nodes,
|
|
get_global_data_related_nodes,
|
|
get_orphaned_nodeset_labels_for_dataset,
|
|
get_shared_slugs_losing_dataset_anchor,
|
|
)
|
|
from cognee.modules.graph.models import Node, Edge
|
|
from cognee.modules.users.methods import get_default_user
|
|
from cognee.shared.logging_utils import get_logger
|
|
from sqlalchemy import select
|
|
|
|
logger = get_logger()
|
|
|
|
|
|
@pytest_asyncio.fixture(autouse=True)
|
|
async def _dispose_relational_engine_after_test():
|
|
yield
|
|
|
|
try:
|
|
db_engine = get_relational_engine()
|
|
engine = getattr(db_engine, "engine", None)
|
|
if engine is not None:
|
|
await engine.dispose(close=True)
|
|
except Exception:
|
|
pass
|
|
|
|
create_relational_engine.cache_clear()
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_get_data_related_nodes_excludes_shared():
|
|
"""
|
|
Test that get_data_related_nodes returns only non-shared nodes.
|
|
|
|
Setup:
|
|
- Data A has nodes: Apple (unique), Google (shared with B)
|
|
- Data B has nodes: Google (shared with A), Microsoft (unique)
|
|
|
|
Expected:
|
|
- get_data_related_nodes(dataset_id, data_A_id) returns [Apple] only
|
|
- get_data_related_nodes(dataset_id, data_B_id) returns [Microsoft] only
|
|
- Google is excluded from both (shared node)
|
|
"""
|
|
data_directory_path = os.path.join(
|
|
pathlib.Path(__file__).parent.parent.parent.parent,
|
|
".data_storage/test_get_data_related_nodes",
|
|
)
|
|
cognee.config.data_root_directory(data_directory_path)
|
|
|
|
cognee_directory_path = os.path.join(
|
|
pathlib.Path(__file__).parent.parent.parent.parent,
|
|
".cognee_system/test_get_data_related_nodes",
|
|
)
|
|
cognee.config.system_root_directory(cognee_directory_path)
|
|
|
|
await cognee.prune.prune_data()
|
|
await cognee.prune.prune_system(metadata=True)
|
|
await setup()
|
|
|
|
user = await get_default_user()
|
|
|
|
# Create dataset
|
|
dataset = await create_dataset("test_shared_nodes", user=user)
|
|
dataset_id = dataset.id
|
|
|
|
await set_database_global_context_variables(dataset_id, user.id)
|
|
|
|
# Create unique data IDs
|
|
data_a_id = uuid4()
|
|
data_b_id = uuid4()
|
|
|
|
# Create node slugs (UUIDs)
|
|
apple_slug = uuid5(NAMESPACE_OID, "Apple")
|
|
google_slug = uuid5(NAMESPACE_OID, "Google") # Shared
|
|
microsoft_slug = uuid5(NAMESPACE_OID, "Microsoft")
|
|
|
|
# Create nodes for data A: Apple (unique), Google (shared)
|
|
node_apple = Node(
|
|
id=uuid4(),
|
|
slug=apple_slug,
|
|
user_id=user.id,
|
|
data_id=data_a_id,
|
|
dataset_id=dataset_id,
|
|
type="Entity",
|
|
indexed_fields=["name"],
|
|
)
|
|
node_google_a = Node(
|
|
id=uuid4(),
|
|
slug=google_slug, # Same slug as in data B
|
|
user_id=user.id,
|
|
data_id=data_a_id,
|
|
dataset_id=dataset_id,
|
|
type="Entity",
|
|
indexed_fields=["name"],
|
|
)
|
|
|
|
# Create nodes for data B: Google (shared), Microsoft (unique)
|
|
node_google_b = Node(
|
|
id=uuid4(),
|
|
slug=google_slug, # Same slug as in data A
|
|
user_id=user.id,
|
|
data_id=data_b_id,
|
|
dataset_id=dataset_id,
|
|
type="Entity",
|
|
indexed_fields=["name"],
|
|
)
|
|
node_microsoft = Node(
|
|
id=uuid4(),
|
|
slug=microsoft_slug,
|
|
user_id=user.id,
|
|
data_id=data_b_id,
|
|
dataset_id=dataset_id,
|
|
type="Entity",
|
|
indexed_fields=["name"],
|
|
)
|
|
|
|
# Insert nodes into database
|
|
db_engine = get_relational_engine()
|
|
async with db_engine.get_async_session() as session:
|
|
session.add(node_apple)
|
|
session.add(node_google_a)
|
|
session.add(node_google_b)
|
|
session.add(node_microsoft)
|
|
await session.commit()
|
|
|
|
logger.info("Inserted 4 nodes: Apple (A), Google (A+B shared), Microsoft (B)")
|
|
|
|
# Test get_data_related_nodes for data A
|
|
nodes_a = await get_data_related_nodes(dataset_id, data_a_id)
|
|
node_a_slugs = [str(node.slug) for node in nodes_a]
|
|
|
|
logger.info(f"Data A related nodes: {len(nodes_a)} nodes")
|
|
logger.info(f"Data A node slugs: {node_a_slugs}")
|
|
|
|
# Assertions for data A
|
|
assert len(nodes_a) == 1, (
|
|
f"Data A should have 1 non-shared node (Apple), but has {len(nodes_a)}"
|
|
)
|
|
assert str(apple_slug) in node_a_slugs, "Apple should be in data A's related nodes"
|
|
assert str(google_slug) not in node_a_slugs, (
|
|
"Google should NOT be in data A's related nodes (shared with B)"
|
|
)
|
|
|
|
# Test get_data_related_nodes for data B
|
|
nodes_b = await get_data_related_nodes(dataset_id, data_b_id)
|
|
node_b_slugs = [str(node.slug) for node in nodes_b]
|
|
|
|
logger.info(f"Data B related nodes: {len(nodes_b)} nodes")
|
|
logger.info(f"Data B node slugs: {node_b_slugs}")
|
|
|
|
# Assertions for data B
|
|
assert len(nodes_b) == 1, (
|
|
f"Data B should have 1 non-shared node (Microsoft), but has {len(nodes_b)}"
|
|
)
|
|
assert str(microsoft_slug) in node_b_slugs, "Microsoft should be in data B's related nodes"
|
|
assert str(google_slug) not in node_b_slugs, (
|
|
"Google should NOT be in data B's related nodes (shared with A)"
|
|
)
|
|
|
|
logger.info("✅ test_get_data_related_nodes_excludes_shared PASSED")
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_delete_data_nodes_and_edges_removes_from_all_systems():
|
|
"""
|
|
Test that delete_data_nodes_and_edges removes data from all systems.
|
|
|
|
Setup:
|
|
- Create data with 3 nodes and 2 edges
|
|
- Insert into relational DB, graph engine, and vector index
|
|
|
|
Operation:
|
|
- Call delete_data_nodes_and_edges()
|
|
|
|
Expected:
|
|
- Relational DB: 0 records with this data_id
|
|
- Graph engine: 0 nodes with these slugs
|
|
- Vector engine: 0 items in collections for these IDs
|
|
"""
|
|
# Enable backend access control for multi-user support
|
|
os.environ["ENABLE_BACKEND_ACCESS_CONTROL"] = "True"
|
|
|
|
data_directory_path = os.path.join(
|
|
pathlib.Path(__file__).parent.parent.parent.parent,
|
|
".data_storage/test_delete_data_nodes_edges",
|
|
)
|
|
cognee.config.data_root_directory(data_directory_path)
|
|
|
|
cognee_directory_path = os.path.join(
|
|
pathlib.Path(__file__).parent.parent.parent.parent,
|
|
".cognee_system/test_delete_data_nodes_edges",
|
|
)
|
|
cognee.config.system_root_directory(cognee_directory_path)
|
|
|
|
await cognee.prune.prune_data()
|
|
await cognee.prune.prune_system(metadata=True)
|
|
await setup()
|
|
|
|
user = await get_default_user()
|
|
|
|
# Create dataset
|
|
dataset = await create_authorized_dataset("test_delete_complete", user=user)
|
|
dataset_id = dataset.id
|
|
|
|
await set_database_global_context_variables(dataset_id, user.id)
|
|
|
|
# Create data ID
|
|
data_id = uuid4()
|
|
|
|
# Create 3 node slugs (all unique to this data_id)
|
|
node_1_slug = uuid5(NAMESPACE_OID, f"Node1_{data_id}")
|
|
node_2_slug = uuid5(NAMESPACE_OID, f"Node2_{data_id}")
|
|
node_3_slug = uuid5(NAMESPACE_OID, f"Node3_{data_id}")
|
|
|
|
# Create nodes
|
|
node_1_id = uuid4()
|
|
node_2_id = uuid4()
|
|
node_3_id = uuid4()
|
|
|
|
node_1 = Node(
|
|
id=node_1_id,
|
|
slug=node_1_slug,
|
|
user_id=user.id,
|
|
data_id=data_id,
|
|
dataset_id=dataset_id,
|
|
type="Entity",
|
|
indexed_fields=["name"],
|
|
)
|
|
node_2 = Node(
|
|
id=node_2_id,
|
|
slug=node_2_slug,
|
|
user_id=user.id,
|
|
data_id=data_id,
|
|
dataset_id=dataset_id,
|
|
type="Entity",
|
|
indexed_fields=["name"],
|
|
)
|
|
node_3 = Node(
|
|
id=node_3_id,
|
|
slug=node_3_slug,
|
|
user_id=user.id,
|
|
data_id=data_id,
|
|
dataset_id=dataset_id,
|
|
type="Entity",
|
|
indexed_fields=["name"],
|
|
)
|
|
|
|
# Create 2 edges
|
|
edge_1_slug = uuid5(NAMESPACE_OID, f"Edge1_{data_id}")
|
|
edge_2_slug = uuid5(NAMESPACE_OID, f"Edge2_{data_id}")
|
|
|
|
edge_1 = Edge(
|
|
id=uuid4(),
|
|
slug=edge_1_slug,
|
|
user_id=user.id,
|
|
data_id=data_id,
|
|
dataset_id=dataset_id,
|
|
relationship_name="related_to",
|
|
source_node_id=node_1_id,
|
|
destination_node_id=node_2_id,
|
|
)
|
|
edge_2 = Edge(
|
|
id=uuid4(),
|
|
slug=edge_2_slug,
|
|
user_id=user.id,
|
|
data_id=data_id,
|
|
dataset_id=dataset_id,
|
|
relationship_name="connected_to",
|
|
source_node_id=node_2_id,
|
|
destination_node_id=node_3_id,
|
|
)
|
|
|
|
# Insert into relational database
|
|
db_engine = get_relational_engine()
|
|
async with db_engine.get_async_session() as session:
|
|
session.add(node_1)
|
|
session.add(node_2)
|
|
session.add(node_3)
|
|
session.add(edge_1)
|
|
session.add(edge_2)
|
|
await session.commit()
|
|
|
|
logger.info("Inserted 3 nodes and 2 edges into relational DB")
|
|
|
|
# Insert into graph engine
|
|
await get_graph_engine()
|
|
|
|
# Note: In real scenario, nodes would be added through normal cognify process
|
|
# For unit test, we verify the relational DB records exist
|
|
|
|
# Verify nodes exist in relational DB before deletion
|
|
async with db_engine.get_async_session() as session:
|
|
nodes_before = await session.scalars(select(Node).where(Node.data_id == data_id))
|
|
nodes_before_list = nodes_before.all()
|
|
assert len(nodes_before_list) == 3, (
|
|
f"Should have 3 nodes before deletion, found {len(nodes_before_list)}"
|
|
)
|
|
|
|
edges_before = await session.scalars(select(Edge).where(Edge.data_id == data_id))
|
|
edges_before_list = edges_before.all()
|
|
assert len(edges_before_list) == 2, (
|
|
f"Should have 2 edges before deletion, found {len(edges_before_list)}"
|
|
)
|
|
|
|
logger.info("Verified: 3 nodes and 2 edges exist before deletion")
|
|
|
|
# Execute delete_data_nodes_and_edges
|
|
logger.info(f"Deleting data nodes and edges for data_id={data_id}...")
|
|
await delete_data_nodes_and_edges(dataset_id, data_id, user.id)
|
|
|
|
# Verify deletion from relational DB
|
|
async with db_engine.get_async_session() as session:
|
|
nodes_after = await session.scalars(select(Node).where(Node.data_id == data_id))
|
|
nodes_after_list = nodes_after.all()
|
|
assert len(nodes_after_list) == 0, (
|
|
f"Should have 0 nodes after deletion, found {len(nodes_after_list)}"
|
|
)
|
|
|
|
edges_after = await session.scalars(select(Edge).where(Edge.data_id == data_id))
|
|
edges_after_list = edges_after.all()
|
|
assert len(edges_after_list) == 0, (
|
|
f"Should have 0 edges after deletion, found {len(edges_after_list)}"
|
|
)
|
|
|
|
logger.info("✅ Verified: All nodes and edges removed from relational DB")
|
|
|
|
# Note: Graph engine and vector engine deletion is tested in integration tests
|
|
# This unit test focuses on the relational DB cleanup
|
|
|
|
logger.info("✅ test_delete_data_nodes_and_edges_removes_from_all_systems PASSED")
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_get_global_data_related_nodes_scopes_by_dataset():
|
|
"""
|
|
Regression test for shared-data-across-datasets: when the same `data_id`
|
|
is linked to multiple datasets in a single global DB (non-multi-user
|
|
mode, e.g. pgvector + neo4j), deleting it from one dataset must not
|
|
hard-delete slugs still co-owned by another dataset. The un-scoped
|
|
legacy path would return every such slug as "exclusive to this
|
|
data_id" and wipe the graph/vector rows for both datasets; the
|
|
dataset-scoped path excludes co-owned slugs.
|
|
|
|
Setup:
|
|
- Single dataset row, but two `(dataset_id, data_id)` ledger pairs:
|
|
(alfa, maria) and (beta, maria) — same `data_id=maria`.
|
|
- Slug `S` is written under both pairs (shared file).
|
|
- Slug `S_alfa_only` written only under (alfa, maria).
|
|
|
|
Expected:
|
|
- `get_global_data_related_nodes(maria)` → both slugs
|
|
(legacy, un-scoped — dangerous for shared data)
|
|
- `get_global_data_related_nodes(maria, dataset_id=alfa)` → [S_alfa_only]
|
|
(S co-owned by beta, must be preserved)
|
|
- `get_global_data_related_nodes(maria, dataset_id=beta)` → []
|
|
(beta's only slug S is co-owned by alfa)
|
|
"""
|
|
data_directory_path = os.path.join(
|
|
pathlib.Path(__file__).parent.parent.parent.parent,
|
|
".data_storage/test_global_dataset_scoped",
|
|
)
|
|
cognee.config.data_root_directory(data_directory_path)
|
|
|
|
cognee_directory_path = os.path.join(
|
|
pathlib.Path(__file__).parent.parent.parent.parent,
|
|
".cognee_system/test_global_dataset_scoped",
|
|
)
|
|
cognee.config.system_root_directory(cognee_directory_path)
|
|
|
|
await cognee.prune.prune_data()
|
|
await cognee.prune.prune_system(metadata=True)
|
|
await setup()
|
|
|
|
user = await get_default_user()
|
|
|
|
dataset = await create_dataset("test_shared_data_across_datasets", user=user)
|
|
alfa_dataset_id = dataset.id
|
|
beta_dataset_id = uuid4() # synthetic second dataset for ledger-only scenario
|
|
|
|
await set_database_global_context_variables(alfa_dataset_id, user.id)
|
|
|
|
maria_data_id = uuid4()
|
|
shared_slug = uuid5(NAMESPACE_OID, "shared-slug")
|
|
alfa_only_slug = uuid5(NAMESPACE_OID, "alfa-only-slug")
|
|
|
|
shared_in_alfa = Node(
|
|
id=uuid4(),
|
|
slug=shared_slug,
|
|
user_id=user.id,
|
|
data_id=maria_data_id,
|
|
dataset_id=alfa_dataset_id,
|
|
type="Entity",
|
|
indexed_fields=["name"],
|
|
)
|
|
shared_in_beta = Node(
|
|
id=uuid4(),
|
|
slug=shared_slug,
|
|
user_id=user.id,
|
|
data_id=maria_data_id,
|
|
dataset_id=beta_dataset_id,
|
|
type="Entity",
|
|
indexed_fields=["name"],
|
|
)
|
|
alfa_only = Node(
|
|
id=uuid4(),
|
|
slug=alfa_only_slug,
|
|
user_id=user.id,
|
|
data_id=maria_data_id,
|
|
dataset_id=alfa_dataset_id,
|
|
type="Entity",
|
|
indexed_fields=["name"],
|
|
)
|
|
|
|
db_engine = get_relational_engine()
|
|
async with db_engine.get_async_session() as session:
|
|
session.add_all([shared_in_alfa, shared_in_beta, alfa_only])
|
|
await session.commit()
|
|
|
|
legacy = await get_global_data_related_nodes(maria_data_id)
|
|
legacy_slugs = {str(n.slug) for n in legacy}
|
|
assert legacy_slugs == {str(shared_slug), str(alfa_only_slug)}, (
|
|
"Un-scoped legacy query should still return both slugs (demonstrates the bug)"
|
|
)
|
|
|
|
alfa_scope = await get_global_data_related_nodes(maria_data_id, dataset_id=alfa_dataset_id)
|
|
alfa_scope_slugs = {str(n.slug) for n in alfa_scope}
|
|
assert alfa_scope_slugs == {str(alfa_only_slug)}, (
|
|
f"Scoped to alfa should only return alfa-exclusive slug, got {alfa_scope_slugs}"
|
|
)
|
|
|
|
beta_scope = await get_global_data_related_nodes(maria_data_id, dataset_id=beta_dataset_id)
|
|
beta_scope_slugs = {str(n.slug) for n in beta_scope}
|
|
assert beta_scope_slugs == set(), (
|
|
f"Scoped to beta should return nothing (shared slug co-owned by alfa), got {beta_scope_slugs}"
|
|
)
|
|
|
|
logger.info("✅ test_get_global_data_related_nodes_scopes_by_dataset PASSED")
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_get_shared_slugs_losing_dataset_anchor():
|
|
"""
|
|
Regression test for the stale `belongs_to_set` label problem.
|
|
|
|
When the same data item is linked to multiple datasets and one of those
|
|
links is removed, slugs that are co-owned by another dataset survive in
|
|
the graph but silently keep the removed dataset's label in their
|
|
`belongs_to_set` array. The orchestrator needs to know which slugs
|
|
need a targeted detag — this helper returns them.
|
|
|
|
Ledger setup:
|
|
(alfa, maria) owns slugs [shared_slug, alfa_only_slug]
|
|
(beta, maria) owns slugs [shared_slug]
|
|
(alfa, mock) owns slugs [mock_only_slug]
|
|
|
|
Deleting (alfa, maria):
|
|
- `shared_slug` → loses alfa's anchor (no other (alfa, _) row) AND
|
|
has another owner (beta, maria) → MUST detag "alfa"
|
|
- `alfa_only_slug` → loses alfa's anchor but has NO other owner
|
|
(will be hard-deleted upstream) → excluded
|
|
- `mock_only_slug` → has another (alfa, mock) row anchoring the
|
|
label → excluded
|
|
|
|
Expected: only `shared_slug` is returned.
|
|
"""
|
|
data_directory_path = os.path.join(
|
|
pathlib.Path(__file__).parent.parent.parent.parent,
|
|
".data_storage/test_shared_slugs_detag_anchor",
|
|
)
|
|
cognee.config.data_root_directory(data_directory_path)
|
|
|
|
cognee_directory_path = os.path.join(
|
|
pathlib.Path(__file__).parent.parent.parent.parent,
|
|
".cognee_system/test_shared_slugs_detag_anchor",
|
|
)
|
|
cognee.config.system_root_directory(cognee_directory_path)
|
|
|
|
await cognee.prune.prune_data()
|
|
await cognee.prune.prune_system(metadata=True)
|
|
await setup()
|
|
|
|
user = await get_default_user()
|
|
|
|
dataset = await create_dataset("test_shared_detag_anchor", user=user)
|
|
alfa_dataset_id = dataset.id
|
|
beta_dataset_id = uuid4() # ledger-only: simulate second dataset's rows
|
|
|
|
await set_database_global_context_variables(alfa_dataset_id, user.id)
|
|
|
|
maria_data_id = uuid4()
|
|
mock_data_id = uuid4()
|
|
shared_slug = uuid5(NAMESPACE_OID, "shared")
|
|
alfa_only_slug = uuid5(NAMESPACE_OID, "alfa_only")
|
|
mock_only_slug = uuid5(NAMESPACE_OID, "mock_only")
|
|
|
|
def _make_node(slug, dataset_id, data_id):
|
|
return Node(
|
|
id=uuid4(),
|
|
slug=slug,
|
|
user_id=user.id,
|
|
data_id=data_id,
|
|
dataset_id=dataset_id,
|
|
type="Entity",
|
|
indexed_fields=["name"],
|
|
)
|
|
|
|
rows = [
|
|
_make_node(shared_slug, alfa_dataset_id, maria_data_id),
|
|
_make_node(alfa_only_slug, alfa_dataset_id, maria_data_id),
|
|
_make_node(shared_slug, beta_dataset_id, maria_data_id),
|
|
_make_node(mock_only_slug, alfa_dataset_id, mock_data_id),
|
|
]
|
|
|
|
db_engine = get_relational_engine()
|
|
async with db_engine.get_async_session() as session:
|
|
session.add_all(rows)
|
|
await session.commit()
|
|
|
|
result = await get_shared_slugs_losing_dataset_anchor(alfa_dataset_id, maria_data_id)
|
|
result_slugs = {str(slug) for slug in result}
|
|
|
|
assert result_slugs == {str(shared_slug)}, (
|
|
f"Expected only the co-owned shared_slug to need detag, got {result_slugs}"
|
|
)
|
|
|
|
logger.info("✅ test_get_shared_slugs_losing_dataset_anchor PASSED")
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_get_orphaned_nodeset_labels_for_dataset():
|
|
"""
|
|
The shared-slug scoped detag must strip NodeSet *names* (the values
|
|
stored in `belongs_to_set`), not dataset names. This helper returns
|
|
the NodeSet labels a `(dataset_id, data_id)` row is fully losing for
|
|
the dataset — i.e. NodeSet ledger rows it owns whose anchor no other
|
|
`(dataset_id, *)` row carries.
|
|
|
|
Ledger setup (single dataset `alfa`):
|
|
(alfa, maria) owns NodeSet rows ["T_only_maria", "T_shared"]
|
|
(alfa, marko) owns NodeSet rows ["T_shared", "T_only_marko"]
|
|
(alfa, maria) also owns an Entity slug (unrelated, ignored).
|
|
|
|
Deleting (alfa, maria):
|
|
- `T_only_maria` → no other `(alfa, *)` row anchors this NodeSet → MUST be returned
|
|
- `T_shared` → `(alfa, marko)` still anchors it → EXCLUDED
|
|
- `T_only_marko` → not owned by (alfa, maria) → EXCLUDED
|
|
- Entity row → wrong `type` → EXCLUDED
|
|
|
|
Expected: only `T_only_maria` is returned.
|
|
"""
|
|
data_directory_path = os.path.join(
|
|
pathlib.Path(__file__).parent.parent.parent.parent,
|
|
".data_storage/test_orphaned_nodeset_labels",
|
|
)
|
|
cognee.config.data_root_directory(data_directory_path)
|
|
|
|
cognee_directory_path = os.path.join(
|
|
pathlib.Path(__file__).parent.parent.parent.parent,
|
|
".cognee_system/test_orphaned_nodeset_labels",
|
|
)
|
|
cognee.config.system_root_directory(cognee_directory_path)
|
|
|
|
await cognee.prune.prune_data()
|
|
await cognee.prune.prune_system(metadata=True)
|
|
await setup()
|
|
|
|
user = await get_default_user()
|
|
|
|
dataset = await create_dataset("test_orphaned_nodeset_labels", user=user)
|
|
alfa_dataset_id = dataset.id
|
|
|
|
await set_database_global_context_variables(alfa_dataset_id, user.id)
|
|
|
|
maria_data_id = uuid4()
|
|
marko_data_id = uuid4()
|
|
|
|
t_only_maria_slug = uuid5(NAMESPACE_OID, "NodeSet:T_only_maria")
|
|
t_shared_slug = uuid5(NAMESPACE_OID, "NodeSet:T_shared")
|
|
t_only_marko_slug = uuid5(NAMESPACE_OID, "NodeSet:T_only_marko")
|
|
entity_slug = uuid5(NAMESPACE_OID, "Entity:apple")
|
|
|
|
def _make_node(slug, dataset_id, data_id, type_, label):
|
|
return Node(
|
|
id=uuid4(),
|
|
slug=slug,
|
|
user_id=user.id,
|
|
data_id=data_id,
|
|
dataset_id=dataset_id,
|
|
type=type_,
|
|
label=label,
|
|
indexed_fields=["name"],
|
|
)
|
|
|
|
rows = [
|
|
_make_node(t_only_maria_slug, alfa_dataset_id, maria_data_id, "NodeSet", "T_only_maria"),
|
|
_make_node(t_shared_slug, alfa_dataset_id, maria_data_id, "NodeSet", "T_shared"),
|
|
_make_node(t_shared_slug, alfa_dataset_id, marko_data_id, "NodeSet", "T_shared"),
|
|
_make_node(t_only_marko_slug, alfa_dataset_id, marko_data_id, "NodeSet", "T_only_marko"),
|
|
_make_node(entity_slug, alfa_dataset_id, maria_data_id, "Entity", "Apple"),
|
|
]
|
|
|
|
db_engine = get_relational_engine()
|
|
async with db_engine.get_async_session() as session:
|
|
session.add_all(rows)
|
|
await session.commit()
|
|
|
|
result = await get_orphaned_nodeset_labels_for_dataset(alfa_dataset_id, maria_data_id)
|
|
|
|
assert sorted(result) == ["T_only_maria"], (
|
|
f"Expected only 'T_only_maria' to be returned (T_shared still anchored by marko; "
|
|
f"T_only_marko not owned by maria; Entity row excluded by type), got {result}"
|
|
)
|
|
|
|
logger.info("✅ test_get_orphaned_nodeset_labels_for_dataset PASSED")
|
|
|
|
|
|
if __name__ == "__main__":
|
|
import asyncio
|
|
|
|
asyncio.run(test_get_data_related_nodes_excludes_shared())
|
|
asyncio.run(test_delete_data_nodes_and_edges_removes_from_all_systems())
|
|
asyncio.run(test_get_global_data_related_nodes_scopes_by_dataset())
|
|
asyncio.run(test_get_shared_slugs_losing_dataset_anchor())
|
|
asyncio.run(test_get_orphaned_nodeset_labels_for_dataset())
|
|
asyncio.run(test_get_shared_slugs_losing_dataset_anchor())
|