Skip to content

Upgrade OGNL to 3.4.7 and expand test coverage#9

Merged
marevol merged 1 commit intomainfrom
chore/upgrade-ognl-and-expand-tests
Mar 15, 2026
Merged

Upgrade OGNL to 3.4.7 and expand test coverage#9
marevol merged 1 commit intomainfrom
chore/upgrade-ognl-and-expand-tests

Conversation

@marevol
Copy link
Contributor

@marevol marevol commented Mar 15, 2026

Summary

  • Upgrade OGNL dependency from 3.2.20 to 3.4.7
  • Add @Test annotations to all existing test methods
  • Add comprehensive tests for OGNL-specific features

Changes Made

  • pom.xml: Bump ognl.version from 3.2.20 to 3.4.7
  • OgnlEngineTest.java:
    • Add missing @Test annotations and org.junit.jupiter.api.Test import
    • Add tests for collection projection ({field}) and selection ({? condition}, {^ ...}, {$ ...})
    • Add tests for constructor invocation via OGNL
    • Add tests for static field access (@Class@FIELD) and static method calls (@Class@method())
    • Add tests for chained method calls
    • Add tests for type conversion and mixed-type arithmetic
    • Add tests for OGNL context variables (#this)
    • Add tests for comma (sequence) expressions
    • Add tests for in / not in operators
    • Add tests for assignment expressions
    • Add tests for list and map literals
    • Add tests for null value handling and null property access
    • Add tests for deeply nested expressions and large collection operations
    • Add tests for combined projection and selection
    • Add tests for bitwise operations (&, |, ^, <<, >>)

Testing

  • All new tests validate OGNL expression evaluation through OgnlEngine.evaluate()
  • Tests cover both positive cases and error/edge cases

🤖 Generated with Claude Code

Bump ognl dependency from 3.2.20 to 3.4.7. Add @test annotations to
existing test methods and add comprehensive tests for OGNL-specific
features including collection projection/selection, static field and
method access, constructor invocation, chained method calls, type
conversion, context variables, comma/in/assignment expressions, list
and map literals, null handling, bitwise operations, and edge cases.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@marevol marevol merged commit 7ce9065 into main Mar 15, 2026
1 check passed
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.

1 participant