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.

Sign up for GitHub

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

Jump to bottom

feat: Return null on accessing a non-existing property #709

Merged
merged 3 commits into from
Sep 13, 2023
Merged

feat: Return null on accessing a non-existing property #709

merged 3 commits into from
Sep 13, 2023

Commits on Sep 11, 2023

  1. test: Verify access of non-existing property 

    Change the expected behavior if an expression access a non-existing property. Instead of failing the evaluation, it should return null.
    @saig0
    saig0 committed Sep 11, 2023
    Configuration menu
    Copy the full SHA
    66b84a5 View commit details
    Browse the repository at this point in the history

  2. feat: Return null if access of non-existing property 

    Change the behavior for accessing a non-existing property. Instead of failing the evaluation, it returns null instead. A failure with a descriptive message is reported.

This change is aligned with the DMN specification and the new direction to be more null-friendly.
    @saig0
    saig0 committed Sep 11, 2023
    Configuration menu
    Copy the full SHA
    c43706b View commit details
    Browse the repository at this point in the history

  3. refactor: Align new test style with evaluation matchers

    @saig0
    saig0 committed Sep 11, 2023
    Configuration menu
    Copy the full SHA
    309cdfa View commit details
    Browse the repository at this point in the history