Skip to content
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

Initial implementation for resolving records #3606

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

MysterAitch
Copy link
Member

Work-in-progress

ref #2446

  • JavaParser (javaparsermodel) source type solver
  • Jar (javassist) type solver - not started
  • Reflection (reflectionmodel) type solver - tentative

@codecov
Copy link

codecov bot commented Jun 7, 2022

Codecov Report

Merging #3606 (0bfe74b) into master (efcbc7f) will decrease coverage by 0.450%.
The diff coverage is 16.120%.

Impacted file tree graph

@@               Coverage Diff               @@
##              master     #3606       +/-   ##
===============================================
- Coverage     58.332%   57.881%   -0.451%     
- Complexity      2660      2681       +21     
===============================================
  Files            633       637        +4     
  Lines          34348     34702      +354     
  Branches        5842      5911       +69     
===============================================
+ Hits           20036     20086       +50     
- Misses         12258     12543      +285     
- Partials        2054      2073       +19     
Flag Coverage Δ
AlsoSlowTests 57.881% <16.120%> (-0.451%) ⬇️
javaparser-core 53.842% <0.000%> (ø)
javaparser-symbol-solver 36.023% <16.120%> (-0.028%) ⬇️
jdk-10 57.790% <8.196%> (-0.535%) ⬇️
jdk-11 57.787% <8.196%> (-0.538%) ⬇️
jdk-12 57.787% <8.196%> (-0.538%) ⬇️
jdk-13 57.787% <8.196%> (-0.538%) ⬇️
jdk-14 57.868% <15.573%> (-0.457%) ⬇️
jdk-15 57.862% <15.573%> (-0.463%) ⬇️
jdk-16 57.868% <15.573%> (-0.457%) ⬇️
jdk-8 57.795% <8.196%> (-0.535%) ⬇️
jdk-9 57.790% <8.196%> (-0.535%) ⬇️
macos-latest 57.872% <16.120%> (-0.451%) ⬇️
ubuntu-latest 57.866% <16.120%> (-0.451%) ⬇️
windows-latest 57.864% <16.120%> (-0.451%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...solution/declarations/ResolvedTypeDeclaration.java 73.076% <0.000%> (-6.090%) ⬇️
...ver/reflectionmodel/ReflectionEnumDeclaration.java 30.769% <0.000%> (-0.977%) ⬇️
...r/reflectionmodel/ReflectionRecordDeclaration.java 0.000% <0.000%> (ø)
...parsermodel/contexts/RecordDeclarationContext.java 20.000% <20.000%> (ø)
...odel/declarations/JavaParserRecordDeclaration.java 23.225% <23.225%> (ø)
...declarations/ResolvedReferenceTypeDeclaration.java 64.705% <50.000%> (-1.009%) ⬇️
...parser/resolution/types/ResolvedReferenceType.java 81.500% <50.000%> (-0.184%) ⬇️
...vaparsermodel/declarations/AstResolutionUtils.java 81.132% <66.666%> (-1.847%) ⬇️
...thub/javaparser/symbolsolver/JavaSymbolSolver.java 58.992% <75.000%> (+0.474%) ⬆️
...lution/declarations/ResolvedRecordDeclaration.java 100.000% <100.000%> (ø)
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update efcbc7f...0bfe74b. Read the comment docs.

@jlerbsc
Copy link
Collaborator

jlerbsc commented Mar 3, 2023

Hi @MysterAitch Is there any chance you can finish this work?

@jzheng2017
Copy link

Hi @MysterAitch Is there any chance you can finish this work?

Also wondering this..

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.

None yet

3 participants