Skip to content

tests: pin CHANGELOG claims that no test covered - #53

Merged
petercorke merged 1 commit into
mainfrom
test/pin-changelog-claims
Oct 12, 2026
Merged

petercorke merged 1 commit into
mainfrom
test/pin-changelog-claims

Conversation

@petercorke

Copy link
Copy Markdown
Owner

Tests only, no source change. An audit of the 1.0.0 CHANGELOG against the test suite found three claims nothing pinned:

  • path_BFS/path_UCS/path_Astar validate the type of the goal (0.6.5 never checked it: BFS/UCS accepted 42, A* died with AttributeError).
  • Components are not recomputed while the graph is unchanged (a spy on the traversal; the existing test only looked at flags).
  • dotfile() to the console leaves sys.stdout open (0.6.5 closed it).

The old behaviors were confirmed against a git archive v0.6.5 export. Edge.vertices() was on the list but is already covered by test_edge_vertices_deprecated.

607 tests pass locally.

🤖 Generated with Claude Code

Planners validate the goal's type (0.6.5 never checked it: BFS/UCS accepted
42, A* died with AttributeError), components are not recomputed while the
graph is unchanged (spy on the traversal), and dotfile() leaves sys.stdout
open (0.6.5 closed it).

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

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 4 complexity · 0 duplication

Metric Results
Complexity 4
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.

@codecov

codecov Bot commented Oct 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.41%. Comparing base (3898a89) to head (4ad159a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #53   +/-   ##
=======================================
  Coverage   96.41%   96.41%           
=======================================
  Files           3        3           
  Lines        1339     1339           
=======================================
  Hits         1291     1291           
  Misses         48       48           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@petercorke
petercorke merged commit b965468 into main Oct 12, 2026
14 checks passed
@petercorke
petercorke deleted the test/pin-changelog-claims branch October 12, 2026 22:41
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