spkenv / spk

A Package Manager for high velocity software environments, built on spfs.
https://spkenv.dev
Apache License 2.0
40 stars 6 forks source link

Add test for graph::Manifest conversion from tracking::Manifest #1014

Closed dcookspi closed 6 months ago

dcookspi commented 7 months ago

This adds a test for graph::Manifest conversion from tracking::Manifest to make sure the converted manifest is not missing any entries, by checking it is identical to the original manifest when converted back.

We encountered a situation with our internal, pre-flatbuffers, build where this was not the case. That bug was patched internally, and we confirmed the problem does not exist in post-flatbuffers changes. But want to add the test that detects it to the test suite.

This also relocates the entry related tests that were in the graph::Manifest test file to the entry_test.rs file with the other entry related tests.