Drop 'pragma: NO COVER' for '__repr__' methods.

tseaver · tseaver · commit a3737e67162b · 2014-10-16T16:29:21.000-04:00
Suppress coverage for them instead via .coveragerc.
diff --git a/.coveragerc b/.coveragerc
@@ -2,3 +2,8 @@
 omit =
 	*/demo/*
 	*/demo.py
+exclude_lines =
+    # Re-enable the standard pragma
+    pragma: NO COVER
+    # Ignore debug-only repr
+    def __repr__
diff --git a/gcloud/datastore/entity.py b/gcloud/datastore/entity.py
@@ -230,7 +230,7 @@ def delete(self):
             dataset_id=self.dataset().id(), key_pb=self.key().to_protobuf())
         # pylint: enable=maybe-no-member
 
-    def __repr__(self):  # pragma NO COVER
+    def __repr__(self):
         # An entity should have a key all the time (even if it's partial).
         if self.key():
             # pylint: disable=maybe-no-member
diff --git a/gcloud/datastore/key.py b/gcloud/datastore/key.py
@@ -286,5 +286,5 @@ def parent(self):
             return None
         return self.path(self.path()[:-1])
 
-    def __repr__(self):  # pragma NO COVER
+    def __repr__(self):
         return '<Key%s>' % self.path()
diff --git a/gcloud/storage/acl.py b/gcloud/storage/acl.py
@@ -110,7 +110,7 @@ def __str__(self):
             else:
                 return '{self.type}-{self.identifier}'.format(self=self)
 
-        def __repr__(self):  # pragma NO COVER
+        def __repr__(self):
             return '<ACL Entity: {self} ({roles})>'.format(
                 self=self, roles=', '.join(self.roles))
 
diff --git a/gcloud/storage/bucket.py b/gcloud/storage/bucket.py
@@ -42,7 +42,7 @@ def from_dict(cls, bucket_dict, connection=None):
         return cls(connection=connection, name=bucket_dict['name'],
                    metadata=bucket_dict)
 
-    def __repr__(self):  # pragma NO COVER
+    def __repr__(self):
         return '<Bucket: %s>' % self.name
 
     def __iter__(self):
@@ -126,7 +126,7 @@ def new_key(self, key):
         # Support Python 2 and 3.
         try:
             string_type = basestring
-        except NameError:  # pragma NO COVER PY3k
+        except NameError:  # pragma: NO COVER PY3k
             string_type = str
 
         if isinstance(key, string_type):
diff --git a/gcloud/storage/connection.py b/gcloud/storage/connection.py
@@ -26,7 +26,7 @@ def _utcnow():
     """
     if _UTCNOW is not None:
         return _UTCNOW
-    return datetime.datetime.utcnow()  # pragma NO COVER
+    return datetime.datetime.utcnow()  # pragma: NO COVER
 
 
 class Connection(connection.Connection):
@@ -430,7 +430,7 @@ def new_bucket(self, bucket):
         # Support Python 2 and 3.
         try:
             string_type = basestring
-        except NameError:  # pragma NO COVER PY3k
+        except NameError:  # pragma: NO COVER PY3k
             string_type = str
 
         if isinstance(bucket, string_type):
diff --git a/gcloud/storage/key.py b/gcloud/storage/key.py
@@ -58,7 +58,7 @@ def from_dict(cls, key_dict, bucket=None):
 
         return cls(bucket=bucket, name=key_dict['name'], metadata=key_dict)
 
-    def __repr__(self):  # pragma NO COVER
+    def __repr__(self):
         if self.bucket:
             bucket_name = self.bucket.name
         else:
