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

Add tests for Facts component #14933

Merged
merged 1 commit into from
Jun 25, 2024
Merged

Add tests for Facts component #14933

merged 1 commit into from
Jun 25, 2024

Conversation

shweta83
Copy link
Contributor

@shweta83 shweta83 commented May 2, 2024 •
edited
Loading

Problem Statement

Missing following test scenarios for Facts:

  • Update facts for client (CLI and UI)

Solution

Added test scenario for update client facts

Related Issues

@shweta83 shweta83 requested a review from a team as a code owner May 2, 2024 06:26
@shweta83 shweta83 marked this pull request as draft May 2, 2024 06:26
@shweta83 shweta83 added CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 QETestCoverage Issues and PRs relating to a Satellite bug labels May 2, 2024
@shweta83 shweta83 force-pushed the facts_update branch 2 times, most recently from 655378f to 4546c82 Compare May 20, 2024 06:53
@shweta83 shweta83 marked this pull request as ready for review May 20, 2024 06:54
ogajduse
ogajduse requested changes Jun 3, 2024
View reviewed changes
Copy link
Member

@ogajduse ogajduse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

co-reviewed with @rmynar.

tests/foreman/ui/test_fact.py Outdated Show resolved Hide resolved
tests/foreman/ui/test_fact.py Outdated Show resolved Hide resolved
tests/foreman/ui/test_fact.py Outdated Show resolved Hide resolved
tests/foreman/ui/test_fact.py Outdated Show resolved Hide resolved
tests/foreman/ui/test_fact.py Outdated Show resolved Hide resolved
tests/foreman/cli/test_fact.py Outdated Show resolved Hide resolved
tests/foreman/cli/test_fact.py Outdated Show resolved Hide resolved
tests/foreman/cli/test_fact.py Outdated Show resolved Hide resolved
tests/foreman/cli/test_fact.py Outdated Show resolved Hide resolved
tests/foreman/cli/test_fact.py Outdated Show resolved Hide resolved
rmynar
rmynar requested changes Jun 3, 2024
View reviewed changes
tests/foreman/ui/test_fact.py Outdated Show resolved Hide resolved
tests/foreman/cli/test_fact.py Outdated Show resolved Hide resolved
tests/foreman/cli/test_fact.py Outdated Show resolved Hide resolved
@shweta83 shweta83 force-pushed the facts_update branch 4 times, most recently from e6e2c4c to eff5c25 Compare June 18, 2024 13:17
@shweta83 shweta83 requested review from ogajduse and rmynar June 18, 2024 13:19
@shweta83
Copy link
Contributor Author

shweta83 commented Jun 18, 2024 •
edited
Loading

trigger: test-robottelo
pytest: tests/foreman/cli/test_fact.py -k 'test_positive_update_client_facts'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7425
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_fact.py -k test_positive_update_client_facts --external-logging
Test Result : =========== 1 passed, 8 deselected, 21 warnings in 992.44s (0:16:32) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jun 18, 2024
@shweta83
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_fact.py -k 'test_positive_update_client_facts'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7426
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_fact.py  -k test_positive_update_client_facts --external-logging
Test Result : ========== 1 passed, 8 deselected, 21 warnings in 1015.05s (0:16:55) ===========

@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Jun 19, 2024
@shweta83
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_fact.py -k test_positive_upload_host_facts
airgun: 1427

@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Jun 25, 2024
@shweta83
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_fact.py tests/foreman/ui/test_fact.py -k 'test_positive_update_client_facts_verify_imported_values or test_positive_upload_host_facts'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7511
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_fact.py tests/foreman/ui/test_fact.py -k test_positive_update_client_facts_verify_imported_values or test_positive_upload_host_facts --external-logging
Test Result : ===== 1 failed, 1 passed, 8 deselected, 55 warnings in 1135.59s (0:18:55) ======

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Jun 25, 2024
@shweta83
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_fact.py tests/foreman/ui/test_fact.py -k 'test_positive_update_client_facts_verify_imported_values or test_positive_upload_host_facts'
airgun: 1437

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7515
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_fact.py tests/foreman/ui/test_fact.py -k test_positive_update_client_facts_verify_imported_values or test_positive_upload_host_facts --external-logging
Test Result : ========== 2 passed, 8 deselected, 57 warnings in 1139.64s (0:18:59) ===========

@Satellite-QE Satellite-QE added PRT-Passed Indicates that latest PRT run is passed for the PR and removed PRT-Failed Indicates that latest PRT run is failed for the PR labels Jun 25, 2024
@ogajduse ogajduse merged commit dac321f into SatelliteQE:master Jun 25, 2024
13 checks passed
github-actions bot pushed a commit that referenced this pull request Jun 25, 2024
@shweta83 @web-flow
Add tests for Facts component (#14933)
1aabb7c 
(cherry picked from commit dac321f)
github-actions bot pushed a commit that referenced this pull request Jun 25, 2024
@shweta83 @web-flow
Add tests for Facts component (#14933)
f2d30ec 
(cherry picked from commit dac321f)
This was referenced Jun 25, 2024
Merged
Merged
github-actions bot pushed a commit that referenced this pull request Jun 25, 2024
@shweta83 @web-flow
Add tests for Facts component (#14933)
3ce75d5 
(cherry picked from commit dac321f)
This was referenced Jun 25, 2024
Merged
Merged
Merged
Merged
shweta83 added a commit to shweta83/robottelo that referenced this pull request Jun 26, 2024
@shweta83
Add tests for Facts component (SatelliteQE#14933)
1e69c94
shweta83 added a commit to shweta83/robottelo that referenced this pull request Jun 26, 2024
@shweta83
Add tests for Facts component (SatelliteQE#14933)
fc27f70 
(cherry picked from commit dac321f)
shweta83 added a commit to shweta83/robottelo that referenced this pull request Jun 26, 2024
@shweta83
Add tests for Facts component (SatelliteQE#14933)
8d26866 
(cherry picked from commit dac321f)
shweta83 added a commit that referenced this pull request Jun 26, 2024
@shweta83
Add tests for Facts component (#14933)
9840f6f 
(cherry picked from commit dac321f)
shweta83 added a commit that referenced this pull request Aug 20, 2024
@shweta83
Add tests for Facts component (#14933)
2ab56c2 
(cherry picked from commit dac321f)
shweta83 added a commit that referenced this pull request Aug 20, 2024
@shweta83
Add tests for Facts component (#14933)
bd4c9e5 
(cherry picked from commit dac321f)
shweta83 added a commit that referenced this pull request Aug 20, 2024
@shweta83
Add tests for Facts component (#14933)
2e93adf 
(cherry picked from commit dac321f)
shweta83 added a commit that referenced this pull request Aug 21, 2024
@shweta83
Add tests for Facts component (#14933)
081364f 
(cherry picked from commit dac321f)
shweta83 added a commit that referenced this pull request Aug 21, 2024
@shweta83
Add tests for Facts component (#14933)
1b18e1e 
(cherry picked from commit dac321f)
Gauravtalreja1 pushed a commit that referenced this pull request Aug 21, 2024
@Satellite-QE @shweta83
[6.13.z] Add tests for Facts component (#15509)
444d969 
Add tests for Facts component (#14933)

(cherry picked from commit dac321f)

Co-authored-by: Shweta Singh <[email protected]>
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@shweta83 @jyejare
Add tests for Facts component (SatelliteQE#14933)
eca87a7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vijaysawant vijaysawant vijaysawant requested changes

@rmynar rmynar rmynar approved these changes

@vsedmik vsedmik vsedmik approved these changes

@ogajduse ogajduse ogajduse approved these changes

@Gauravtalreja1 Gauravtalreja1 Awaiting requested review from Gauravtalreja1

Assignees

@shweta83 shweta83

Labels
6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR QETestCoverage Issues and PRs relating to a Satellite bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@shweta83 @Satellite-QE @ogajduse @Gauravtalreja1 @vijaysawant @vsedmik @rmynar