Sitelet https://github.com/starkware-libs/cairo/pull/9332
Skip to content

fix: correct error message in bool conversion test - #9332

Merged
orizi merged 1 commit into
starkware-libs:mainfrom
lolkdsanyaw:fix/bool-test-error-message
Dec 25, 2025
Merged

orizi merged 1 commit into
starkware-libs:mainfrom
lolkdsanyaw:fix/bool-test-error-message

Conversation

@lolkdsanyaw

Copy link
Copy Markdown
Contributor

Summary

Fix error in bool_test.cairo: error message says 'f.into() != 1' but should be 't.into() != 1' since it tests true, not false.


Type of change

  • Bug fix (fixes incorrect behavior)

Why is this change needed?

Misleading error message during debugging — suggests problem with false when actually testing true.

@reviewable-StarkWare

Copy link
Copy Markdown

This change is Reviewable

@orizi orizi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

@orizi reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @lolkdsanyaw).

@orizi
orizi enabled auto-merge December 25, 2025 11:31
@orizi
orizi added this pull request to the merge queue Dec 25, 2025
Merged via the queue into starkware-libs:main with commit cfae896 Dec 25, 2025
55 checks passed
Princetimix69 pushed a commit to Princetimix69/cairo that referenced this pull request Jan 5, 2026
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.

3 participants