Closes #539 by removing quota checks altogether, as requested by @retzkek in #539 discussion.
I'd like to note that while this may address the issue immediately, a few issues in the testing came up by the fact that we run commands using os.system rather than the subprocess functions. I'm going to open a new issue specifically to address that in the tarfiles.tar_up func, since that's the one that made things tougher here.
Another note: For render_files, I had to split the render_files func up to make our fake filesystem tests work with functions that have tracing decorators. The trace should still look the same, and we may want to do for other functions as well.
Final note: The fake filesystem was implemented using (formerly Google's) pyfakefs pytest plugin. To run those unit tests, the reviewer will have to run pip install pyfakefs.
Closes #539 by removing quota checks altogether, as requested by @retzkek in #539 discussion.
I'd like to note that while this may address the issue immediately, a few issues in the testing came up by the fact that we run commands using
os.systemrather than thesubprocessfunctions. I'm going to open a new issue specifically to address that in thetarfiles.tar_upfunc, since that's the one that made things tougher here.Another note: For
render_files, I had to split therender_filesfunc up to make our fake filesystem tests work with functions that have tracing decorators. The trace should still look the same, and we may want to do for other functions as well.Final note: The fake filesystem was implemented using (formerly Google's)
pyfakefspytest plugin. To run those unit tests, the reviewer will have to runpip install pyfakefs.