Skip to content

Conversation

@vsedmik
Copy link
Contributor

@vsedmik vsedmik commented Jun 25, 2025

Problem Statement

We are extending the Capsule Flatpak endpoint by the one filtered by Katello. We already have a test for the pulpcore endpoint, now the time has come to extend it for the newly added endpoint too.

Solution

This PR parametrizes the test to check both endpoints.
Another test case will be added for the CV/LCE filtering on the new endpoint once it's implemented.

Requires

Katello/smart_proxy_container_gateway#54
theforeman/puppet-foreman_proxy_content#522

Related Issues

https://issues.redhat.com/browse/SAT-30899

PRT test Cases example

We are not able to use packit/PRT since the patch needs to be applied on the Capsule side.
However, locally it passed:

$ pytest tests/foreman/api/test_capsulecontent.py -k test_flatpak_endpoint
========================== test session starts ==========================
collected 31 items / 29 deselected / 2 selected

tests/foreman/api/test_capsulecontent.py ..                       [100%]

============= 2 passed, 29 deselected, 5 warnings in 42.68s =============

@vsedmik vsedmik self-assigned this Jun 25, 2025
@vsedmik vsedmik requested a review from a team June 25, 2025 10:07
@vsedmik vsedmik added No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Jun 25, 2025
Copy link
Contributor

@LadislavVasina1 LadislavVasina1 left a comment

Choose a reason for hiding this comment

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

ACK

@LadislavVasina1 LadislavVasina1 requested a review from a team June 25, 2025 10:12
@vsedmik
Copy link
Contributor Author

vsedmik commented Jul 21, 2025

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

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 12087
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_capsulecontent.py -k test_flatpak_endpoint --external-logging
Test Result: ========== 1 passed, 29 deselected, 9 warnings in 1521.09s (0:25:21) ===========

@vsedmik vsedmik merged commit b1906c6 into SatelliteQE:master Jul 22, 2025
9 checks passed
@vsedmik vsedmik deleted the extend-capsule-flatpak-endpoint branch July 22, 2025 07:54
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 Stream Introduced in or relating directly to Satellite Stream/Master

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants