Return a list of dates instead of a range

[AthiraHari77]

There is a DMN TCK failure, when running with Drools:

[ERROR] 0084-feel-for-loops-test-01.decision_024 – Time elapsed: 0.001 s <<< ERROR!
java.lang.RuntimeException: ERROR: FAILURE: 'decision_024' expected='[1980-01-01, 1980-01-02, 1980-01-03]' but found='[[ 1980-01-01 .. 1980-01-03 ]]'

The test is located here (1). Based on the error, it looks like that Drools is returning a range instead of a list in this test case.

Options how to debug:

Replicating the test case directly in Drools repository, so it gets a proper stack trace pointing to the code path being executed, to be able to see the code areas, which may need a fix. If the test case is mainly just a FEEL expression (FEEL is a language used to write expressions in DMN), the FEEL expression could be added to a test class, e.g. this one (2), so it can be run in isolation.

(1) https://github.com/dmn-tck/tck/tree/master/TestCases/compliance-level-3/0084-feel-for-loops

(2) https://github.com/apache/incubator-kie-drools/blob/053d70c8f9145706e771f90e8ce2dafa3899b59b/kie-dmn/kie-dmn-feel/src/test/java/org/kie/dmn/feel/runtime/FEELConditionsAndLoopsTest.java#L31

[jomarko]

Thank you for reporting the issue. I am not expert in DMN arrays syntax. To me the current behavior makes a sense to some extent. However, if you say it is wrong, then probably more cases needs to be addressed. I mean decision_020_a for example:

for i in [@"1980-01-03T00:00:00"..@"1980-01-01T00:00:00"] return i

It currently also returns [[ 1980-01-03T00:00 .. 1980-01-01T00:00 ]]

[baldimir]

Hi @jomarko, the test mentioned in the description is just one picked for fixing. The actual fix may fix also other TCK failures, like the one you mentioned.

[yesamer]

Failures in 1156 Test suite could be related, it would be nice to double check.