facultyai / faculty-sync

Synchronise a local directory with your Faculty platform workspace
https://pypi.org/project/faculty_sync/
Apache License 2.0
10 stars 1 forks source link

Fix broken cli tests and move cli tests to the right subpackage #6

Closed pbugnion closed 6 years ago

pbugnion commented 6 years ago

Moving the cli modules to a subpackage had broken an import path: it was looking for Configuration in models, rather than in cli.models.

This fixes that by moving cli tests to /cli/tests/, rather than just /tests/. I think having a separate subdirectory for each subpackage is more common (e.g. https://github.com/scikit-learn/scikit-learn/tree/master/sklearn/ensemble, https://github.com/scipy/scipy/tree/master/scipy/constants)