Remove pragma no cover from code

Add exclude_lines in the report section of .coveragerc to ignore
abstract in test coverage

Change-Id: I7863a8ba7e20358fb7cdf3cc7e4d83871a5104ef

.coveragerc

@@ -5,3 +5,5 @@ omit = watcher/tests/*
 
 [report]
 ignore_errors = True 
+exclude_lines =
+    @abstract

watcher/applier/action_plan/base.py

@@ -25,5 +25,4 @@ import six
 class BaseActionPlanHandler(object):
     @abc.abstractmethod
     def execute(self):
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma:no cover
+        raise NotImplementedError()

watcher/applier/base.py

@@ -25,5 +25,4 @@ import six
 class BaseApplier(object):
     @abc.abstractmethod
     def execute(self, action_plan_uuid):
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma:no cover
+        raise NotImplementedError()

watcher/applier/mapping/base.py

@@ -30,5 +30,4 @@ class BaseActionMapper(object):
         :type action: watcher.decision_engine.action.BaseAction
         :return: the associated Primitive
         """
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma:no cover
+        raise NotImplementedError()

watcher/applier/primitives/base.py

@@ -26,11 +26,9 @@ class BasePrimitive(object):
     @Promise
     @abc.abstractmethod
     def execute(self):
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma:no cover
+        raise NotImplementedError()
 
     @Promise
     @abc.abstractmethod
     def undo(self):
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma:no cover
+        raise NotImplementedError()

watcher/decision_engine/messaging/command/base.py

@@ -24,5 +24,4 @@ import six
 class BaseDecisionEngineCommand(object):
     @abc.abstractmethod
     def execute(self):
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma:no cover
+        raise NotImplementedError()

watcher/decision_engine/messaging/event_consumer.py

@@ -36,4 +36,4 @@ class BaseEventConsumer(object):
 
     @abc.abstractmethod
     def execute(self, request_id, context, data):
-        raise NotImplementedError('Not implemented ...')  # pragma:no cover
+        raise NotImplementedError()

watcher/decision_engine/planner/base.py

@@ -33,5 +33,4 @@ class BasePlanner(object):
         and performance requirements are met.
         """
         # example: directed acyclic graph
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma:no cover
+        raise NotImplementedError()

watcher/decision_engine/solution/base.py

@@ -53,10 +53,8 @@ class BaseSolution(object):
 
     @abc.abstractmethod
     def add_change_request(self, r):
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma:no cover
+        raise NotImplementedError()
 
     @abc.abstractproperty
     def actions(self):
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma:no cover
+        raise NotImplementedError()

watcher/decision_engine/solution/solution_comparator.py

@@ -24,5 +24,4 @@ import six
 class BaseSolutionComparator(object):
     @abc.abstractmethod
     def compare(self, sol1, sol2):
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma:no cover
+        raise NotImplementedError()

watcher/decision_engine/solution/solution_evaluator.py

@@ -24,5 +24,4 @@ import six
 class BaseSolutionEvaluator(object):
     @abc.abstractmethod
     def evaluate(self, solution):
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma:no cover
+        raise NotImplementedError()

watcher/decision_engine/strategy/context/base.py

@@ -24,5 +24,4 @@ import six
 class BaseStrategyContext(object):
     @abc.abstractmethod
     def execute_strategy(self, model):
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma:no cover
+        raise NotImplementedError()

watcher/decision_engine/strategy/selection/base.py

@@ -24,5 +24,4 @@ import six
 class BaseSelector(object):
     @abc.abstractmethod
     def define_from_goal(self, goal_name):
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma:no cover
+        raise NotImplementedError()

watcher/metrics_engine/cluster_history/api.py

@@ -27,18 +27,14 @@ class BaseClusterHistory(object):
     @abc.abstractmethod
     def statistic_aggregation(self, resource_id, meter_name, period,
                               aggregate='avg'):
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma: nocover
+        raise NotImplementedError()
 
     @abc.abstractmethod
     def get_last_sample_values(self, resource_id, meter_name, limit=1):
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma: nocover
+        raise NotImplementedError()
 
     def query_sample(self, meter_name, query, limit=1):
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma: nocover
+        raise NotImplementedError()
 
     def statistic_list(self, meter_name, query=None, period=None):
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma: nocover
+        raise NotImplementedError()

watcher/metrics_engine/cluster_model_collector/api.py

@@ -24,5 +24,4 @@ import six
 class BaseClusterModelCollector(object):
     @abc.abstractmethod
     def get_latest_cluster_data_model(self):
-        raise NotImplementedError(
-            "Should have implemented this")  # pragma: nocover
+        raise NotImplementedError()