camunda / dmn-tck

Decision Model and Notation Technology Compatibility Kit
0 stars 1 forks source link

Clarify `null` handling in different cases #67

Closed nikku closed 1 year ago

nikku commented 1 year ago

What shall we do

Related to this conversation we should clarify the handling of null in DMN FEEL.

Open questions

nikku commented 1 year ago

As discussed with @barmac he plans to bring attention to these issues in the DMN 1.5 RTF call later today.

nikku commented 1 year ago

I've added TCK test cases for the most important cases (filter and path expression null handling): https://github.com/dmn-tck/tck/issues/536#issuecomment-1396591619.

nikku commented 1 year ago

Will follow up with function invocation as I find the time. Let's get all aspects of null handling verified by the TCK :rocket:

nikku commented 1 year ago

With https://github.com/dmn-tck/tck/issues/539 clarified + additional tests merged this issue is closed.

nikku commented 1 year ago

TLDR: DMN FEEL works as I assumed, with invalid conversions / operations resulting in null and an errorResult indicating that there was an undesired conversion.