Skip to content

Commit

Permalink
[6.13.z] deleted tc's by considering recommendation present in reposi…
Browse files Browse the repository at this point in the history 
…tories component audit (#14152)

deleted tc's by considering recommendation present in repositories component audit
  • Loading branch information
@vijaysawant
vijaysawant authored Feb 21, 2024
1 parent e147840 commit 96b7d33
Show file tree
Hide file tree
Showing 5 changed files with 0 additions and 660 deletions.
16 changes: 0 additions & 16 deletions tests/foreman/api/test_docker.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -191,22 +191,6 @@ def test_positive_create_repos_using_multiple_products(self, module_org, module_
product = product.read()
assert repo.id in [repo_.id for repo_ in product.repository]

@pytest.mark.tier1
def test_positive_sync(self, module_product, module_target_sat):
"""Create and sync a Docker-type repository
:id: 80fbcd84-1c6f-444f-a44e-7d2738a0cba2
:expectedresults: A repository is created with a Docker repository and
it is synchronized.
:CaseImportance: Critical
"""
repo = _create_repository(module_target_sat, module_product)
repo.sync(timeout=600)
repo = repo.read()
assert repo.content_counts['docker_manifest'] >= 1

@pytest.mark.tier1
@pytest.mark.parametrize('new_name', **parametrized(valid_docker_repository_names()))
def test_positive_update_name(self, repo, new_name):
Expand Down
180 changes: 0 additions & 180 deletions tests/foreman/api/test_repository.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -689,20 +689,6 @@ def test_positive_update_gpg(self, module_org, module_product, module_target_sat
repo = repo.update(['gpg_key'])
assert repo.gpg_key.id == gpg_key_2.id

@pytest.mark.tier2
def test_positive_update_contents(self, repo):
"""Create a repository and upload RPM contents.
:id: 8faa64f9-b620-4c0a-8c80-801e8e6436f1
:expectedresults: The repository's contents include one RPM.
"""
# Upload RPM content.
repo.upload_content(files={'content': DataFile.RPM_TO_UPLOAD.read_bytes()})
# Verify the repository's contents.
assert repo.read().content_counts['rpm'] == 1

@pytest.mark.tier1
@pytest.mark.upgrade
def test_positive_upload_delete_srpm(self, repo, target_sat):
Expand Down Expand Up @@ -1544,37 +1530,6 @@ def test_positive_create(self, repo_options, repo):
for k in 'name', 'docker_upstream_name', 'content_type':
assert getattr(repo, k) == repo_options[k]

@pytest.mark.tier1
@pytest.mark.parametrize(
'repo_options',
**datafactory.parametrized(
{
constants.CONTAINER_UPSTREAM_NAME: {
'content_type': 'docker',
'docker_upstream_name': constants.CONTAINER_UPSTREAM_NAME,
'name': gen_string('alphanumeric', 10),
'url': constants.CONTAINER_REGISTRY_HUB,
}
}
),
indirect=True,
)
def test_positive_synchronize(self, repo):
"""Create and sync a Docker-type repository
:id: 27653663-e5a7-4700-a3c1-f6eab6468adf
:parametrized: yes
:expectedresults: A repository is created with a Docker repository and
it is synchronized.
:CaseImportance: Critical
"""
# TODO: add timeout support to sync(). This repo needs more than the default 300 seconds.
repo.sync()
assert repo.read().content_counts['docker_manifest'] >= 1

@pytest.mark.tier3
@pytest.mark.parametrize(
'repo_options',
Expand Down Expand Up @@ -1715,44 +1670,6 @@ def test_positive_synchronize_private_registry(self, repo):
repo.sync()
assert repo.read().content_counts['docker_manifest'] >= 1

@pytest.mark.tier2
@pytest.mark.parametrize(
'repo_options',
**datafactory.parametrized(
{
'private_registry': {
'content_type': 'docker',
'docker_upstream_name': settings.docker.private_registry_name,
'name': gen_string('alpha'),
'upstream_username': settings.docker.private_registry_username,
'upstream_password': 'ThisIsaWrongPassword',
'url': settings.docker.private_registry_url,
}
}
),
indirect=True,
)
def test_negative_synchronize_private_registry_wrong_password(self, repo_options, repo):
"""Create and try to sync a Docker-type repository from a private
registry providing wrong credentials the sync must fail with
reasonable error message.
:id: 2857fce2-fed7-49fc-be20-bf2e4726c9f5
:parametrized: yes
:expectedresults: A repository is created with a private Docker \
repository and sync fails with reasonable error message.
:customerscenario: true
:BZ: 1475121, 1580510
"""
msg = "401, message=\'Unauthorized\'"
with pytest.raises(TaskFailedError, match=msg):
repo.sync()

@pytest.mark.tier2
@pytest.mark.parametrize(
'repo_options',
Expand Down Expand Up @@ -2320,27 +2237,6 @@ def test_positive_upload_file_to_file_repo(self, repo, target_sat):
)
assert constants.RPM_TO_UPLOAD == filesearch[0].name

@pytest.mark.stubbed
@pytest.mark.tier1
def test_positive_file_permissions(self):
"""Check file permissions after file upload to File Repository
:id: 03b4b7dd-0505-4302-ae00-5de33ad420b0
:Setup:
1. Create a File Repository
2. Upload an arbitrary file to it
:steps: Retrieve file permissions from File Repository
:expectedresults: uploaded file permissions are kept after upload
:CaseImportance: Critical
:CaseAutomation: NotAutomated
"""
pass

@pytest.mark.tier1
@pytest.mark.upgrade
@pytest.mark.parametrize(
Expand Down Expand Up @@ -2374,82 +2270,6 @@ def test_positive_remove_file(self, repo, target_sat):
repo.remove_content(data={'ids': [file_detail[0].id], 'content_type': 'file'})
assert repo.read().content_counts['file'] == 0

@pytest.mark.stubbed
@pytest.mark.tier2
@pytest.mark.upgrade
def test_positive_remote_directory_sync(self):
"""Check an entire remote directory can be synced to File Repository
through http
:id: 5c29b758-004a-4c71-a860-7087a0e96747
:Setup:
1. Create a directory to be synced with a pulp manifest on its root
2. Make the directory available through http
:steps:
1. Create a File Repository with url pointing to http url
created on setup
2. Initialize synchronization
:expectedresults: entire directory is synced over http
:CaseAutomation: NotAutomated
"""
pass

@pytest.mark.stubbed
@pytest.mark.tier1
def test_positive_local_directory_sync(self):
"""Check an entire local directory can be synced to File Repository
:id: 178145e6-62e1-4cb9-b825-44d3ab41e754
:Setup:
1. Create a directory to be synced with a pulp manifest on its root
locally (on the Satellite/Foreman host)
:steps:
1. Create a File Repository with url pointing to local url
created on setup
2. Initialize synchronization
:expectedresults: entire directory is synced
:CaseImportance: Critical
:CaseAutomation: NotAutomated
"""
pass

@pytest.mark.stubbed
@pytest.mark.tier1
def test_positive_symlinks_sync(self):
"""Check synlinks can be synced to File Repository
:id: 438a8e21-3502-4995-86db-c67ba0f3c469
:Setup:
1. Create a directory to be synced with a pulp manifest on its root
locally (on the Satellite/Foreman host)
2. Make sure it contains synlinks
:steps:
1. Create a File Repository with url pointing to local url
created on setup
2. Initialize synchronization
:expectedresults: entire directory is synced, including files
referred by symlinks
:CaseImportance: Critical
:CaseAutomation: NotAutomated
"""
pass


@pytest.mark.skip_if_not_set('container_repo')
class TestTokenAuthContainerRepository:
Expand Down
Loading

0 comments on commit 96b7d33

Please sign in to comment.