-
Notifications
You must be signed in to change notification settings - Fork 44
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
f50c0bf
commit f93f19a
Showing
13 changed files
with
212 additions
and
212 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
115 changes: 0 additions & 115 deletions
115
pulp_smash/tests/pulp3/file/api_v3/test_crud_importers.py
This file was deleted.
Oops, something went wrong.
115 changes: 115 additions & 0 deletions
115
pulp_smash/tests/pulp3/file/api_v3/test_crud_remotes.py
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,115 @@ | ||
| # coding=utf-8 | ||
| """Tests that CRUD remotes.""" | ||
| import random | ||
| import unittest | ||
|
|
||
| from requests.exceptions import HTTPError | ||
|
|
||
| from pulp_smash import api, config, selectors, utils | ||
| from pulp_smash.constants import FILE_FEED_URL, FILE2_FEED_URL | ||
| from pulp_smash.tests.pulp3.constants import FILE_REMOTE_PATH, REPO_PATH | ||
| from pulp_smash.tests.pulp3.file.api_v3.utils import gen_remote | ||
| from pulp_smash.tests.pulp3.file.utils import set_up_module as setUpModule # noqa pylint:disable=unused-import | ||
| from pulp_smash.tests.pulp3.pulpcore.utils import gen_repo | ||
| from pulp_smash.tests.pulp3.utils import get_auth | ||
|
|
||
|
|
||
| class CRUDRemotesTestCase(unittest.TestCase, utils.SmokeTest): | ||
| """CRUD remotes.""" | ||
|
|
||
| @classmethod | ||
| def setUpClass(cls): | ||
| """Create class-wide variables. | ||
| In order to create an remote a repository has to be created first. | ||
| """ | ||
| cls.cfg = config.get_config() | ||
| cls.client = api.Client(cls.cfg, api.json_handler) | ||
| cls.client.request_kwargs['auth'] = get_auth() | ||
| cls.remote = {} | ||
| cls.repo = cls.client.post(REPO_PATH, gen_repo()) | ||
|
|
||
| @classmethod | ||
| def tearDownClass(cls): | ||
| """Clean class-wide variable.""" | ||
| cls.client.delete(cls.repo['_href']) | ||
|
|
||
| def test_01_create_remote(self): | ||
| """Create an remote.""" | ||
| body = _gen_verbose_remote() | ||
| type(self).remote = self.client.post(FILE_REMOTE_PATH, body) | ||
| for key in ('username', 'password'): | ||
| del body[key] | ||
| for key, val in body.items(): | ||
| with self.subTest(key=key): | ||
| self.assertEqual(self.remote[key], val) | ||
|
|
||
| @selectors.skip_if(bool, 'remote', False) | ||
| def test_02_read_remote(self): | ||
| """Read an remote by its href.""" | ||
| remote = self.client.get(self.remote['_href']) | ||
| for key, val in self.remote.items(): | ||
| with self.subTest(key=key): | ||
| self.assertEqual(remote[key], val) | ||
|
|
||
| @selectors.skip_if(bool, 'remote', False) | ||
| def test_02_read_remotes(self): | ||
| """Read an remote by its name.""" | ||
| page = self.client.get(FILE_REMOTE_PATH, params={ | ||
| 'name': self.remote['name'] | ||
| }) | ||
| self.assertEqual(len(page['results']), 1) | ||
| for key, val in self.remote.items(): | ||
| with self.subTest(key=key): | ||
| self.assertEqual(page['results'][0][key], val) | ||
|
|
||
| @selectors.skip_if(bool, 'remote', False) | ||
| def test_03_partially_update(self): | ||
| """Update an remote using HTTP PATCH.""" | ||
| body = _gen_verbose_remote() | ||
| self.client.patch(self.remote['_href'], body) | ||
| for key in ('username', 'password'): | ||
| del body[key] | ||
| type(self).remote = self.client.get(self.remote['_href']) | ||
| for key, val in body.items(): | ||
| with self.subTest(key=key): | ||
| self.assertEqual(self.remote[key], val) | ||
|
|
||
| @selectors.skip_if(bool, 'remote', False) | ||
| def test_04_fully_update(self): | ||
| """Update an remote using HTTP PUT.""" | ||
| body = _gen_verbose_remote() | ||
| self.client.put(self.remote['_href'], body) | ||
| for key in ('username', 'password'): | ||
| del body[key] | ||
| type(self).remote = self.client.get(self.remote['_href']) | ||
| for key, val in body.items(): | ||
| with self.subTest(key=key): | ||
| self.assertEqual(self.remote[key], val) | ||
|
|
||
| @selectors.skip_if(bool, 'remote', False) | ||
| def test_05_delete(self): | ||
| """Delete an remote.""" | ||
| self.client.delete(self.remote['_href']) | ||
| with self.assertRaises(HTTPError): | ||
| self.client.get(self.remote['_href']) | ||
|
|
||
|
|
||
| def _gen_verbose_remote(): | ||
| """Return a semi-random dict for use in defining an remote. | ||
| For most tests, it's desirable to create remotes with as few attributes | ||
| as possible, so that the tests can specifically target and attempt to break | ||
| specific features. This module specifically targets remotes, so it makes | ||
| sense to provide as many attributes as possible. | ||
| Note that 'username' and 'password' are write-only attributes. | ||
| """ | ||
| attrs = gen_remote() | ||
| attrs.update({ | ||
| 'feed_url': random.choice((FILE_FEED_URL, FILE2_FEED_URL)), | ||
| 'password': utils.uuid4(), | ||
| 'username': utils.uuid4(), | ||
| 'validate': random.choice((False, True)), | ||
| }) | ||
| return attrs |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.