Skip to content

Conversation

@GeethaGadde99
Copy link
Contributor

@GeethaGadde99 GeethaGadde99 commented Feb 4, 2026

Pull Request

Summary

Enhance Test data for Calculator CLI. For below cases
Parameter Type Preservation in Macro Resolution
1.Simple Type Resolution
2.Complex Structure Resolution
Complex nested parameter structure case is not covered in code previously. Added fix for it.

Issue

Link to the issue(s) this PR addresses (e.g., Fixes #123 or Closes #456). If no issue exists, explain why this change is necessary.

Breaking Change?

  • Yes
  • No

If yes, describe the breaking change and its impact (e.g., API changes, behavior changes, or required updates for users).

Scope / Project

Specify the component, module, or project area affected by this change (e.g., docs, actions, workflows).

Implementation Notes

Provide details on how the change was implemented, including any technical considerations, trade-offs, or notable design decisions. Leave blank if not applicable.
1.Added a pure-reference shortcut in processValue to detect values of the form ${param} and resolve them directly from Binding before template rendering.
This preserves the original object type (e.g., Map, List) and prevents unintended stringification by JinJava/Groovy.
2.added tescases as per this doc
https://github.com/Netcracker/qubership-envgene/blob/main/docs/use-cases/calculator-cli.md

Tests / Evidence

Describe how the changes were verified, including:
Tested with instance pipeline

  • Tests added or updated (e.g., unit, integration, end-to-end)
  • Manual testing steps or results
  • Screenshots, logs, or other evidence (if applicable)

Additional Notes

Include any extra information, such as:

  • Dependencies introduced
  • Future work or follow-up tasks
  • Reviewer instructions or context
  • References to related PRs or discussions

Leave blank if not applicable.

@GeethaGadde99 GeethaGadde99 force-pushed the feature/enhance-test-data-for-calculator-cli branch from 537a774 to 82ce9cd Compare February 9, 2026 10:49
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.

1 participant