stacklok / toolhive / 17674767881

73%

Build Type

push

github

Committed by web-flow

Commit Message

[Kubernetes MCPRegistry] ConfigMap source and store manager (#1829)

* - SourceHandler interface
- ConfigMap source handler
- Basic unit tests

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* - SourceHandler and FormatConverter types

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* using Registry schema

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* Refactor ConfigMap handling to use a constant key for registry data

- Updated tests and implementation to replace hardcoded "registry.json" key with ConfigMapSourceDataKey constant.
- Introduced ConfigMapStorageManager for managing registry data in ConfigMaps, including methods for storing, retrieving, and deleting data.
- Added comprehensive unit tests for the new storage manager functionality.

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* Refactor storage manager tests to use constant for ConfigMap key

- Updated test cases to replace hardcoded "registry.json" key with ConfigMapStorageDataKey constant.
- Ensured consistency in data handling across storage manager methods.

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* Refactor registry handling and improve test coverage

- Updated ConfigMapSourceHandler to validate and parse registry data directly, removing legacy methods.
- Enhanced test cases to utilize a new TestRegistryBuilder for generating test data, improving readability and maintainability.
- Introduced new methods for handling registry data in a more structured way, including validation and conversion between formats.
- Updated SyncResult to encapsulate parsed registry data instead of raw data, streamlining the sync process.

Signed-off-by: Daniele Martinoli <dmartino@redhat.com>

* Refactor storage manager to use constant for component label

- Updated the storage manager implementation and tests to replace the hardcoded "registry-storage" component label with the new RegistryStorageComponent constant.
- Improved consistency and maintain... (continued)

Coverage Stats

356 of 636 new or added lines in 7 files covered. (55.97%)

10 existing lines in 4 files now uncovered.

13174 of 31790 relevant lines covered (41.44%)

16.67 hits per line