delete subscription manifest so it can re imported again

[vijaysawant]

Problem Statement

• test_sca_end_to_end and test_positive_expired_SCA_cert_handling tests are failing imported manifest issue.
• Error: This subscription management application has already been imported by another owner
• I guess test_positive_create test uses mdoule_manifest_org fixture and it is creating issue while importing manifest

Solution

Delete test_positive_create test, as it uses module_sca_manifest fixture and which is used inside any *_sca_manifest_org fixture that executed on so many places.

Local test execution

============================= test session starts ==============================
collecting ... 2024-12-10 22:29:23 - robottelo.collection - INFO - Processing test items to add testimony token markers
collected 11 items / 8 deselected / 3 selected

tests/foreman/api/test_subscription.py::test_positive_create 
tests/foreman/api/test_subscription.py::test_sca_end_to_end[rhel7-ipv4] 
tests/foreman/api/test_subscription.py::test_positive_expired_SCA_cert_handling[rhel7-ipv4] 

=========== 3 passed, 8 deselected, 58 warnings in 697.74s (0:11:37) ===========

PRT test Cases example

trigger: test-robottelo
pytest: tests/foreman/api/test_subscription.py

[vijaysawant]

trigger: test-robottelo
pytest: tests/foreman/api/test_subscription.py

[Satellite-QE]

PRT Result

Build Number: 9580
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_subscription.py --external-logging
Test Result : ===== 1 failed, 9 passed, 1 deselected, 182 warnings in 1293.27s (0:21:33) =====

[vijaysawant]

I could see one test failed test_positive_async_endpoint_for_manifest_refresh, There is no relation with the changes made in this PR, assertion error tells that after manifest refresh expected message did not present in the production log, and hence this test failed.
Error:

AssertionError: assert 'Sending GET request to upstream Candlepin' in 'stdout:\n\nstderr:\n\nstatus: 1'
 +  where 'stdout:\n\nstderr:\n\nstatus: 1' = str(stdout:\n\nstderr:\n\nstatus: 1)

[Satellite-QE]

PRT Result

Build Number: 9584
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_subscription.py --external-logging
Test Result : ===== 1 failed, 9 passed, 1 deselected, 191 warnings in 1393.59s (0:23:13) =====

[vijaysawant]

trigger: test-robottelo
pytest: tests/foreman/api/test_subscription.py -k 'test_sca_end_to_end or test_positive_expired_SCA_cert_handling or test_positive_create'

[vijaysawant]

Local test execution result:

(uv-venv-master) visawant@fedora:~/Desktop/workspace/robottelo$ pytest tests/foreman/api/test_subscription.py -k 'test_sca_end_to_end or test_positive_expired_SCA_cert_handling or test_positive_create'
============================================================================ test session starts ============================================================================
platform linux -- Python 3.11.1, pytest-8.3.4, pluggy-1.5.0
Mandatory Requirements Mismatch: productmd==1.43 apypie==0.5.0 pytest-reportportal==5.4.7 wrapanapi python-box==7.3.0
Optional Requirements Mismatch: ruff==0.8.2 sphinx-autoapi==3.4.0 pre-commit==4.0.1 redis==5.2.1
To update requirements, run the pytest with '--update-required-reqs' OR '--update-all-reqs' option.
shared_function enabled - OFF - scope:  - storage: file
rootdir: /home/visawant/Desktop/workspace/robottelo
configfile: pyproject.toml
plugins: ibutsu-2.2.4, mock-3.14.0, services-2.2.1, cov-6.0.0, reportportal-5.4.5, fixturecollection-0.1.2, xdist-3.6.1, order-1.3.0
collected 11 items / 7 deselected / 4 selected                                                                                                                              

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

========================================================= 4 passed, 7 deselected, 78 warnings in 611.74s (0:10:11) ==========================================================

[Satellite-QE]

PRT Result

Build Number: 9586
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_subscription.py -k test_sca_end_to_end or test_positive_expired_SCA_cert_handling or test_positive_create --external-logging
Test Result : ========== 4 passed, 7 deselected, 117 warnings in 943.12s (0:15:43) ===========

[vsedmik]

I tend to remove the test_positive_create completely.

In only runs sat.upload_manifest(org.id, module_sca_manifest.content) which is done in any *_sca_manifest_org fixture and executed on so many places. Half of the test suite would fail if this was broken.

[vijaysawant]

I tend to remove the test_positive_create completely.

In only runs sat.upload_manifest(org.id, module_sca_manifest.content) which is done in any *_sca_manifest_org fixture and executed on so many places. Half of the test suite would fail if this was broken.

Agree, same functionality has tested in fixture *_sca_manifest_org
Let me remove this test then.

[vijaysawant]

trigger: test-robottelo
pytest: tests/foreman/api/test_subscription.py -k 'test_sca_end_to_end or test_positive_expired_SCA_cert_handling'

[Satellite-QE]

PRT Result

Build Number: 9587
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_subscription.py -k test_sca_end_to_end or test_positive_expired_SCA_cert_handling --external-logging
Test Result : =========== 2 passed, 8 deselected, 54 warnings in 821.51s (0:13:41) ===========