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

Making automated tests #16

Closed
wants to merge 3 commits into from
Closed

Making automated tests #16

wants to merge 3 commits into from

Conversation

callumrollo
Copy link
Member

@MOchiara this PR adds automated tests that will run on each new PR. Have a look at the individual pieces that come together to make it work

@callumrollo
Copy link
Member Author

You can see how the tests have already triggered. Learning points:

  • note how the test directory is called "tests" and every file and function in it started with test
  • I've added pytest to requirements-dev, as well as adding all the stuff that has been added to the regular requirements
  • the file .github/workflows/tests/yml is what controls making these tests run on github on a variety of computers and pyhon versions

@callumrollo
Copy link
Member Author

notice how it fails for python version 3.8. Can you work out why?

@callumrollo
Copy link
Member Author

Try checking out the details of the failed tests

@callumrollo
Copy link
Member Author

note how the tests trigger again after I made a new commit and pushed it

@MOchiara
Copy link
Member

MOchiara commented Jul 1, 2024

This looks faster than mine...

@callumrollo
Copy link
Member Author

hmmm, what's the difference between the two?

@callumrollo callumrollo closed this Jul 1, 2024
@callumrollo callumrollo mentioned this pull request Jul 1, 2024
@callumrollo callumrollo deleted the callum-patch-2 branch October 29, 2024 14:15
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