beanshell / beanshell

Beanshell scripting language
Apache License 2.0
815 stars 183 forks source link

Hidden field access on super classes #690

Closed nickl- closed 1 year ago

nickl- commented 1 year ago

Fixes #358 cast instance This for generated super type Fixes #357 make super hidden variables ačcessible Fixed get super namespace to include class namespace of parent classes not just scope Cast to instance This and allow assignment to typed variable of cast type Instance this cast variable getClass reports the class of the instance Also suppressing debug output for get prompt which is unnecessarily noisy

codecov[bot] commented 1 year ago

Codecov Report

Merging #690 (580a6e7) into master (e0b096d) will increase coverage by 0.02%. The diff coverage is 78.94%.

@@             Coverage Diff              @@
##             master     #690      +/-   ##
============================================
+ Coverage     72.13%   72.15%   +0.02%     
- Complexity     2868     2877       +9     
============================================
  Files           107      107              
  Lines          9201     9213      +12     
  Branches       1796     1804       +8     
============================================
+ Hits           6637     6648      +11     
  Misses         2171     2171              
- Partials        393      394       +1     
Flag Coverage Δ
unittests 72.15% <78.94%> (+0.02%) :arrow_up:

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

Impacted Files Coverage Δ
src/main/java/bsh/BSHTypedVariableDeclaration.java 100.00% <ø> (ø)
src/main/java/bsh/Reflect.java 82.49% <0.00%> (ø)
src/main/java/bsh/This.java 80.71% <0.00%> (+0.26%) :arrow_up:
src/main/java/bsh/Variable.java 78.94% <ø> (ø)
src/main/java/bsh/Types.java 76.35% <83.33%> (+0.07%) :arrow_up:
src/main/java/bsh/NameSpace.java 86.38% <87.50%> (-0.04%) :arrow_down:
src/main/java/bsh/Interpreter.java 79.27% <100.00%> (+0.13%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more