Skip to content

fix: update catalog item fixtures and display for multi-resource schema#28

Merged
chadcrum merged 1 commit into
dcm-project:mainfrom
chadcrum:flpath-4617-cli-test-multi-resource-schema
Jul 21, 2026
Merged

fix: update catalog item fixtures and display for multi-resource schema#28
chadcrum merged 1 commit into
dcm-project:mainfrom
chadcrum:flpath-4617-cli-test-multi-resource-schema

Conversation

@chadcrum

@chadcrum chadcrum commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Update test fixtures in catalog_item_test.go to use the new multi-resource catalog item schema (spec.resources[] instead of flat spec.service_type)
  • Fix catalogItemTableDef RowFunc in catalog_item.go to extract service_type from resources[0] so the SERVICE TYPE table column renders correctly against the updated control-plane

Relates to: FLPATH-4617

Depends on control-plane PR #11 (dcm-project/control-plane#11) which introduced the multi-resource catalog item schema.

User Impact

Users will now see catalog items in the CLI correctly when the control-plane uses the new multi-resource schema, with service type information displayed properly in table output.

The control-plane moved to a multi-resource catalog item schema where
service_type lives under spec.resources[] instead of directly in spec.
Update test fixtures to match and fix the table display RowFunc to
extract service_type from the first resource entry.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Chad Crum <1462069+chadcrum@users.noreply.github.com>
@chadcrum
chadcrum marked this pull request as ready for review July 21, 2026 13:22
@chadcrum
chadcrum requested a review from a team as a code owner July 21, 2026 13:22

@gciavarrini gciavarrini left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@chadcrum
chadcrum merged commit 7316690 into dcm-project:main Jul 21, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants