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.

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

initial testing for usage reports #17765

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

Conversation

pondrejk
Copy link
Contributor

@pondrejk pondrejk commented Mar 5, 2025

Problem Statement

SAT-31592

Solution

I added a basic comparison test to check that high level report item keys match with the expected set. For the expected set I use a report output from a fresh satellite.

More tests and fixtures will likely follow as the feature crystalizes. I used Hammer component mainly for convenience, it is not yet clear if this will become a new component

Related Issues

@pondrejk pondrejk added No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master Do Not Merge labels Mar 5, 2025
@pondrejk pondrejk self-assigned this Mar 5, 2025
@pondrejk pondrejk requested a review from a team as a code owner March 5, 2025 14:12
@pondrejk pondrejk marked this pull request as draft March 5, 2025 14:12
@pondrejk pondrejk force-pushed the data-poc branch 3 times, most recently from a85f793 to 9e7b76b Compare March 6, 2025 15:42
@pondrejk pondrejk marked this pull request as ready for review March 6, 2025 15:48
@pondrejk
Copy link
Contributor Author

pondrejk commented Mar 6, 2025

trigger: test-robottelo
pytest: tests/foreman/cli/test_usage_report.py

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10432
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_usage_report.py --external-logging
Test Result : ================== 1 passed, 9 warnings in 829.94s (0:13:49) ===================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Mar 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR Stream Introduced in or relating directly to Satellite Stream/Master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants