Merge pull request #180 from Crunch-io/coverage-attempt

Coverage attempt

src/cr/cube/cube.py

@@ -544,7 +544,7 @@ def raw_cube_array(self):
         return array
 
     @lazyproperty
-    def _flat_values(self):
+    def _flat_values(self):  # pragma: no cover
         """Return tuple of mean values as found in cube response.
 
         This property must be implemented by each subclass.

src/cr/cube/dimension.py

@@ -32,7 +32,9 @@ def _dimensions(self):
         This composed tuple is the source for the dimension objects in this
         collection.
         """
-        raise NotImplementedError("must be implemented by each sublass")
+        raise NotImplementedError(
+            "must be implemented by each subclass"
+        )  # pragma: no cover
 
 
 class AllDimensions(_BaseDimensions):
@@ -755,7 +757,9 @@ def fill(self):
         A str value must be a hash character ("#") followed by six hexadecimal digits.
         Three-character color contractions (like "#D07") are not valid.
         """
+
         fill = self._element_transforms_dict.get("fill")
+
         if not fill:
             return None
         return fill
@@ -931,14 +935,6 @@ def addend_idxs(self):
             for addend_id in self.addend_ids
         )
 
-    @property
-    def is_insertion(self):
-        """True if this item represents an insertion (e.g. subtotal).
-
-        Unconditionally True for _Subtotal objects.
-        """
-        return True
-
     @lazyproperty
     def label(self):
         """str display name for this subtotal, suitable for use as label."""