Skip to content

fix(common-test): update configmap validation test for binaryData support#45007

Closed
Copilot wants to merge 4 commits intocommon2026from
copilot/sub-pr-41017-another-one
Closed

fix(common-test): update configmap validation test for binaryData support#45007
Copilot wants to merge 4 commits intocommon2026from
copilot/sub-pr-41017-another-one

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 15, 2026

Configmap validation test was failing due to updated error message after binaryData support was added to ConfigMaps.

Changes

  • Updated configmap validation test error expectation from Expected non-empty [data] to Expected non-empty [data] or [binaryData]
  • Bumped common-test chart version 1.0.0 → 1.0.1
  • Updated 31 test files with hardcoded chart version references to match new version

The validation template now allows either data or binaryData fields, but the test was checking for the old single-field error message.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 3 commits February 15, 2026 13:38
…nclude binaryData

Co-authored-by: PrivatePuffin <7613738+PrivatePuffin@users.noreply.github.com>
Co-authored-by: PrivatePuffin <7613738+PrivatePuffin@users.noreply.github.com>
Co-authored-by: PrivatePuffin <7613738+PrivatePuffin@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove env-list support and improve network policy fix(common-test): update configmap validation test for binaryData support Feb 15, 2026
Copilot AI requested a review from Crow-Control February 15, 2026 13:53
@truecharts-admin
Copy link
Copy Markdown
Contributor

This PR is locked to prevent necro-posting on closed PRs. Please create a issue or contact staff on discord if you want to further discuss this

@trueforge-org trueforge-org locked as resolved and limited conversation to collaborators Feb 23, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants