chore: cli - update LocalBlob/v1 assertions to UpperCamelCase [3/4] by morri-son · Pull Request #2325 · open-component-model/open-component-model

morri-son · 2026-04-17T20:14:13Z

Summary

Updates test assertion strings in cli/cmd/cmd_test.go from "localBlob/v1" to "LocalBlob/v1" to match the new primary UpperCamelCase type name.

Part of the localBlob → LocalBlob UpperCamelCase migration for ocm-project/ocm-project#962.

Changes

cli/cmd/cmd_test.go: 5 assertion strings updated ("localBlob/v1" → "LocalBlob/v1")
- Line 829: r.Equal assertion
- Line 1240: YAML output string
- Line 1353: JSON map value
- Lines 1777, 1817: r.Equal assertions

Not changed: --upload-as localBlob CLI flag (stays lowercase — enum value, not a type name)

Dependencies

Depends on #2323 being merged and published (descriptor/v2 with LocalBlob as primary type).

Related PRs

chore: descriptor - rename in tests: localBlob type to LocalBlob UpperCamelCase [1/4] #2323 — PR 1: descriptor/v2 + descriptor/runtime (constants, scheme, tests)
chore: constructor - update LocalBlob YAML fixtures to UpperCamelCase [2/4] #2324 — PR 2: constructor (YAML fixtures)
This PR — PR 3: cli (test assertions)
chore: controller - update LocalBlob/v1 type assertions to UpperCamelCase [4/4] #2326 — PR 4: kubernetes/controller (test JSON)

Resolves part of ocm-project/ocm-project#962

netlify · 2026-04-17T20:14:18Z

✅ Deploy Preview for ocm-website ready!

Name	Link
🔨 Latest commit	`0501265`
🔍 Latest deploy log	https://app.netlify.com/projects/ocm-website/deploys/69f06e0d56279500085cca9e
😎 Deploy Preview	https://deploy-preview-2325--ocm-website.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

coderabbitai · 2026-04-17T20:14:21Z

Warning

Rate limit exceeded

@morri-son has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 1 minutes and 58 seconds before requesting another review.

To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 5d8ef42b-f483-4b42-a75d-100f1835a2d1

📥 Commits

Reviewing files that changed from the base of the PR and between f71796b and 0501265.

⛔ Files ignored due to path filters (2)

cli/go.sum is excluded by !**/*.sum
cli/integration/go.sum is excluded by !**/*.sum

📒 Files selected for processing (3)

cli/cmd/cmd_test.go
cli/go.mod
cli/integration/go.mod

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

… [2/4] (#2324) ## Summary Updates YAML test fixtures in `bindings/go/constructor/` to use `LocalBlob` (UpperCamelCase) instead of `localBlob`. Part of the localBlob → LocalBlob UpperCamelCase migration for ocm-project/ocm-project#962. ## Changes - `construct_resource_test.go`: YAML fixture `type: localBlob` → `type: LocalBlob` - `construct_source_test.go`: YAML fixture `type: localBlob` → `type: LocalBlob` ## Dependencies Depends on #2323 being merged and published (descriptor/v2 with `LocalBlob` as primary type). ## Related PRs - #2323 — PR 1: descriptor/v2 + descriptor/runtime (constants, scheme, tests) - **This PR** — PR 2: constructor (YAML fixtures) - #2325 — PR 3: cli (test assertions) - #2326 — PR 4: kubernetes/controller (test JSON) Resolves part of ocm-project/ocm-project#962 --------- Signed-off-by: Gerald Morrison (SAP) <gerald.morrison@sap.com> Co-authored-by: Gerald Morrison (SAP) <gerald.morrison@sap.com>

… [2/4] (open-component-model#2324) ## Summary Updates YAML test fixtures in `bindings/go/constructor/` to use `LocalBlob` (UpperCamelCase) instead of `localBlob`. Part of the localBlob → LocalBlob UpperCamelCase migration for ocm-project/ocm-project#962. ## Changes - `construct_resource_test.go`: YAML fixture `type: localBlob` → `type: LocalBlob` - `construct_source_test.go`: YAML fixture `type: localBlob` → `type: LocalBlob` ## Dependencies Depends on open-component-model#2323 being merged and published (descriptor/v2 with `LocalBlob` as primary type). ## Related PRs - open-component-model#2323 — PR 1: descriptor/v2 + descriptor/runtime (constants, scheme, tests) - **This PR** — PR 2: constructor (YAML fixtures) - open-component-model#2325 — PR 3: cli (test assertions) - open-component-model#2326 — PR 4: kubernetes/controller (test JSON) Resolves part of ocm-project/ocm-project#962 --------- Signed-off-by: Gerald Morrison (SAP) <gerald.morrison@sap.com> Co-authored-by: Gerald Morrison (SAP) <gerald.morrison@sap.com> e25a69c

…Case [4/4] (#2326) ## Summary Updates test JSON strings and hardcoded type references in `kubernetes/controller/` to use `LocalBlob/v1` (UpperCamelCase) instead of `localBlob/v1`. Part of the localBlob → LocalBlob UpperCamelCase migration for ocm-project/ocm-project#962. ## Changes - `deployer_controller_test.go`: 9× `{"type":"localBlob/v1"}` → `{"type":"LocalBlob/v1"}` - `deployer_controller_rgd_test.go`: 4× same replacement - `cel/functions/oci_test.go`: replaced hardcoded `runtime.NewVersionedType("localBlob", "v1")` with `runtime.NewVersionedType(v2.LocalBlobAccessType, v2.LocalBlobAccessTypeVersion)` using the exported constant ## Dependencies Depends on #2323 being merged and published (descriptor/v2 with `LocalBlob` as primary type). ## Related PRs - #2323 — PR 1: descriptor/v2 + descriptor/runtime (constants, scheme, tests) - #2324 — PR 2: constructor (YAML fixtures) - #2325 — PR 3: cli (test assertions) - **This PR** — PR 4: kubernetes/controller (test JSON) Resolves part of ocm-project/ocm-project#962 --------- Signed-off-by: Gerald Morrison (SAP) <gerald.morrison@sap.com> Co-authored-by: Gerald Morrison (SAP) <gerald.morrison@sap.com>

Signed-off-by: Gerald Morrison (SAP) <gerald.morrison@sap.com>

… [1/4] (#2323) ## Summary Renames `localBlob` → `LocalBlob` (UpperCamelCase) as the primary type name, preserving `localBlob` as a legacy alias for backward compatibility. This is the first of 4 PRs completing the `localBlob` portion of ocm-project/ocm-project#962. Follows the exact pattern established in #2057 for File, Dir, UTF8, and Helm input types. ## Changes **Production code:** - `descriptor/v2/local_access.go` — `LocalBlobAccessType = "LocalBlob"`, new `LegacyLocalBlobAccessType = "localBlob"`, added `+ocm:jsonschema-gen:enum` markers - `descriptor/v2/scheme.go` — 4 aliases: `LocalBlob/v1` (primary), `LocalBlob`, `localBlob/v1` (legacy), `localBlob` - `descriptor/runtime/local_access.go` — same constant + marker changes **Tests:** - `descriptor/v2/local_access_test.go` — marshal asserts `"LocalBlob/v1"`, added `_UpperCamelCase` unmarshal test, renamed existing unmarshal to `_LegacyType` - `descriptor/v2/scheme_test.go` — **NEW**: 4 scheme resolution tests (all alias variants: versioned/unversioned × primary/legacy) - `descriptor/v2/helpers_test.go` — added `TestIsLocalBlob_RawLegacy`, updated Raw data JSON **Generated:** - Both `schemas/LocalBlob.schema.json` now have enum constraints: `LocalBlob/v1`, `localBlob/v1`, `LocalBlob` (deprecated), `localBlob` (deprecated) ## Follow-up PRs (after this merges + publishes) These are independent of each other and can merge in parallel: - #2324 — PR 2 (constructor): update YAML `type: localBlob` → `type: LocalBlob` in test fixtures - #2325 — PR 3 (cli): update 5× `"localBlob/v1"` → `"LocalBlob/v1"` string assertions - #2326 — PR 4 (kubernetes/controller): update 13× JSON type strings + use constants in CEL test Resolves the localBlob portion of ocm-project/ocm-project#962 --------- Signed-off-by: Gerald Morrison (SAP) <gerald.morrison@sap.com> Co-authored-by: Gerald Morrison (SAP) <gerald.morrison@sap.com>

## Summary - Rename all `localBlob/v1` references to `LocalBlob/v1` across website documentation (6 files) - Fix unversioned `localBlob` → `LocalBlob/v1` in signing concept doc - Add "Legacy alias: `localBlob`" note in the reference doc (consistent with `OCIImageLayer/v1` and `Helm/v1` style) - Remove unnecessary `Helm/v1` deprecation callout (no other type has one) ## What's unchanged (by design) - `--upload-as localBlob` CLI flag — this is an enum value, not a type name - `content_versioned/` legacy docs — intentionally frozen (same approach as #2244) ## Dependencies - **Depends on:** #2244 — this PR is rebased on top of `website-upper-camel-case-types` and should be merged after #2244 ## Related - Part of open-component-model/ocm-project#962 - Related code PRs: #2323 (descriptor), #2324 (constructor), #2325 (cli), #2326 (controller) --------- Signed-off-by: Gerald Morrison (SAP) <gerald.morrison@sap.com> Co-authored-by: Gerald Morrison (SAP) <gerald.morrison@sap.com>

## Summary - Rename all `localBlob/v1` references to `LocalBlob/v1` across website documentation (6 files) - Fix unversioned `localBlob` → `LocalBlob/v1` in signing concept doc - Add "Legacy alias: `localBlob`" note in the reference doc (consistent with `OCIImageLayer/v1` and `Helm/v1` style) - Remove unnecessary `Helm/v1` deprecation callout (no other type has one) ## What's unchanged (by design) - `--upload-as localBlob` CLI flag — this is an enum value, not a type name - `content_versioned/` legacy docs — intentionally frozen (same approach as #2244) ## Dependencies - **Depends on:** #2244 — this PR is rebased on top of `website-upper-camel-case-types` and should be merged after #2244 ## Related - Part of open-component-model/ocm-project#962 - Related code PRs: #2323 (descriptor), #2324 (constructor), #2325 (cli), #2326 (controller) --------- Signed-off-by: Gerald Morrison (SAP) <gerald.morrison@sap.com> Co-authored-by: Gerald Morrison (SAP) <gerald.morrison@sap.com> b742838

morri-son requested a review from a team as a code owner April 17, 2026 20:14

github-actions Bot added area/testing Testing related size/m Medium labels Apr 17, 2026

morri-son added the kind/chore chore, maintenance, etc. label Apr 17, 2026

morri-son changed the title ~~test(cli): update LocalBlob/v1 assertions to UpperCamelCase~~ chore: cli - update LocalBlob/v1 assertions to UpperCamelCase Apr 17, 2026

morri-son mentioned this pull request Apr 17, 2026

docs(website): rename types to UpperCamelCase #2327

Merged

morri-son linked an issue Apr 17, 2026 that may be closed by this pull request

align type notation for access and input open-component-model/ocm-project#962

Closed

7 tasks

morri-son changed the title ~~chore: cli - update LocalBlob/v1 assertions to UpperCamelCase~~ chore: cli - update LocalBlob/v1 assertions to UpperCamelCase [3/4] Apr 20, 2026

morri-son marked this pull request as draft April 24, 2026 08:08

test(cli): update LocalBlob/v1 assertions to UpperCamelCase

7c65060

Signed-off-by: Gerald Morrison (SAP) <gerald.morrison@sap.com>

morri-son force-pushed the feat/localblob-uppercamelcase-cli branch from 3996bbc to 7c65060 Compare April 24, 2026 15:07

github-actions Bot added size/s Small and removed size/m Medium labels Apr 24, 2026

morrison-sap added 2 commits April 27, 2026 18:17

chore(deps): bump descriptor/v2 to v2.0.3-alpha3

549ee74

Signed-off-by: Gerald Morrison (SAP) <gerald.morrison@sap.com>

chore(deps): bump cli/integration descriptor/v2 to v2.0.3-alpha3

313eb72

Signed-off-by: Gerald Morrison (SAP) <gerald.morrison@sap.com>

morri-son marked this pull request as ready for review April 28, 2026 07:48

morri-son enabled auto-merge (squash) April 28, 2026 07:49

Merge branch 'main' into feat/localblob-uppercamelcase-cli

0501265

frewilhelm approved these changes Apr 28, 2026

View reviewed changes

fabianburth approved these changes Apr 28, 2026

View reviewed changes

morri-son merged commit 0492157 into open-component-model:main Apr 28, 2026
31 of 34 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: cli - update LocalBlob/v1 assertions to UpperCamelCase [3/4]#2325

chore: cli - update LocalBlob/v1 assertions to UpperCamelCase [3/4]#2325
morri-son merged 4 commits into
open-component-model:mainfrom
morri-son:feat/localblob-uppercamelcase-cli

morri-son commented Apr 17, 2026 •

edited

Loading

Uh oh!

netlify Bot commented Apr 17, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented Apr 17, 2026 •

edited

Loading

Rate limit exceeded

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

morri-son commented Apr 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

Dependencies

Related PRs

Uh oh!

netlify Bot commented Apr 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for ocm-website ready!

Uh oh!

coderabbitai Bot commented Apr 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Rate limit exceeded

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

morri-son commented Apr 17, 2026 •

edited

Loading

netlify Bot commented Apr 17, 2026 •

edited

Loading

coderabbitai Bot commented Apr 17, 2026 •

edited

Loading