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

fix unittests and run them in gh-actions #54

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hiaselhans
Copy link

@hiaselhans hiaselhans commented Nov 18, 2021

this is a new take on #43

  • drop python version <3.5
  • fix the failing tests
  • add time.sleep() to the tests to avoid hitting the VIES rate-limit
  • add flake-config and fix minor flake issues

@hiaselhans hiaselhans force-pushed the unittests branch 2 times, most recently from 797167a to 49984cc Compare November 18, 2021 17:02
@hiaselhans
Copy link
Author

i would like to push this topic again.

Most lines are formatting issues, if you are not fine with those changes please let me know and i can reduce this pr to the minimum.
Other than that, i think we can definitely make use of automated testing with a library crucial for business logic.

@hiaselhans
Copy link
Author

@martinleblanc can i get your feedback on this? Tests are failing in the current master.

i fixed the tests and automated testing might help to reduce your workload.

@italomg
Copy link

italomg commented Jul 20, 2022

I am currently packing pyvat in an RPM for Fedora. It would be great if we can run tests without unittest2 🙂

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.

2 participants