search

camunda / feel-scala

FEEL parser and interpreter written in Scala
https://camunda.github.io/feel-scala/
Apache License 2.0
120 stars 50 forks source link

Support list and context values for string() function #650

Closed saig0 closed 8 months ago

saig0 commented 1 year ago

Describe the bug Currently, the string() function doesn't work for list or context values. Instead, it returns null.

To Reproduce Steps to reproduce the behavior:

  1. Evaluate the expression string([1,2,3])
  2. Verify that the result is null

Expected behavior The string() function returns the string representation of list and context values. (see related DMN TCK case).

string([1, 2, 3])
// "[1, 2, 3]"

string({x: 1})
// "{x: 1}"

Environment

saig0 commented 1 year ago

Related OMG issue:

saig0 commented 11 months ago

The issue was fixed by https://github.com/camunda/feel-scala/pull/717 in version 1.17.0 as a side effect. :rocket:

We need to add a few test cases to verify the behavior of the string() function before closing the issue.