Skip to content

Treesitter java version #126

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Treesitter java version #126

wants to merge 8 commits into from

Conversation

divsan93
Copy link
Collaborator

@divsan93 divsan93 commented Apr 15, 2025

Motivation and Context

Change for enabling aster to run with the latest version of CLDK

How Has This Been Tested?

Confirmed by running Aster on few applications

Breaking Changes

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the Codellm-Devkit Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

@divsan93 divsan93 requested a review from rahlk April 15, 2025 16:00
Copy link
Collaborator

@rahlk rahlk left a comment

Choose a reason for hiding this comment

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

Looks good. Please:

  • Create an issue or tag this PR with an existing issue
  • Update the PR and issue descriptions with all context
  • Ensure no failing test cases

@rahlk rahlk self-requested a review April 15, 2025 17:32
@divsan93 divsan93 linked an issue Apr 24, 2025 that may be closed by this pull request
@divsan93 divsan93 force-pushed the treesitter-java-version branch from 4516a27 to 59fc080 Compare April 24, 2025 19:04
@rahlk
Copy link
Collaborator

rahlk commented Apr 29, 2025

25 unit tests are failing.

Steps to reproduce:

  1. Clone repo: git clone git@github.com:codellm-devkit/python-sdk
  2. Switch to this branch git checkout treesitter-java-version
  3. Run unit tests: poetry install && poetry run pytest --pspec --cov=cldk --disable-warnings

I get the below failures---

================================================================================================================================================================= short test summary info ==================================================================================================================================================================
FAILED tests/analysis/python/test_python_analysis.py::::Should return all of the methods - TypeError: an integer is required
FAILED tests/analysis/python/test_python_analysis.py::::Should return all of the functions - TypeError: an integer is required
FAILED tests/analysis/python/test_python_analysis.py::::Should return all of the modules - TypeError: an integer is required
FAILED tests/analysis/python/test_python_analysis.py::::Should return the method details - TypeError: an integer is required
FAILED tests/analysis/python/test_python_analysis.py::::Should be able to parse the code - TypeError: an integer is required
FAILED tests/analysis/python/test_python_analysis.py::::Should return the raw AST - TypeError: an integer is required
FAILED tests/analysis/python/test_python_analysis.py::::Should return all of the imports - TypeError: an integer is required
FAILED tests/analysis/python/test_python_analysis.py::::Should return all of the variables - TypeError: an integer is required
FAILED tests/analysis/python/test_python_analysis.py::::Should return all of the classes - TypeError: an integer is required
FAILED tests/analysis/python/test_python_analysis.py::::Should return all of the classes that match the criteria - TypeError: an integer is required
FAILED tests/analysis/python/test_python_analysis.py::::Should return all of the subclasses - TypeError: an integer is required
FAILED tests/analysis/python/test_python_analysis.py::::Should return all of the nested classes - TypeError: an integer is required
FAILED tests/analysis/python/test_python_analysis.py::::Should return all of the constructors - TypeError: an integer is required
FAILED tests/analysis/python/test_python_analysis.py::::Should return all of the methods in the class - TypeError: an integer is required
FAILED tests/analysis/python/test_python_analysis.py::::Should return all of the fields in the class - TypeError: an integer is required
FAILED tests/analysis/python/test_treesitter_python.py::::Should be able to parse the code - TypeError: an integer is required
FAILED tests/analysis/python/test_treesitter_python.py::::Should return the raw AST - TypeError: an integer is required
FAILED tests/analysis/python/test_treesitter_python.py::::Should return all of the methods - TypeError: an integer is required
FAILED tests/analysis/python/test_treesitter_python.py::::Should return all of the functions - TypeError: an integer is required
FAILED tests/analysis/python/test_treesitter_python.py::::Should return the method details - TypeError: an integer is required
FAILED tests/analysis/python/test_treesitter_python.py::::Should return all of the imports - TypeError: an integer is required
FAILED tests/analysis/python/test_treesitter_python.py::::Should return the module details - TypeError: an integer is required
FAILED tests/analysis/python/test_treesitter_python.py::::Should return all of the import details - TypeError: an integer is required
FAILED tests/analysis/python/test_treesitter_python.py::::Should return all of the classes - TypeError: an integer is required
FAILED tests/analysis/python/test_treesitter_python.py::::Should return all of the modules - TypeError: an integer is required
======================================================================================================================================================== 25 failed, 98 passed in 218.64s (0:03:38) =========================================================================================================================================================

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.

Enable Aster to run with latest version of CLDK
3 participants