Closed aidanheerdegen closed 4 years ago
Added logic to choose 'short' or 'scratch' as default shortpath location depending on which exists
Added tests for lab path creation.
Rearranged existing tests and shared code to common.py
Hello @aidanheerdegen! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
test/test_paths.py
Line 89:1: E302 expected 2 blank lines, found 1
Added logic to choose 'short' or 'scratch' as default shortpath location depending on which exists
Added tests for lab path creation.
Rearranged existing tests and shared code to common.py