use pytest instead of nose for automated tests (#204)

* remove module from singlediode

* remove unused partial

* use pytest in solarposition and pvsystem

* remove all nose imports and make tests pass

* add nose to envs for pandas

* add requires_numba decorator

* add pytest-runner to setup

* use pytest on appveyor

* update whatsnew

* clean up rebase mistake

* replace xfails with pytest.raises

* remove init in favor of conftest.py

* switch to pip install

Author: wholmgren

.travis.yml

@@ -41,7 +41,7 @@ before_install:
     - conda config --set always_yes yes --set changeps1 no
     - conda update -q conda
     - conda info -a
-    
+
 install:
     - echo "install"
     - conda env create --file ci/requirements-$CONDA_ENV.yml
@@ -51,10 +51,10 @@ install:
     - ls -l /home/travis/miniconda/envs/test_env/lib
     #- pip install . # use pip to automatically install anything not in the yml files (i.e. numpy/scipy/pandas for py3*)
     #- pip install scipy # won't do anything if already installed
-    - python setup.py install
+    - pip install -e .
 
 script:
-    - nosetests -v --with-coverage --cover-package=pvlib pvlib
+    - py.test pvlib --cov=pvlib --cov-report term-missing
 
 after_success:
     coveralls

appveyor.yml

@@ -37,4 +37,4 @@ install:
 build: false
 
 test_script:
-    - "nosetests -v pvlib"
+    - "py.test -v pvlib"

ci/requirements-py27-min.yml

@@ -1,9 +1,11 @@
 name: test_env
 dependencies:
-    - python=2.7 
+    - python=2.7
     - numpy==1.8.2
     - pandas==0.13.1
-    - nose
     - pytz
+    - pytest
+    - pytest-cov
+    - nose
     - pip:
-        - coveralls
+        - coveralls

ci/requirements-py27.yml

@@ -1,12 +1,14 @@
 name: test_env
 dependencies:
-    - python=2.7 
+    - python=2.7
     - numpy
     - scipy
     - pandas
-    - nose
     - pytz
     - ephem
     - numba
+    - pytest
+    - pytest-cov
+    - nose
     - pip:
-        - coveralls
+        - coveralls

ci/requirements-py34.yml

@@ -4,9 +4,11 @@ dependencies:
     - numpy
     - scipy
     - pandas
-    - nose
     - pytz
     - ephem
     - numba
+    - pytest
+    - pytest-cov
+    - nose
     - pip:
-        - coveralls
+        - coveralls

ci/requirements-py35.yml

@@ -4,9 +4,11 @@ dependencies:
     - numpy
     - scipy
     - pandas
-    - nose
     - pytz
     - ephem
     - numba
+    - pytest
+    - pytest-cov
+    - nose
     - pip:
-        - coveralls
+        - coveralls

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

@@ -38,6 +38,7 @@ Documentation
 Other
 ~~~~~
 
+* Switch to the py.test testing framework. (:issue:`204`)
 
 
 Code Contributors

pvlib/test/__init__.py

@@ -0,0 +1,64 @@
+import sys
+import platform
+
+import pandas as pd
+import numpy as np
+import pytest
+
+
+try:
+    import scipy
+    has_scipy = True
+except ImportError:
+    has_scipy = False
+
+requires_scipy = pytest.mark.skipif(not has_scipy, reason='requires scipy')
+
+
+try:
+    import ephem
+    has_ephem = True
+except ImportError:
+    has_ephem = False
+
+requires_ephem = pytest.mark.skipif(not has_ephem, reason='requires ephem')
+
+
+incompatible_pandas_0131 = pytest.mark.skipif(
+    pd.__version__ == '0.13.1', reason='requires numpy 1.10 or greater')
+
+
+def numpy_1_10():
+    version = tuple(map(int, np.__version__.split('.')))
+    if version[0] <= 1 and version[1] < 10:
+        return False
+    else:
+        return True
+
+needs_numpy_1_10 = pytest.mark.skipif(
+    not numpy_1_10(), reason='requires numpy 1.10 or greater')
+
+
+def has_spa_c():
+    try:
+        from pvlib.spa_c_files.spa_py import spa_calc
+    except ImportError:
+        return False
+    else:
+        return True
+
+requires_spa_c = pytest.mark.skipif(not has_spa_c(), reason="requires spa_c")
+
+def has_numba():
+    try:
+        import numba
+    except ImportError:
+        return True
+    else:
+        vers = numba.__version__.split('.')
+        if int(vers[0] + vers[1]) < 17:
+            return False
+        else:
+            return True
+
+requires_numba = pytest.mark.skipif(not has_numba(), reason="requires numba")

pvlib/test/conftest.py

@@ -4,8 +4,7 @@
 import numpy as np
 import pandas as pd
 
-from nose.tools import raises
-from nose.tools import assert_almost_equals
+import pytest
 from numpy.testing import assert_allclose
 
 from pvlib.location import Location
@@ -30,25 +29,21 @@
 def test_pres2alt():
     atmosphere.pres2alt(100000)
 
+
 def test_alt2press():
     atmosphere.pres2alt(1000)
 
 
-# two functions combined will generate unique unit tests for each model
-def test_airmasses():
-    models = ['simple', 'kasten1966', 'youngirvine1967', 'kastenyoung1989',
-              'gueymard1993', 'young1994', 'pickering2002']
-    for model in models:
-        yield run_airmass, model, ephem_data['zenith']
-
-
-def run_airmass(model, zenith):
-    atmosphere.relativeairmass(zenith, model)
+@pytest.mark.parametrize("model",
+    ['simple', 'kasten1966', 'youngirvine1967', 'kastenyoung1989',
+     'gueymard1993', 'young1994', 'pickering2002'])
+def test_airmass(model):
+    atmosphere.relativeairmass(ephem_data['zenith'], model)
 
 
-@raises(ValueError)
 def test_airmass_invalid():
-    atmosphere.relativeairmass(ephem_data['zenith'], 'invalid')
+    with pytest.raises(ValueError):
+        atmosphere.relativeairmass(ephem_data['zenith'], 'invalid')
 
 
 def test_absoluteairmass():
@@ -79,31 +74,26 @@ def test_gueymard94_pw():
     assert_allclose(pws, expected, atol=0.01)
 
 
-def test_first_solar_spectral_correction():
-    ams = np.array([1, 3, 5])
-    pws = np.array([1, 3, 5])
-    ams, pws = np.meshgrid(ams, pws)
-
-    expect = {}
-    expect['cdte'] = np.array(
+@pytest.mark.parametrize("module_type,expect", [
+    ('cdte', np.array(
         [[ 0.99134828,  0.97701063,  0.93975103],
          [ 1.02852847,  1.01874908,  0.98604776],
-         [ 1.04722476,  1.03835703,  1.00656735]])
-    expect['monosi'] = np.array(
+         [ 1.04722476,  1.03835703,  1.00656735]])),
+    ('monosi', np.array(
         [[ 0.9782842 ,  1.02092726,  1.03602157],
          [ 0.9859024 ,  1.0302268 ,  1.04700244],
-         [ 0.98885429,  1.03351495,  1.05062687]])
-    expect['polysi'] = np.array(
+         [ 0.98885429,  1.03351495,  1.05062687]])),
+    ('polysi', np.array(
         [[ 0.9774921 ,  1.01757872,  1.02649543],
          [ 0.98947361,  1.0314545 ,  1.04226547],
-         [ 0.99403107,  1.03639082,  1.04758064]])
-
-    def run_fs_test(module_type):
-        out = atmosphere.first_solar_spectral_correction(pws, ams, module_type)
-        assert_allclose(out, expect[module_type], atol=0.001)
-
-    for module_type in expect.keys():
-        yield run_fs_test, module_type
+         [ 0.99403107,  1.03639082,  1.04758064]]))
+])
+def test_first_solar_spectral_correction(module_type, expect):
+    ams = np.array([1, 3, 5])
+    pws = np.array([1, 3, 5])
+    ams, pws = np.meshgrid(ams, pws)
+    out = atmosphere.first_solar_spectral_correction(pws, ams, module_type)
+    assert_allclose(out, expect, atol=0.001)
 
 
 def test_first_solar_spectral_correction_supplied():
@@ -114,6 +104,6 @@ def test_first_solar_spectral_correction_supplied():
     assert_allclose(out, expected, atol=1e-3)
 
 
-@raises(TypeError)
 def test_first_solar_spectral_correction_ambiguous():
-    atmosphere.first_solar_spectral_correction(1, 1)
+    with pytest.raises(TypeError):
+        atmosphere.first_solar_spectral_correction(1, 1)