-
Notifications
You must be signed in to change notification settings - Fork 3
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
Write unit tests for wordpress, google docs import_export #53
Comments
Shannon wrote a ton of tests for the google translate translation_service. We should imitate her tests. |
@liquidsteves I took a quick look at this, based on how I structured the import/export docs code it might be hard to make unit tests without serious refactoring/modularizing. But if you see any low-hanging fruit let me know. |
Could also do integration tests where we have a designated source and destination folder in Google drive and do a simple caps translation or something, but not sure who would be in charge of the Google drive account. Could mock the responses, but that gets us back to unit testing which has the current code limitations I commented above. |
Refactoring and modularizing would be good if you have the energy for it, but it's not urgent. |
Integration tests are hard to maintain so I'd rather not build them until we have a clear need. (Eg if several new people were working on the code and important things kept breaking.) |
No description provided.
The text was updated successfully, but these errors were encountered: