Bring coverage back to 100%

src/cr/cube/crunch_cube.py

@@ -17,7 +17,6 @@
 from cr.cube.dimension import AllDimensions
 from cr.cube.enum import DIMENSION_TYPE as DT
 from cr.cube.measures.index import Index
-from cr.cube.measures.pairwise_pvalues import PairwisePvalues
 from cr.cube.measures.scale_means import ScaleMeans
 from cr.cube.util import lazyproperty
 
@@ -724,7 +723,7 @@ def pairwise_pvals(self, axis=0):
         are implemented currently.
         :returns pvals: list of symmetric matrices of column-comparison p-values.
         """
-        return [PairwisePvalues(slice_, axis=axis).values for slice_ in self.slices]
+        return [slice_.pairwise_pvals(axis=axis) for slice_ in self.slices]
 
     def _adjust_axis(self, axis):
         """Return raw axis/axes corresponding to apparent axis/axes.

src/cr/cube/cube_slice.py

@@ -19,7 +19,7 @@
 
 try:
     xrange
-except NameError:
+except NameError:  # pragma: no cover
     xrange = range
 
 
@@ -318,6 +318,8 @@ def pairwise_pvals(self, axis=0):
         are implemented currently.
         :returns pvals: square, symmetric matrix of column-comparison p-values.
         """
+        if axis != 0:
+            raise NotImplementedError("Pairwise comparison only implemented for colums")
         return PairwisePvalues(self, axis=axis).values
 
     def pvals(self, weighted=True, prune=False, hs_dims=None):

src/cr/cube/distributions/wishart.py

@@ -19,7 +19,7 @@
 
 try:
     xrange
-except NameError:
+except NameError:  # pragma: no cover
     xrange = range
 
 

src/cr/cube/measures/pairwise_pvalues.py

@@ -11,7 +11,7 @@
 
 try:
     xrange
-except NameError:
+except NameError:  # pragma: no cover
     xrange = range
 
 

tests/integration/test_pairwise.py

@@ -3,14 +3,15 @@
 """Integration tests for pairwise comparisons."""
 
 from unittest import TestCase
+import pytest
 
 import numpy as np
 
-from ..fixtures import CR
-
 from cr.cube.crunch_cube import CrunchCube
 from cr.cube.measures.pairwise_pvalues import PairwisePvalues
 
+from ..fixtures import CR
+
 
 # pylint: disable=missing-docstring, invalid-name, no-self-use
 class TestStandardizedResiduals(TestCase):
@@ -162,6 +163,10 @@ def test_same_col_pvals(self):
         actual = cube.pairwise_pvals(axis=0)
         np.testing.assert_equal(actual, expected)
 
+        # Assert correct exception in case of not-implemented direction
+        with pytest.raises(NotImplementedError):
+            cube.pairwise_pvals(axis=1)
+
     def test_hirotsu_pvals(self):
         cube = CrunchCube(CR.PAIRWISE_HIROTSU_ILLNESS_X_OCCUPATION)
         actual = cube.pairwise_pvals(axis=0)