Removing patch() from Bucket.configure_website.

Relates to #728.
googleapis · Mar 28, 2015 · 7cf10dd · 7cf10dd
1 parent 4d69bb3
commit 7cf10dd
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/gcloud/storage/bucket.py b/gcloud/storage/bucket.py
@@ -713,7 +713,6 @@ def configure_website(self, main_page_suffix=None, not_found_page=None):
             },
         }
         self._patch_properties(data)
-        return self.patch()
 
     def disable_website(self):
         """Disable the website configuration for this bucket.

diff --git a/gcloud/storage/test_bucket.py b/gcloud/storage/test_bucket.py
@@ -932,7 +932,8 @@ def test_configure_website_defaults(self):
                                'notFoundPage': None}}
         connection = _Connection(patched)
         bucket = self._makeOne(NAME, connection)
-        self.assertTrue(bucket.configure_website() is bucket)
+        bucket.configure_website()
+        bucket.patch()
         self.assertEqual(bucket.properties, patched)
         kw = connection._requested
         self.assertEqual(len(kw), 1)
@@ -947,7 +948,8 @@ def test_configure_website_explicit(self):
                                'notFoundPage': '404.html'}}
         connection = _Connection(patched)
         bucket = self._makeOne(NAME, connection)
-        self.assertTrue(bucket.configure_website('html', '404.html') is bucket)
+        bucket.configure_website('html', '404.html')
+        bucket.patch()
         self.assertEqual(bucket.properties, patched)
         kw = connection._requested
         self.assertEqual(len(kw), 1)
@@ -962,7 +964,8 @@ def test_disable_website(self):
                                'notFoundPage': None}}
         connection = _Connection(patched)
         bucket = self._makeOne(NAME, connection)
-        self.assertTrue(bucket.disable_website() is bucket)
+        bucket.disable_website()
+        bucket.patch()
         self.assertEqual(bucket.properties, patched)
         kw = connection._requested
         self.assertEqual(len(kw), 1)