Skip to content

Conversation

@ivokub
Copy link
Collaborator

@ivokub ivokub commented Nov 11, 2025

We have API tests in internal/backend/circuits package. Move this single test to expected path.


Note

Moves the isCrumb test from test/ into internal/backend/circuits using registry-based cases and removes the old test.

  • Tests:
    • Add internal/backend/circuits/crumb.go:
      • Define isCrumbCircuit and register cases via addNewEntry:
        • "isCrumb/case=good" with valid crumbs []frontend.Variable{0,1,2,3}.
        • "isCrumb/case=bad" with randomized invalid values (math/rand/v2).
    • Remove legacy test/api_assertions_test.go that used testing.T-based assertions for AssertIsCrumb.

Written by Cursor Bugbot for commit 7792ede. This will update automatically on new commits. Configure here.

@ivokub ivokub requested a review from YaoJGalteland November 11, 2025 11:04
@ivokub ivokub self-assigned this Nov 11, 2025
@YaoJGalteland YaoJGalteland merged commit 124b693 into master Nov 12, 2025
9 checks passed
@YaoJGalteland YaoJGalteland deleted the test/crumb branch November 12, 2025 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants