Skip to content

Fix broken async partitioner integration tests - #193

Open
Leon-Algo wants to merge 1 commit into
InternScience:mainfrom
Leon-Algo:pr-a-sync-partitioner-tests
Open

Leon-Algo wants to merge 1 commit into
InternScience:mainfrom
Leon-Algo:pr-a-sync-partitioner-tests

Conversation

@Leon-Algo

Copy link
Copy Markdown

Summary

  • Convert BFS/DFS/ECE partitioner integration tests from broken @pytest.mark.asyncio coroutine calls to synchronous calls matching the current synchronous BasePartitioner/storage APIs.
  • Fix the invalid from graphgen.models import NetworkXStorage import (storage lives in graphgen.storage); these tests fail to even collect on main today.
  • No production code changes; all 12 partitioner integration tests pass.

This is the first of three PRs split from #192 per review feedback (smallest first). PR B (prompt profiles/gates/concurrency/LLM patches) and PR C (event-entity pipeline) follow separately.

Validation

  • pytest tests/integration_tests/models/partitioner/ → 12 passed.

🤖 Generated with Claude Code

移除 asyncio 标记并同步调用存储与分区接口。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant