Trying some things for codecov

Author: rvasav26

src/skmatter/decomposition/_kpcov.py

@@ -77,7 +77,6 @@ def _get_kernel(self, X, Y=None):
     @abstractmethod
     def fit(self, X, Y):
         """Fit the model with X and Y. Subclasses should implement this method."""
-        pass
 
     def _set_fit_params(self, X):
         """Initializes common fit parameters for PCovR and PCovC."""

src/skmatter/decomposition/_pcov.py

@@ -51,7 +51,6 @@ def __init__(
     @abstractmethod
     def fit(self, X, Y):
         """Fit the model with X and Y. Subclasses should implement this method."""
-        pass
 
     def _set_fit_params(self, X):
         """Initializes common fit parameters for PCovR and PCovC."""

tests/test_pcovc.py

@@ -25,9 +25,13 @@ def __init__(self, *args, **kwargs):
         )
 
         self.error_tol = 1e-5
-
         self.X, self.Y = get_dataset(return_X_y=True)
 
+        # n_samples > 500 to ensure our svd_solver tests catch all cases
+        X_stacked = np.tile(self.X, (4, 1))
+        Y_stacked = np.tile(self.Y, 4)
+        self.X, self.Y = X_stacked, Y_stacked
+
         scaler = StandardScaler()
         self.X = scaler.fit_transform(self.X)
 
@@ -175,8 +179,7 @@ def test_select_sample_space(self):
         Check that PCovC implements the sample space version
         when :math:`n_{features} > n_{samples}``.
         """
-        pcovc = self.model(n_components=2, tol=1e-12)
-
+        pcovc = self.model(n_components=1, tol=1e-12, svd_solver="arpack")
         n_samples = 2
 
         # select range where there are at least 2 classes in Y