stacklok / minder / 11163840094

53%

Build Type

push

github

Committed by web-flow

Commit Message

Remove RefreshRepositoryByUpstreamID from the repo reconciler (#4620)

* Rename WithOwner to WithOriginator

We renamed Owner to Originator in the new refresh-and-do handlers, but
forgot to rename the method that sets that attribute.

* Add a new Get-By-ID-Refresh-And-Do handler

* Use NewEntityRefreshAndDoMessage instead of calling RefreshRepositoryByUpstreamID in reconcilers

Gets rid of the RefreshRepositoryByUpstreamID request that we are tryign
to phase out in favour of the generic new handlers. This will also allow
the reconcile handler to be useful for the github reconciler.

* Handle messages with no entityID

* Do not retry the event if we can't unmarshall the JSON

* Add a test for reconciling a repo with string upstream ID

Run Details

83 of 120 new or added lines in 13 files covered. (69.17%)

3 existing lines in 2 files now uncovered.

14478 of 27239 relevant lines covered (53.15%)

41.58 hits per line