Closed adj-smith closed 3 years ago
hey @adj-smith, nice test. Just a few things:
patch-1
branch, so that PR #253 automatically gets updated. That way we don't have two separate PRs (and we'll just close this one)if __name__ == "__main__"
at the bottom of the file. This prevents the test being called twice in the pytest framework, and the bottom call only runs if a user runs it manually from the command line. Check out some of our other tests for examples. Thanks!
Done. I was trying to work out how to include the test in my other pull request. Now I know. Thanks!
thanks! closing this PR to review changes in #253 instead.
See pull request #253 for the function being tested by this script.