Skip to content

tests: update expected Binary Ninja version to 5.3#3011

Merged
mike-hunhoff merged 1 commit intomasterfrom
fix-binja-version-test
Apr 16, 2026
Merged

tests: update expected Binary Ninja version to 5.3#3011
mike-hunhoff merged 1 commit intomasterfrom
fix-binja-version-test

Conversation

@mr-tz
Copy link
Copy Markdown
Collaborator

@mr-tz mr-tz commented Apr 16, 2026

Updates the expected Binary Ninja version in test_binja_version to 5.3. This fix addresses the test failure caused by the recent update of Binary Ninja to version 5.3 in the environment.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Please add bug fixes, new features, breaking changes and anything else you think is worthwhile mentioning to the master (unreleased) section of CHANGELOG.md. If no CHANGELOG update is needed add the following to the PR description: [x] No CHANGELOG update needed

@mr-tz mr-tz force-pushed the fix-binja-version-test branch from f791240 to df6df82 Compare April 16, 2026 10:36
@github-actions github-actions Bot dismissed their stale review April 16, 2026 10:36

CHANGELOG updated or no update needed, thanks! 😄

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the Binary Ninja version check in the test suite from 5.2 to 5.3. The review feedback suggests using a minimum version check with tuple comparison instead of an exact equality check to make the test more robust against future version updates.

Comment thread tests/test_binja_features.py Outdated
@mr-tz mr-tz force-pushed the fix-binja-version-test branch from df6df82 to 5eb9a0f Compare April 16, 2026 10:43
@mr-tz
Copy link
Copy Markdown
Collaborator Author

mr-tz commented Apr 16, 2026

cc: @xusheng6

@mr-tz mr-tz requested a review from a team April 16, 2026 13:52
@xusheng6
Copy link
Copy Markdown
Contributor

cc: @xusheng6

Looks good to me. This will free us from building the version every time we release a new stable binja

@mike-hunhoff mike-hunhoff merged commit 557f521 into master Apr 16, 2026
39 checks passed
@mike-hunhoff mike-hunhoff deleted the fix-binja-version-test branch April 16, 2026 18:35
@mr-tz
Copy link
Copy Markdown
Collaborator Author

mr-tz commented Apr 16, 2026

Thanks!

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