improve forecast test reliability (#303)

* put conda-forge first in fx builds

* add pytest timeout

* add skip windows

* update whatsnew

Author: wholmgren

ci/requirements-py27.yml

@@ -1,7 +1,7 @@
 name: test_env
 channels:
+    - conda-forge
     - defaults
-    - http://conda.anaconda.org/conda-forge
 dependencies:
     - python=2.7
     - numpy
@@ -16,3 +16,4 @@ dependencies:
     - nose
     - pip:
         - coveralls
+        - pytest-timeout

ci/requirements-py34.yml

@@ -1,7 +1,7 @@
 name: test_env
 channels:
-    - defaults
     - conda-forge
+    - defaults
 dependencies:
     - python=3.4
     - numpy
@@ -16,3 +16,4 @@ dependencies:
     - nose
     - pip:
         - coveralls
+        - pytest-timeout

ci/requirements-py35.yml

@@ -1,7 +1,7 @@
 name: test_env
 channels:
-    - defaults
     - conda-forge
+    - defaults
 dependencies:
     - python=3.5
     - numpy
@@ -16,3 +16,4 @@ dependencies:
     - nose
     - pip:
         - coveralls
+        - pytest-timeout

docs/sphinx/source/whatsnew/v0.4.4.txt

@@ -18,6 +18,15 @@ Documentation
 * Fixes broken Classes link in the v0.3.0 documentation.
 
 
+Bug fixes
+~~~~~~~~~
+
+* Resolved several issues with the forecast module tests. Library import
+  errors were resolved by prioritizing the conda-forge channel over the
+  default channel. Stalled ci runs were resolved by adding a timeout to
+  the HRRR_ESRL test. (:issue:`293`)
+
+
 Contributors
 ~~~~~~~~~~~~
 

pvlib/test/conftest.py

@@ -7,6 +7,9 @@
 import pytest
 
 
+skip_windows = pytest.mark.skipif('win' in sys.platform,
+                                  reason='does not run on windows')
+
 try:
     import scipy
     has_scipy = True

pvlib/test/test_forecast.py

@@ -9,7 +9,7 @@
 import pytest
 from numpy.testing import assert_allclose
 
-from conftest import requires_siphon, has_siphon
+from conftest import requires_siphon, has_siphon, skip_windows
 
 pytestmark = pytest.mark.skipif(not has_siphon, reason='requires siphon')
 
@@ -30,7 +30,10 @@
     _end = _start + pd.Timedelta(days=1)
     _modelclasses = [
         GFS, NAM, HRRR, NDFD, RAP,
-        pytest.mark.xfail(HRRR_ESRL, reason="HRRR_ESRL is unreliable")]
+        skip_windows(
+            pytest.mark.xfail(
+                pytest.mark.timeout(HRRR_ESRL, timeout=60),
+                reason="HRRR_ESRL is unreliable"))]
     _working_models = []
     _variables = ['temp_air', 'wind_speed', 'total_clouds', 'low_clouds',
                   'mid_clouds', 'high_clouds', 'dni', 'dhi', 'ghi',]