kcl-lang / kcl-openapi

66%

main: 66%

LAST BUILD ON BRANCH fix/mixin_protocol_suffix

branch: fix/mixin_protocol_suffix

CHANGE BRANCH

x

Reset

Sync Branches

• fix/mixin_protocol_suffix
• add-map-to-kcl-value
• amyXia1994-patch-1
• bugfix-143-multiple-validations
• bump/cla
• chore-bump-regex-match-func
• chore-deprecated-cli
• chore-update-cla-version
• chore/docker-release-config
• chore/migrate-repo
• chore/update-release-ci
• ci-update-test-condition
• cleanup_ioutil
• dependabot/github_actions/actions/checkout-4
• dependabot/github_actions/actions/checkout-5
• dependabot/github_actions/actions/checkout-6
• dependabot/github_actions/actions/checkout-7
• dependabot/github_actions/actions/setup-go-5
• dependabot/github_actions/actions/setup-go-6
• dependabot/github_actions/actions/setup-python-5
• dependabot/github_actions/actions/setup-python-6
• dependabot/github_actions/docker/login-action-3
• dependabot/github_actions/goreleaser/goreleaser-action-6
• dependabot/go_modules/github.com/go-openapi/errors-0.22.0
• dependabot/go_modules/github.com/go-openapi/errors-0.22.1
• dependabot/go_modules/github.com/go-openapi/errors-0.22.2
• dependabot/go_modules/github.com/go-openapi/inflect-0.21.0
• dependabot/go_modules/github.com/go-openapi/inflect-0.21.2
• dependabot/go_modules/github.com/go-openapi/inflect-0.21.3
• dependabot/go_modules/github.com/go-openapi/strfmt-0.23.0
• dependabot/go_modules/github.com/jessevdk/go-flags-1.6.1
• dependabot/go_modules/github.com/kr/pretty-0.3.1
• dependabot/go_modules/github.com/stretchr/testify-1.10.0
• dependabot/go_modules/github.com/stretchr/testify-1.11.0
• dependabot/go_modules/github.com/stretchr/testify-1.11.1
• dependabot/go_modules/github.com/stretchr/testify-1.9.0
• doc/contributing
• doc/install-approaches
• doc/k8s-pkg-generate
• enhance-int-or-string-array-pattern
• enhance/support-v1-crd-versions
• feat-item-all-pattern
• feat-mangle-property-name-with-quote
• feat-protocol-mixin-suffix
• feat-use-raw-string-for-generated-schema
• feat-yaml-stream-format-input
• feat/preprocess-script
• fix
• fix-additional-property-pattern
• fix-all-of-pattern
• fix-default-value-with-x-order-generation
• fix-disable-auto-block-pr-option
• fix-multiple-pattern-in-items-and-additional-properties
• fix-nil-pointer-for-array-branch
• fix-optional-attr-min-and-max-validation-gen
• fix-schema-generation
• fix-split-documents
• fix-typo-in-documents
• fix-yaml-stream-format-input
• fix/conflict-import
• fix/docker-username-pw
• fix/empty-check-blocks
• fix/import-path-under-pkgroot
• fix/multiline-default-string
• fix/multiple-version-crdext-v1
• fix/pre-process
• fix/regex-check
• fix/wrong-crd-apiversion
• fix_link
• main
• mangle-system-module-import
• minor/add-version
• refs/tags/v0.5.0
• refs/tags/v0.5.1
• refs/tags/v0.5.2
• refs/tags/v0.5.3
• refs/tags/v0.5.4
• refs/tags/v0.5.5
• refs/tags/v0.6.0
• refs/tags/v0.6.1
• refs/tags/v0.6.2
• refs/tags/v0.6.3
• refs/tags/v0.6.4
• refs/tags/v0.6.5
• refs/tags/v0.6.6
• refs/tags/v0.7.0
• refs/tags/v0.8.0
• refs/tags/v0.9.0
• support-int-or-str
• workflow/sync-k8s-models

Build # 22738688129

Build Type

Pull #149

github

Committed by project0

Commit Message

fix: mixin/protocol schema suffix

fix feature and wrong behavior introduced by #146

* fix Mixin/Protocol schema suffix protection applying to property field names instead of generated schema type names: remove suffix logic from kclName (which is shared between property and schema naming) and add a protectedSchemaName helper applied only at inline schema creation in buildProperties

* add integration test case suffix_protected_mixin_protocol covering CRD properties whose names end with Mixin or Protocol

* fix CompareDir early-returning after first subdirectory, causing remaining files to be skipped in integration test comparison

Signed-off-by: Richard Hillmann <richard@flowers-software.de>

Pull Request Pull Request #149: fix: mixin/protocol schema suffix

Coverage Stats

14 of 16 new or added lines in 2 files covered. (87.5%)

2355 of 3553 relevant lines covered (66.28%)

0.75 hits per line