Skip to content

Skip clustering test: clustering_targets not in targetdb#142

Merged
monodera merged 1 commit into
mainfrom
fix/skip-clustering-test
May 21, 2026
Merged

Skip clustering test: clustering_targets not in targetdb#142
monodera merged 1 commit into
mainfrom
fix/skip-clustering-test

Conversation

@monodera

@monodera monodera commented May 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • tests/test_clustering.py was causing an ImportError at collection time because clustering_targets no longer exists in the targetdb package
  • Removed the module-level from targetdb import clustering_targets import
  • Added @pytest.mark.skip decorator with a descriptive reason
  • Kept the test body intact (with # noqa: F821) as a reference if clustering is re-introduced in the future

Test plan

  • pytest tests/ collects and runs without errors
  • test_run_clustering shows as SKIPPED in output
  • test_relations_consistency continues to pass

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@codacy-production

Copy link
Copy Markdown

Not up to standards ⛔

🔴 Issues 1 high

Alerts:
⚠ 1 issue (≤ 0 issues of at least minor severity)

Results:
1 new issue

Category Results
Security 1 high

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@monodera monodera merged commit 549c9b4 into main May 21, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant