-
|
I have tried following the instructions in contributing.md but when running all tests I consistently get an error running the test Load DialogueGraph, the error message is: I have found that the test tries to load simple.json which finds a file with the path 'FXGL/fxgl-samples/target/classes/assets/dialogues/simple.json', the subsequent test Load DialogueGraph from URL specifies the file as being under fxglassets which gets the json file under 'FXGL/fxgl/target/test-classes/fxglassets/dialogues/simple.json' which works perfectly. If I rename the simple.json file under fxgl-samples that fixes the problem as the test finds the one under the fxgl module. I am not sure if the assetLoader is supposed to find the file under fxgl-samples or fxgl, if it is the former I don't know why the assertion fails as the json does have 3 nodes and if it is the latter I don't know why the assetLoader prioritises searching under the fxgl-samples module. I was hoping someone might have a suggestion or solution, thank you |
Beta Was this translation helpful? Give feedback.
Replies: 1 comment
-
|
Hi, thanks for this. You are right, this is a problem. Given you already have a fix, feel free to submit a PR for it. When the |
Beta Was this translation helpful? Give feedback.
Hi, thanks for this. You are right, this is a problem. Given you already have a fix, feel free to submit a PR for it.
When the
fxgl-samplesjson file is renamed, we also need to rename it in whatever sample is accessing that json.