Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Possible issue when writing spatials that have lists as userdata #2108

Open
riccardobl opened this issue Oct 14, 2023 · 0 comments
Open

Possible issue when writing spatials that have lists as userdata #2108

riccardobl opened this issue Oct 14, 2023 · 0 comments
Labels
bug Something that is supposed to work, but doesn't. More severe than a "defect". To be investigated

Comments

@riccardobl
Copy link
Member

riccardobl commented Oct 14, 2023

I wrote an unit test for this pr: #2093 and i've discovered a possible issue in the serialization of user data (that is unrelated to the PR).
You can reproduce it by setting testLists=true in the unit test:
https://github.com/jMonkeyEngine/jmonkeyengine/blob/master/jme3-plugins/src/test/java/com/jme3/export/JmeExporterTest.java#L139-L239
and then run ./gradlew test
It will fail with this log:

com.jme3.export.JmeExporterTest > testExporterConsistency[0] FAILED
    java.lang.AssertionError: Value does not match in map 1 for key int-list expected [1, 2, 3] but got null
        at org.junit.Assert.fail(Assert.java:89)
        at org.junit.Assert.assertTrue(Assert.java:42)
        at com.jme3.export.JmeExporterTest.compareMaps(JmeExporterTest.java:260)
        at com.jme3.export.JmeExporterTest.testExporterConsistency(JmeExporterTest.java:238)

@riccardobl riccardobl added bug Something that is supposed to work, but doesn't. More severe than a "defect". To be investigated labels Oct 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something that is supposed to work, but doesn't. More severe than a "defect". To be investigated
Projects
None yet
Development

No branches or pull requests

1 participant