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

ogr mock support for packit tests with unittest classes #310

Merged
merged 1 commit into from Jun 28, 2019

Conversation

jscotka
Copy link
Contributor

@jscotka jscotka commented May 7, 2019

replaces: #303

tested by command: PYTHONPATH=. pytest-3 -v tests/integration_ogr_mock/ with version of ogr what support it.

Update:
with local run tests works as charm.

  • via make check locally
    • 2 tests fails because of missing bodhi package
    • 1 test failed because there were mentioned libpaguire dependency, what dowloads latest instead lower what is mentioned in ogr deps

@packit-as-a-service
Copy link

Congratulations! The build has finished successfully. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/packit-service-packit-310
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

1 similar comment
@packit-as-a-service
Copy link

Congratulations! The build has finished successfully. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/packit-service-packit-310
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

Copy link
Member

@TomasTomecek TomasTomecek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

design LGTM



class ProposeUpdate(PackitUnittestOgr):
datafile_github = "TestPropose_github.yaml"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these could be generated in the end, right? something like : TEST_DATA_TOP_DIR / test_file_name / test_class_name + ".yaml"

Copy link
Contributor Author

@jscotka jscotka May 9, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@TomasTomecek you suggestion is to create trere another subdirectory for each "test file" and inside this have files for each testclass?
it is stored in tearDown function so it dumps data after every test.

update: adapted your suggestion and removed manual class name files handling

@packit-as-a-service
Copy link

Congratulations! The build has finished successfully. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/packit-service-packit-310
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

@jpopelka
Copy link
Member

jpopelka commented May 14, 2019

ogr 0.3.0 has been released. Let's re-run the tests.

@jpopelka
Copy link
Member

@jscotka please rebase to re-run the tests, thanks

@packit-as-a-service
Copy link

Congratulations! The build has finished successfully. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/packit-service-packit-310
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

@packit-as-a-service
Copy link

Congratulations! The build has finished successfully. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/packit-service-packit-310
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

2 similar comments
@packit-as-a-service
Copy link

Congratulations! The build has finished successfully. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/packit-service-packit-310
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

@packit-as-a-service
Copy link

Congratulations! The build has finished successfully. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/packit-service-packit-310
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

@packit-as-a-service
Copy link

There was an error while creating a SRPM.

Output:

error: Bad source: /sandcastle/packitos-0.4.2.dev86+g5e51f33.tar.gz: No such file or directory

Return code: 1

@packit-as-a-service
Copy link

There was an error while creating a SRPM.

Output:

error: Bad source: /sandcastle/packitos-0.4.2.dev87+g92ed0a7.tar.gz: No such file or directory

Return code: 1

@packit-as-a-service
Copy link

There was an error while creating a SRPM.

Output:

error: Bad source: /sandcastle/packitos-0.4.2.dev88+g3e8e15f.tar.gz: No such file or directory

Return code: 1

@packit-as-a-service
Copy link

There was an error while creating a SRPM.

Output:

error: Bad source: /sandcastle/packitos-0.4.2.dev81+gb8c7e51.tar.gz: No such file or directory

Return code: 1

@packit-as-a-service
Copy link

There was an error while creating a SRPM.

Output:

error: Bad source: /sandcastle/packitos-0.4.2.dev81+gd2b96e5.tar.gz: No such file or directory

Return code: 1

@packit-as-a-service
Copy link

There was an error while creating a SRPM.

Output:

error: Bad source: /sandcastle/packitos-0.4.2.dev81+g0c4c0fa.tar.gz: No such file or directory

Return code: 1

@packit-as-a-service
Copy link

There was an error while creating a SRPM.

Output:

error: Bad source: /sandcastle/packitos-0.4.2.dev81+ge06083c.tar.gz: No such file or directory

Return code: 1

Copy link
Member

@lachmanfrantisek lachmanfrantisek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, try to use the current master version of OGR if it is possible.

If not, I'll create a follow-up for that...

@packit-as-a-service
Copy link

There was an error while creating a SRPM.

Output:

error: Bad source: ./packitos-0.4.2.dev81+ge06083c.tar.gz: No such file or directory

Return code: 1

1 similar comment
@packit-as-a-service
Copy link

There was an error while creating a SRPM.

Output:

error: Bad source: ./packitos-0.4.2.dev81+ge06083c.tar.gz: No such file or directory

Return code: 1

@TomasTomecek
Copy link
Member

I'm trying to fix the issue so I'm using this PR for testing, please don't mind packit's comments

@jscotka
Copy link
Contributor Author

jscotka commented Jun 28, 2019

Please, try to use the current master version of OGR if it is possible.

If not, I'll create a follow-up for that...

It is not possible to do it without not nice workarounds.

@packit-as-a-service
Copy link

Congratulations! The build has finished successfully. 🍾

You can install the built RPMs by following these steps:

  • sudo yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/packit-service-packit-310
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

@jscotka
Copy link
Contributor Author

jscotka commented Jun 28, 2019

I've force pushed after rebase bad data, need to do it again from proper computer

move status from base test class to status test

adapt PR suggestion for datafile path location for test classes
@jscotka
Copy link
Contributor Author

jscotka commented Jun 28, 2019

rebased to proper branch data

@jscotka jscotka merged commit c92acc4 into packit:master Jun 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants