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

[6.15.z] - remove the unused testimony tokens & none used for reporting (#13479) #13667

Merged
merged 1 commit into from
Jan 10, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
5 changes: 5 additions & 0 deletions scripts/polarion-test-case-upload.sh
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,11 @@ DEFAULT_APPROVERS_VALUE = '${POLARION_USERNAME}:approved'
DEFAULT_STATUS_VALUE = 'approved'
DEFAULT_SUBTYPE2_VALUE = '-'
TESTCASE_CUSTOM_FIELDS = default_config.TESTCASE_CUSTOM_FIELDS + ('customerscenario',) + ('team',) + ('markers',)
# converting TESTCASE_CUSTOM_FIELDS to list for removing tokens since the tokens are defined as defaults/mandatory in betelgeuse
TESTCASE_CUSTOM_FIELDS = list(TESTCASE_CUSTOM_FIELDS)
REMOVE_TOKEN_LIST = ['caselevel', 'upstream', 'testtype']
TESTCASE_CUSTOM_FIELDS = tuple([token for token in TESTCASE_CUSTOM_FIELDS if token not in REMOVE_TOKEN_LIST])

REQUIREMENT_CUSTOM_FIELDS = default_config.REQUIREMENT_CUSTOM_FIELDS + ('team',)
TRANSFORM_CUSTOMERSCENARIO_VALUE = default_config._transform_to_lower
DEFAULT_CUSTOMERSCENARIO_VALUE = 'false'
Expand Down
23 changes: 0 additions & 23 deletions testimony.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -121,15 +121,6 @@ CaseImportance:
- Low
required: true
type: choice
CaseLevel:
casesensitive: true
choices:
- Component
- Integration
- System
- Acceptance
required: true
type: choice
CasePosNeg:
casesensitive: false
choices:
Expand Down Expand Up @@ -157,20 +148,6 @@ SubComponent:
type: choice
Subtype1: {}
Teardown: {}
TestType:
casesensitive: false
choices:
- Functional
- Structural
required: true
type: choice
Upstream:
casesensitive: false
choices:
- 'Yes'
- 'No'
required: true
type: choice
Parametrized:
casesensitive: false
choices:
Expand Down
7 changes: 0 additions & 7 deletions tests/foreman/api/test_acs.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,17 +4,12 @@

:CaseAutomation: Automated

:CaseLevel: Acceptance

:CaseComponent: AlternateContentSources

:Team: Phoenix-content

:TestType: Functional

:CaseImportance: High

:Upstream: No
"""
from fauxfactory import gen_string
import pytest
Expand Down Expand Up @@ -50,7 +45,6 @@ def test_positive_CRUD_all_types(
3. ACS can be updated and read with new name.
4. ACS can be refreshed.
5. ACS can be deleted.

"""
if 'rhui' in request.node.name and 'file' in request.node.name:
pytest.skip('unsupported parametrize combination')
Expand Down Expand Up @@ -127,7 +121,6 @@ def test_positive_run_bulk_actions(module_target_sat, module_yum_repo):
:expectedresults:
1. All ACSes can be refreshed via bulk action.
2. Only the proper ACSes are deleted on bulk destroy.

"""
acs_ids = []
for _ in range(3):
Expand Down
18 changes: 1 addition & 17 deletions tests/foreman/api/test_activationkey.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,17 +4,13 @@

:CaseAutomation: Automated

:CaseLevel: Component

:CaseComponent: ActivationKeys

:team: Phoenix-subscriptions

:TestType: Functional

:CaseImportance: High

:Upstream: No

"""
import http

Expand Down Expand Up @@ -278,8 +274,6 @@ def test_positive_get_releases_status_code(target_sat):

:expectedresults: HTTP 200 is returned with an ``application/json``
content-type

:CaseLevel: Integration
"""
act_key = target_sat.api.ActivationKey().create()
path = act_key.path('releases')
Expand All @@ -296,8 +290,6 @@ def test_positive_get_releases_content(target_sat):
:id: 2fec3d71-33e9-40e5-b934-90b03afc26a1

:expectedresults: A list of results is returned.

:CaseLevel: Integration
"""
act_key = target_sat.api.ActivationKey().create()
response = client.get(act_key.path('releases'), auth=get_credentials(), verify=False).json()
Expand All @@ -319,8 +311,6 @@ def test_positive_add_host_collections(module_org, module_target_sat):
collections and reading that activation key, the correct host
collections are listed.

:CaseLevel: Integration

:CaseImportance: Critical
"""
# An activation key has no host collections by default.
Expand Down Expand Up @@ -358,8 +348,6 @@ def test_positive_remove_host_collection(module_org, module_target_sat):
3. Disassociating host collection from the activation key actually
removes it from the list

:CaseLevel: Integration

:CaseImportance: Critical
"""
# An activation key has no host collections by default.
Expand Down Expand Up @@ -451,8 +439,6 @@ def test_positive_fetch_product_content(
:expectedresults: Both Red Hat and custom product subscriptions are
assigned as Activation Key's product content

:CaseLevel: Integration

:CaseImportance: Critical
"""
module_target_sat.upload_manifest(module_org.id, session_entitlement_manifest.content)
Expand Down Expand Up @@ -506,8 +492,6 @@ def test_positive_add_future_subscription():

:CaseAutomation: NotAutomated

:CaseLevel: Integration

:CaseImportance: Critical
"""

Expand Down
13 changes: 4 additions & 9 deletions tests/foreman/api/test_ansible.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,17 +4,12 @@

:CaseAutomation: Automated

:CaseLevel: Acceptance

:CaseComponent: Ansible

:Team: Rocket

:TestType: Functional

:CaseImportance: High

:Upstream: No
"""
from fauxfactory import gen_string
import pytest
Expand All @@ -33,7 +28,7 @@ def test_fetch_and_sync_ansible_playbooks(target_sat):

:customerscenario: true

:Steps:
:steps:

1. Install ansible collection with playbooks.
2. Try to fetch the playbooks via api.
Expand Down Expand Up @@ -79,7 +74,7 @@ def test_positive_ansible_job_on_host(

:id: c8dcdc54-cb98-4b24-bff9-049a6cc36acb

:Steps:
:steps:
1. Register a content host with satellite
2. Import a role into satellite
3. Assign that role to a host
Expand Down Expand Up @@ -220,7 +215,7 @@ def test_add_and_remove_ansible_role_hostgroup(target_sat):

:id: 7672cf86-fa31-11ed-855a-0fd307d2d66b

:Steps:
:steps:
1. Create a hostgroup
2. Sync few ansible roles
3. Assign a few ansible roles with the host group
Expand Down Expand Up @@ -264,7 +259,7 @@ def test_add_and_remove_ansible_role_hostgroup(target_sat):
@pytest.fixture
def filtered_user(target_sat, module_org, module_location):
"""
:Steps:
:steps:
1. Create a role with a host view filtered
2. Create a user with that role
3. Setup a host
Expand Down
5 changes: 0 additions & 5 deletions tests/foreman/api/test_architecture.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,17 +4,12 @@

:CaseAutomation: Automated

:CaseLevel: Acceptance

:CaseComponent: Hosts

:Team: Endeavour

:TestType: Functional

:CaseImportance: High

:Upstream: No
"""
from fauxfactory import gen_choice
import pytest
Expand Down
5 changes: 0 additions & 5 deletions tests/foreman/api/test_audit.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,17 +4,12 @@

:CaseAutomation: Automated

:CaseLevel: Acceptance

:CaseComponent: AuditLog

:Team: Endeavour

:TestType: Functional

:CaseImportance: High

:Upstream: No
"""
import pytest

Expand Down
31 changes: 13 additions & 18 deletions tests/foreman/api/test_bookmarks.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,17 +4,12 @@

:CaseAutomation: Automated

:CaseLevel: Acceptance

:CaseComponent: Search

:Team: Endeavour

:TestType: Functional

:CaseImportance: High

:Upstream: No
"""
import random

Expand All @@ -38,7 +33,7 @@ def test_positive_create_with_name(controller, target_sat):

:parametrized: yes

:Steps:
:steps:

1. Create a bookmark with a random name and valid controller.
2. List the bookmarks.
Expand All @@ -64,7 +59,7 @@ def test_positive_create_with_query(controller, target_sat):

:parametrized: yes

:Steps:
:steps:

1. Create a bookmark with a random query and valid controller.
2. List the bookmarks.
Expand All @@ -91,7 +86,7 @@ def test_positive_create_public(controller, public, target_sat):

:parametrized: yes

:Steps:
:steps:

1. Create a bookmark with a valid controller and public attribute True or False.
2. List the bookmarks.
Expand All @@ -116,7 +111,7 @@ def test_negative_create_with_invalid_name(controller, target_sat):

:parametrized: yes

:Steps:
:steps:

1. Attempt to create a bookmark with an invalid name.
2. List the bookmarks.
Expand Down Expand Up @@ -144,7 +139,7 @@ def test_negative_create_empty_query(controller, target_sat):

:parametrized: yes

:Steps:
:steps:

1. Attempt to create a bookmark with a random name, valid controller, and empty query.
2. List the bookmarks.
Expand Down Expand Up @@ -172,7 +167,7 @@ def test_negative_create_same_name(controller, target_sat):

:parametrized: yes

:Steps:
:steps:

1. Create a bookmark with a random name and valid controller.
2. Attempt to create a second bookmark, using the same name as the previous bookmark.
Expand Down Expand Up @@ -202,7 +197,7 @@ def test_negative_create_null_public(controller, target_sat):

:parametrized: yes

:Steps:
:steps:

1. Attempt to create a bookmark with a random name and valid controller, with public
attribute set to None.
Expand Down Expand Up @@ -233,7 +228,7 @@ def test_positive_update_name(controller, target_sat):

:parametrized: yes

:Steps:
:steps:

1. Create a bookmark with a valid controller.
2. Update the bookmark with a random name.
Expand All @@ -260,7 +255,7 @@ def test_negative_update_same_name(controller, target_sat):

:parametrized: yes

:Steps:
:steps:

1. Create a bookmark with a random name and valid controller.
2. Create a second bookmark for the same controller.
Expand Down Expand Up @@ -291,7 +286,7 @@ def test_negative_update_invalid_name(controller, target_sat):

:parametrized: yes

:Steps:
:steps:

1. Create a bookmark with valid controller.
2. Attempt to update the bookmark with an invalid name.
Expand Down Expand Up @@ -320,7 +315,7 @@ def test_positive_update_query(controller, target_sat):

:parametrized: yes

:Steps:
:steps:

1. Create a bookmark with a valid controller.
2. Update the bookmark's query with a random value.
Expand All @@ -347,7 +342,7 @@ def test_negative_update_empty_query(controller, target_sat):

:parametrized: yes

:Steps:
:steps:

1. Create a bookmark for a valid controller.
2. Attempt to update the query to an empty value.
Expand Down Expand Up @@ -376,7 +371,7 @@ def test_positive_update_public(controller, public, target_sat):

:parametrized: yes

:Steps:
:steps:

1. Create a bookmark for a valid controller.
2. Update the bookmark's public attribute.
Expand Down
Loading