fix: resolve refs in EnumArrayRenderer by LukasBoll · Pull Request #2204 · eclipsesource/jsonforms
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @LukasBoll the changes look pretty good to me. There is one typo fix inline that also fixes the out of memory of the tests. I'm not sure why the typo led to an out of memory but it should not be related to this PR:
Likely it is ue to the implementation of mapStateToMultiEnumControlProps. However, I did not see anything obvious there. Furthermore, in a running application the method should never be invoked because the tester does not apply for effectively empty items. I tested this in the example material app and there simply a No applicable renderer found. is shown.
| colors: { | ||
| type: 'array', | ||
| items: { | ||
| ref$: '#/definitions/colors', |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixing this typo makes the tests work again locally for me. I.e. it fixes the out of memory issue of the tests.
| ref$: '#/definitions/colors', | |
| $ref: '#/definitions/colors', |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, what a stupid mistake. The test is working now, thank you for taking a look @lucas-koehler !
I think the out of memory issues occurred in a view other test cases as well.
So maybe there is an issue with one of the core functionalities, that is called by mapStateToMultiEnumControlProps.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I am also aware of at least on other PR with out of memory issues. Thanks for the hint. However, we don't need to look at that in this PR :D