Fix bug with Engine node

[saintentropy]

Purpose

The Graph Engine node is failing in some circumstances due to incorrect check. Currently the values pushed to DefineData via Player are serialized via ParseJSON(playerValue); which does not return an ArrayList.

Declarations

Check these if you believe they are true

• Is documented according to the standards
• The level of testing this PR includes is appropriate
• Changes to the API follow Semantic Versioning and are documented in the API Changes document.

Release Notes

Engine node bug is fixed for supporting list data.

Reviewers

@twastvedt

FYIs

[Copilot]

Pull Request Overview

This PR fixes a bug in the Engine node where list data validation was failing due to an incorrect type check. The node was checking for ArrayList specifically, but values serialized through ParseJSON(playerValue) via the Player do not return ArrayList instances, causing valid list data to be incorrectly rejected.

Key Changes:

• Updated type check from ArrayList to IEnumerable to support all enumerable collections

[QilongTang]

Is this PR aiming for 4.0 release or 4.1? Curious if there is a Jira task

[saintentropy]

4.1 I think @QilongTang

[QilongTang]

@copilot Help me create a unit test on this code change

[Copilot]

@QilongTang I've opened a new pull request, #16888, to work on those changes. Once the pull request is ready, I'll request review from you.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud