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 API tests for capsule content counts #13068

Merged
merged 1 commit into from
Nov 14, 2023
Merged

Add API tests for capsule content counts #13068

merged 1 commit into from
Nov 14, 2023

Conversation

vsedmik
Copy link
Contributor

@vsedmik vsedmik commented Nov 9, 2023

Problem Statement

Satellite 6.15 will introduce new a feature to provide content counts synced to a Capsule and it needs to be tested.

Solution

This PR adds test cases to verify capsule content counts via API for unfiltered and filtered mixed-content CVs and a few additions to support it.

@vsedmik vsedmik added API Issues and PRs involving the API No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Nov 9, 2023
@vsedmik vsedmik self-assigned this Nov 9, 2023
@vsedmik vsedmik requested review from a team as code owners November 9, 2023 14:59
@vsedmik
Copy link
Contributor Author

vsedmik commented Nov 9, 2023

trigger: test-robottelo
pytest: tests/foreman/api/test_capsulecontent.py -k test_positive_content_counts_for_mixed_cv

@vsedmik vsedmik force-pushed the caps-api-countent-counts branch 2 times, most recently from 3e8f565 to 0a7e2fb Compare November 9, 2023 18:20
@vsedmik
Copy link
Contributor Author

vsedmik commented Nov 9, 2023

trigger: test-robottelo
pytest: tests/foreman/api/test_capsulecontent.py -k test_positive_content_counts_for_mixed_cv

pondrejk
pondrejk approved these changes Nov 13, 2023
View reviewed changes
Copy link
Contributor

@pondrejk pondrejk left a comment

Choose a reason for hiding this comment

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

provided test_positive_content_counts_blank_update passes too, local test would suffice

Griffin-Sullivan
Griffin-Sullivan approved these changes Nov 13, 2023
View reviewed changes
Copy link
Contributor

@Griffin-Sullivan Griffin-Sullivan left a comment

Choose a reason for hiding this comment

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

Ack pending the local test

robottelo/constants/__init__.py Show resolved Hide resolved
@vsedmik
Copy link
Contributor Author

vsedmik commented Nov 13, 2023

trigger: test-robottelo
pytest: tests/foreman/api/test_capsulecontent.py -k test_positive_content_counts_blank_update

synkd
synkd approved these changes Nov 13, 2023
View reviewed changes
robottelo/host_helpers/repository_mixins.py Outdated Show resolved Hide resolved
@vsedmik vsedmik force-pushed the caps-api-countent-counts branch from 0a7e2fb to 9111114 Compare November 13, 2023 17:00
@vsedmik
Add API tests for capsule content counts
7acf557 
This PR adds test cases to verify capsule content counts via API
for unfiltered and filtered mixed-content CVs and few additions
to support it.
@vsedmik vsedmik force-pushed the caps-api-countent-counts branch from 9111114 to 7acf557 Compare November 14, 2023 08:15
@vsedmik
Copy link
Contributor Author

vsedmik commented Nov 14, 2023 •
edited
Loading

Seems that in the latest snap we get empty list after contents update, instead of None.
Updated, rebased, PRT rerun and PASSED (4636)

@Griffin-Sullivan Griffin-Sullivan merged commit adf6b7e into SatelliteQE:master Nov 14, 2023
5 checks passed
This was referenced Jan 19, 2024
Closed
Closed
Closed
@vsedmik vsedmik mentioned this pull request Feb 2, 2024
Merged
@Satellite-QE Satellite-QE mentioned this pull request Feb 5, 2024
Merged
@Gauravtalreja1 Gauravtalreja1 mentioned this pull request Mar 27, 2024
Merged
shweta83 pushed a commit to shweta83/robottelo that referenced this pull request Apr 10, 2024
@vsedmik @shweta83
Add API tests for capsule content counts (SatelliteQE#13068)
53df2d7 
This PR adds test cases to verify capsule content counts via API
for unfiltered and filtered mixed-content CVs and few additions
to support it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Griffin-Sullivan Griffin-Sullivan Griffin-Sullivan approved these changes

@synkd synkd synkd approved these changes

@pondrejk pondrejk pondrejk approved these changes

Assignees

@vsedmik vsedmik

Labels
API Issues and PRs involving the API No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@vsedmik @pondrejk @synkd @Griffin-Sullivan