Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Correction in yaml value #17229

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Correction in yaml value #17229

wants to merge 1 commit into from

Conversation

amolpati30
Copy link
Contributor

@amolpati30 amolpati30 commented Dec 27, 2024 •
edited
Loading

Problem Statement

When we provide a string value as input in YAML, it is accepting the value. Ideally, it should not accept an invalid value. The value provided in the YAML is incorrect and should be rejected.

Solution

Updated the value for yaml parameter to ensure it works for the negative test case.

@amolpati30 amolpati30 added Easy Fix :) Easiest Fix to review and quick merge request. CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 6.16.z Introduced in or relating directly to Satellite 6.16 labels Dec 27, 2024
@amolpati30 amolpati30 requested a review from a team as a code owner December 27, 2024 06:51
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_classparameters.py::TestSmartClassParameters -k test_negative_update_parameter_type

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9714
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_classparameters.py::TestSmartClassParameters -k test_negative_update_parameter_type --external-logging
Test Result : ========= 7 passed, 22 deselected, 145 warnings in 1360.38s (0:22:40) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Dec 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@amolpati30 amolpati30

Labels
6.16.z Introduced in or relating directly to Satellite 6.16 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs CherryPick to previous branches Easy Fix :) Easiest Fix to review and quick merge request. PRT-Passed Indicates that latest PRT run is passed for the PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@amolpati30 @Satellite-QE