Skip to content

Fix broken async partitioner integration tests - #193

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

Leon-Algo wants to merge 2 commits 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>
@Leon-Algo

Copy link
Copy Markdown
Author

Retriggering CI — the third-party translate action failed with a transient HTTP error (same workflow passes on #194/#195).

@Leon-Algo Leon-Algo closed this Oct 21, 2026
@Leon-Algo Leon-Algo reopened this Oct 21, 2026
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Leon-Algo

Copy link
Copy Markdown
Author

Friendly ping for review when you have a moment 🙂 The three-way split from #192 is ready and stable:

CI note: the earlier translate red X here was a transient HTTPError from the third-party translate action (it passes on #194/#195 and on rerun); the branch head was advanced to retrigger it. Happy to adjust anything — thanks!

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