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.

Sign up for GitHub

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

Jump to bottom

Test that --force_full is supported in export #301

Closed
preethit opened this issue Jun 10, 2016 · 4 comments
Closed

Test that --force_full is supported in export #301

preethit opened this issue Jun 10, 2016 · 4 comments
Labels
Issue Type: Test Case
Milestone
Chongrui - Summer...

Comments

@preethit
Copy link

preethit commented Jun 10, 2016 •
edited by elyezer

https://pulp.plan.io/issues/1938

Steps

  1. Create a repository
  2. Sync the repo
  3. Export the repo with --force_full option

Assert

  1. --force_full can be set to true
  2. --force_full can be set to false
  3. --force-full option can be omitted

$ pulp-admin -vv rpm repo export run --repo-id foo --force-full

+----------------------------------------------------------------------+
Publishing Repository [ina]
+----------------------------------------------------------------------+

The following publish configuration options will be used:

Force Full: True
@Ichimonji10
Copy link
Contributor

We have a test for the force_full API v2 option in module pulp_smash.tests.rpm.api_v2.test_force_full. @preethit, is this existing test satisfactory?

@preethit
Copy link
Author

@Ichimonji10 Those tests only seem to cover the --force_full option in publish.

Repo export also has the option and this issue is for that.

@Ichimonji10
Copy link
Contributor

Ahh. So if pulp_smash.tests.rpm.api_v2.test_export were expanded, then that'd fix this issue. Correct?

@preethit
Copy link
Author

Yes. That is the one.

@danuzclaudes danuzclaudes self-assigned this Jun 30, 2016
@danuzclaudes danuzclaudes removed their assignment Jul 8, 2016
@danuzclaudes danuzclaudes self-assigned this Aug 5, 2016
danuzclaudes added a commit to danuzclaudes/pulp-smash that referenced this issue Aug 9, 2016
@danuzclaudes
Fix up Pulp Smash pulp#301
8aa56b5
danuzclaudes added a commit to danuzclaudes/pulp-smash that referenced this issue Aug 9, 2016
@danuzclaudes
Fix up Pulp Smash pulp#301
06b2ff6 
This commit fixes the Pulp Smash issue pulp#301 and Pulp issue #1938.

Test steps:
    1. Create and sync a repository.
    2. Checkout to content directory /var/lib/pulp/content/units/rpm/.
    3. Remove some random contents in that folder.
    4. Trigger a force sync to verify if the removed contents will be restored.

Test output:

```
$ python -m unittest pulp_smash.tests.rpm.cli.test_sync.ForceSyncTestCase
[sudo] password for chozhang:
.
----------------------------------------------------------------------
Ran 1 test in 28.589s

OK
```
@danuzclaudes danuzclaudes mentioned this issue Aug 9, 2016
Closed
danuzclaudes added a commit to danuzclaudes/pulp-smash that referenced this issue Aug 9, 2016
@danuzclaudes
Fix up Pulp Smash pulp#301
28ad5ba 
This commit fixes the Pulp Smash issue pulp#301 and Pulp issue #1938.

Test steps:
    1. Create and sync a repository.
    2. Checkout to content directory /var/lib/pulp/content/units/rpm/.
    3. Remove some random contents in that folder.
    4. Trigger a force sync to verify if the removed contents will be restored.

Test output:

```
$ python -m unittest pulp_smash.tests.rpm.cli.test_sync.ForceSyncTestCase
[sudo] password for chozhang:
.
----------------------------------------------------------------------
Ran 1 test in 28.589s

OK
```
elyezer pushed a commit that referenced this issue Aug 10, 2016
@danuzclaudes @elyezer
Fix up Pulp Smash #301
cd814fe 
This commit fixes the Pulp Smash issue #301 and Pulp issue #1938.

Test steps:
    1. Create and sync a repository.
    2. Checkout to content directory /var/lib/pulp/content/units/rpm/.
    3. Remove some random contents in that folder.
    4. Trigger a force sync to verify if the removed contents will be restored.

Test output:

```
$ python -m unittest pulp_smash.tests.rpm.cli.test_sync.ForceSyncTestCase
[sudo] password for chozhang:
.
----------------------------------------------------------------------
Ran 1 test in 28.589s

OK
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Issue Type: Test Case
Projects
None yet
Milestone
Chongrui - Summer Internship
Development

No branches or pull requests
3 participants
@Ichimonji10 @preethit @danuzclaudes