search

LionWeb-io / lionweb-repository

Reference implementation of LionWeb repository
Apache License 2.0
2 stars 1 forks source link

Correct serialization of responses #53

Closed ftomassetti closed 2 months ago

ftomassetti commented 2 months ago

...for nodesByClassifier and nodesByLanguage.

The way in which the JSON was built (manually, to use streams) was incorrect. Also, the expected results did not contain the additional size fields.

This correct an issue introduced in #50

ftomassetti commented 2 months ago

This should fix the failures of the tests reported by @joswarmer . I used a tool (act) to run the GitHub actions locally, however it would be convenient for GitHub actions to run as part of the PR, so that we can more easily catch PR breaking tests. I will open an issue for that