civisanalytics / civis-python

Civis API Python Client
BSD 3-Clause "New" or "Revised" License
34 stars 26 forks source link

BUG Fix unintentional dependency on scikit-learn for testing #303

Closed stephen-hoover closed 5 years ago

stephen-hoover commented 5 years ago

The test_parallel.test_setup_remote_backend function would give an error if scikit-learn was not installed. This is because we try to mock the parallel._sklearn_reg_para_backend function, and that doesn't exist if scikit-learn isn't installed. Fix by ensuring that attribute is always defined.

Closes #245 .