-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
General tests cleanup #549
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this is necessary, the project paths aren't being setup correctly. This will need refactored...
@mbeacom I'll look into fixing the path issues. |
@mbeacom see moviepy/editor.py line 53 to see why we are getting resize errors. It seems that editor.py does more than just import a few files. For the test files that are producing the resize errors I'm going to put back the from editor import *. |
this PR looks good to me. |
The purpose of this PR is to cleanup some of the
tests/*
handling, sprinkle some pep-8, simplify some imports, etc. This isn't meant to be a complete cleanup and refactor of tests, but a step towards instituting pep-8/pep-256/257 standards (line limits/syntax, docstrings, etc) while making it a little more DRY.