fix: parameterize getEnvDefinition tests using structured test cases#1277
Open
bijjalmayank wants to merge 18 commits into
Open
fix: parameterize getEnvDefinition tests using structured test cases#1277bijjalmayank wants to merge 18 commits into
bijjalmayank wants to merge 18 commits into
Conversation
…etEnvDefinition method (Netcracker#1252) * fix: shared creds (Netcracker#1240) * fix: update shared creds * fix: update shared creds * fix: add log' * fix: add additional dir * fix: make search case sensitive * chore: Update docker image tags and envgene_version for branch main [skip ci] * docs: add use cases for DevCI (Netcracker#1241) * docs: add Effective Set use cases * docs: add extra US * docs: add use-cases * docs: add Effective Set use cases * docs: add extra US * docs: add use-cases * docs: add SSL sert use-cases * docs: update ssl use cases * docs: update GSF Use Cases * docs: remove UC 2,3,4 * docs: add cred rotation UC * docs: add UC-SC-NEX-1 to Environment Template Artifact Download and remove system-certificate UC * docs: added auto-environment-name use cases * docs: fixed links * docs: fix markdown system-certificate use case * docs: fixed ssl use cases * fix: Handled inventory as default in getEnvDefinition method * fix: Remove 'inventory' from required fields in schema --------- Co-authored-by: dysmon <120464230+dysmon@users.noreply.github.com> Co-authored-by: qubership-actions[bot] <qubership-actions[bot]@users.noreply.github.com> Co-authored-by: ismglvd-hub <ismglv.d@gmail.com>
…r#1269) * fix: Handled inventory as default in getEnvDefinition method * fix: Remove 'inventory' from required fields in schema * fix: Added test_getEnvDefinition.py for business_helper.py * fix: Added test_business_helper.py for business_helper.py
bijjalmayank
requested review from
GlimmerCape,
chethana-shastry-p and
miyamuraga
as code owners
…etEnvDefinition method (Netcracker#1252) * fix: shared creds (Netcracker#1240) * fix: update shared creds * fix: update shared creds * fix: add log' * fix: add additional dir * fix: make search case sensitive * chore: Update docker image tags and envgene_version for branch main [skip ci] * docs: add use cases for DevCI (Netcracker#1241) * docs: add Effective Set use cases * docs: add extra US * docs: add use-cases * docs: add Effective Set use cases * docs: add extra US * docs: add use-cases * docs: add SSL sert use-cases * docs: update ssl use cases * docs: update GSF Use Cases * docs: remove UC 2,3,4 * docs: add cred rotation UC * docs: add UC-SC-NEX-1 to Environment Template Artifact Download and remove system-certificate UC * docs: added auto-environment-name use cases * docs: fixed links * docs: fix markdown system-certificate use case * docs: fixed ssl use cases * fix: Handled inventory as default in getEnvDefinition method * fix: Remove 'inventory' from required fields in schema --------- Co-authored-by: dysmon <120464230+dysmon@users.noreply.github.com> Co-authored-by: qubership-actions[bot] <qubership-actions[bot]@users.noreply.github.com> Co-authored-by: ismglvd-hub <ismglv.d@gmail.com>
…r#1269) * fix: Handled inventory as default in getEnvDefinition method * fix: Remove 'inventory' from required fields in schema * fix: Added test_getEnvDefinition.py for business_helper.py * fix: Added test_business_helper.py for business_helper.py
* fix: Handled inventory as default in getEnvDefinition method * fix: Remove 'inventory' from required fields in schema * fix: Added test_getEnvDefinition.py for business_helper.py * fix: Added test_business_helper.py for business_helper.py * fix: Added logging in test_business_helper.py * fix: Added logging in test_business_helper.py --------- Co-authored-by: KamalArya <42619309+KamalArya@users.noreply.github.com>
KamalArya
force-pushed
the
feature/inventory-optional
branch
from
b5f8a1c to
a877c88
Compare
KamalArya
force-pushed
the
feature/inventory-optional
branch
from
a877c88 to
05e112d
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request
Summary
Provide a concise description of what this pull request does and why it is needed.
Issue
Link to the issue(s) this PR addresses (e.g.,
Fixes #123orCloses #456). If no issue exists, explain why this change is necessary.Breaking Change?
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.
Tests / Evidence
Describe how the changes were verified, including:
Additional Notes
Include any extra information, such as:
Leave blank if not applicable.