diff --git a/google/cloud/storage/blob.py b/google/cloud/storage/blob.py index 81e7e103f..1d283ee30 100644 --- a/google/cloud/storage/blob.py +++ b/google/cloud/storage/blob.py @@ -875,11 +875,19 @@ def _extract_headers_from_download(self, response): :class requests.models.Response :param response: The server response from downloading a non-chunked file """ - self.content_encoding = response.headers.get("Content-Encoding", None) - self.content_type = response.headers.get("Content-Type", None) - self.cache_control = response.headers.get("Cache-Control", None) - self.storage_class = response.headers.get("X-Goog-Storage-Class", None) - self.content_language = response.headers.get("Content-Language", None) + self._properties["contentEncoding"] = response.headers.get( + "Content-Encoding", None + ) + self._properties[_CONTENT_TYPE_FIELD] = response.headers.get( + "Content-Type", None + ) + self._properties["cacheControl"] = response.headers.get("Cache-Control", None) + self._properties["storageClass"] = response.headers.get( + "X-Goog-Storage-Class", None + ) + self._properties["contentLanguage"] = response.headers.get( + "Content-Language", None + ) self._properties["etag"] = response.headers.get("ETag", None) self._properties["generation"] = response.headers.get("X-goog-generation", None) self._properties["metageneration"] = response.headers.get( @@ -896,8 +904,8 @@ def _extract_headers_from_download(self, response): method, digest = match.groups() digests[method] = digest - self.crc32c = digests.get("crc32c", None) - self.md5_hash = digests.get("md5", None) + self._properties["crc32c"] = digests.get("crc32c", None) + self._properties["md5Hash"] = digests.get("md5", None) def _do_download( self, diff --git a/tests/unit/test_blob.py b/tests/unit/test_blob.py index 41934be33..f74e6e111 100644 --- a/tests/unit/test_blob.py +++ b/tests/unit/test_blob.py @@ -1090,6 +1090,7 @@ def test__extract_headers_from_download_gzipped(self): self.assertEqual(blob.etag, "kittens") self.assertEqual(blob.generation, 42) self.assertEqual(blob.metageneration, 4) + self.assertEqual(blob._changes, set()) def test__extract_headers_from_download_empty(self): blob_name = "blob-name" @@ -1120,6 +1121,7 @@ def test__extract_headers_from_download_empty(self): self.assertEqual(blob.etag, "kittens") self.assertEqual(blob.generation, 42) self.assertEqual(blob.metageneration, 4) + self.assertEqual(blob._changes, set()) def test__extract_headers_from_download_w_hash_response_header_none(self): blob_name = "blob-name"