Skip to content

Commit

Permalink
remove the unused testimony tokens & none used for reporting (#13479)
Browse files Browse the repository at this point in the history
Bump flake8 from 6.1.0 to 7.0.0

Bumps [flake8](https://github.com/pycqa/flake8) from 6.1.0 to 7.0.0.
- [Commits](PyCQA/flake8@6.1.0...7.0.0)

---
updated-dependencies:
- dependency-name: flake8
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
  • Loading branch information
omkarkhatavkar and dependabot[bot] authored Jan 8, 2024
1 parent ffadf54 commit 14cdcc2
Show file tree
Hide file tree
Showing 316 changed files with 360 additions and 4,133 deletions.
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 i 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 @@ -260,7 +255,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

0 comments on commit 14cdcc2

Please sign in to comment.