diff --git a/gcloud/storage/bucket.py b/gcloud/storage/bucket.py index e90b85c60ab6..d51225885724 100644 --- a/gcloud/storage/bucket.py +++ b/gcloud/storage/bucket.py @@ -712,7 +712,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 index e3e1683b210f..0dfe8670be28 100644 --- a/gcloud/storage/test_bucket.py +++ b/gcloud/storage/test_bucket.py @@ -885,7 +885,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) @@ -900,7 +901,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) @@ -915,7 +917,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)