stacklok / minder / 11269207927

push

github

Use single and unique entity ID to deal with entity info wrapper (#4704)

* Use single and unique entity ID to deal with entity info wrapper

This drops our reliance on a per-entity-type ID and instead just uses a
single entity ID in the entity info wrapper. The intention is to
simplify the code and make adding new entities simpler.

Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>

Fix entity ID serialized in wrong place and data race issue

Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>

Use local error variable for EEA test

Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>

Add more guardrails

Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>

fix more unit tests

Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>

remove unused function

Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>

fix tests

Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>

* Use entity ID instead of repo ID in message meta in gh handler test

Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>

---------

Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>

65 of 100 new or added lines in 8 files covered. (65.0%)

14441 of 27086 relevant lines covered (53.32%)

41.72 hits per line