Skip to content

test(python): improve bindings test to detect ABI incompatibilities#4555

Merged
ObserverOfTime merged 1 commit intotree-sitter:masterfrom
rmuir:python_test_parser
Jul 25, 2025
Merged

test(python): improve bindings test to detect ABI incompatibilities#4555
ObserverOfTime merged 1 commit intotree-sitter:masterfrom
rmuir:python_test_parser

Conversation

@rmuir
Copy link
Contributor

@rmuir rmuir commented Jul 3, 2025

Previously, python bindings test would not detect ABI incompatibilities. At this time, the latest pypi release of py-tree-sitter does not support 15, which is the default, and it creates the problem.

Improve the test to create a Parser, which will detect it:

======================================================================
ERROR: test_can_load_grammar
----------------------------------------------------------------------
Traceback (most recent call last):
  File "../tests/test_binding.py", line 10, in test_can_load_grammar
    _ = Parser(language)
ValueError: Incompatible Language version 15. Must be between 13 and 14

----------------------------------------------------------------------
Ran 1 test in 0.001s

FAILED (errors=1)

ObserverOfTime
ObserverOfTime reviewed Jul 13, 2025
View reviewed changes
@rmuir @ObserverOfTime
feat(bindings): improve python binding test
6320ccb 
Previously, the test would not detect ABI incompatibilities.
@ObserverOfTime ObserverOfTime merged commit 8c61bbd into tree-sitter:master Jul 25, 2025
19 checks passed
@tree-sitter-ci-bot
Copy link

tree-sitter-ci-bot bot commented Jul 25, 2025

Successfully created backport PR for release-0.25:

@tree-sitter-ci-bot tree-sitter-ci-bot bot mentioned this pull request Jul 25, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ObserverOfTime ObserverOfTime ObserverOfTime approved these changes

+1 more reviewer

@alpoz11 alpoz11 alpoz11 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

ci:backport release-0.25 Backport label

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rmuir @ObserverOfTime @alpoz11