IRL2 / nanover-imd

https://irl2.github.io/nanover-docs/#
Other
0 stars 1 forks source link

Run the tests in CI #11

Closed jbarnoud closed 8 months ago

jbarnoud commented 8 months ago

The current CI make the build but does not run the tests. This PR adds a job that runs the tests.

jbarnoud commented 8 months ago

The current issue with this pull request is that some tests are failing. The error seems to be related to loading the ChemicalComponentDictionary; the dictionary's content is passed as a compressed file encoded in base64 and stored in the serialised field of an asset. Loading this asset appears recursive in some cases, making Unity unhappy.

Investigating this part of the code, here are things I notice and should be cleaned up: