Ensure that ManagedZone:exists() does not misreport True result. (googleapis#6884)

charles-salmon · tseaver · commit dfb3458c2f65 · 2018-12-14T12:11:32.000-05:00
Closes googleapis#5818.
diff --git a/dns/google/cloud/dns/zone.py b/dns/google/cloud/dns/zone.py
@@ -280,9 +280,7 @@ def exists(self, client=None):
         client = self._require_client(client)
 
         try:
-            client._connection.api_request(
-                method="GET", path=self.path, query_params={"fields": "id"}
-            )
+            client._connection.api_request(method="GET", path=self.path)
         except NotFound:
             return False
         else:
diff --git a/dns/tests/unit/test_zone.py b/dns/tests/unit/test_zone.py
@@ -330,7 +330,6 @@ def test_exists_miss_w_bound_client(self):
         req = conn._requested[0]
         self.assertEqual(req["method"], "GET")
         self.assertEqual(req["path"], "/%s" % PATH)
-        self.assertEqual(req["query_params"], {"fields": "id"})
 
     def test_exists_hit_w_alternate_client(self):
         PATH = "projects/%s/managedZones/%s" % (self.PROJECT, self.ZONE_NAME)
@@ -347,7 +346,6 @@ def test_exists_hit_w_alternate_client(self):
         req = conn2._requested[0]
         self.assertEqual(req["method"], "GET")
         self.assertEqual(req["path"], "/%s" % PATH)
-        self.assertEqual(req["query_params"], {"fields": "id"})
 
     def test_reload_w_bound_client(self):
         PATH = "projects/%s/managedZones/%s" % (self.PROJECT, self.ZONE_NAME)
