Skip to content

fix(test): update blob paths used in storage.it.ITStorageTest#testDownloadPublicBlobWithoutAuthentication - #759

Merged
BenWhitehead merged 1 commit into
googleapis:masterfrom
BenWhitehead:fix-755
Mar 16, 2021
Merged

fix(test): update blob paths used in storage.it.ITStorageTest#testDownloadPublicBlobWithoutAuthentication#759
BenWhitehead merged 1 commit into
googleapis:masterfrom
BenWhitehead:fix-755

Conversation

@BenWhitehead

Copy link
Copy Markdown
Collaborator

@BenWhitehead
BenWhitehead requested a review from a teamMarch 16, 2021 14:52
@product-auto-labelproduct-auto-labelBot added the api: storage Issues related to the googleapis/java-storage API. label Mar 16, 2021
@google-clagoogle-claBot added the cla: yes This human has signed the Contributor License Agreement. label Mar 16, 2021
@codecov

codecovBot commented Mar 16, 2021

Copy link
Copy Markdown

Codecov Report

Merging #759 (d43c244) into master (6cd04d1) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## master #759 +/- ##
=========================================
Coverage 64.61% 64.61% Complexity 634 634 =========================================
Files 32 32 Lines 5313 5313 Branches 519 519 =========================================
Hits 3433 3433 Misses 1718 1718 Partials 162 162 

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6cd04d1...d43c244. Read the comment docs.

@BenWhitehead
BenWhitehead merged commit 9a6619c into googleapis:masterMar 16, 2021
@BenWhitehead
BenWhitehead deleted the fix-755 branch March 16, 2021 15:15
gcf-merge-on-greenBot pushed a commit that referenced this pull request Apr 13, 2021
🤖 I have created a release \*beep\* \*boop\*
---
### [1.113.15](https://www.github.com/googleapis/java-storage/compare/v1.113.14...v1.113.15) (2021-04-13)
### Bug Fixes
* **test:** update blob paths used in storage.it.ITStorageTest#testDownloadPublicBlobWithoutAuthentication ([#759](https://www.github.com/googleapis/java-storage/issues/759)) ([9a6619c](https://www.github.com/googleapis/java-storage/commit/9a6619c39a89e2c2ee8d0000d595d09ac7b7825f))
* typo ([#779](https://www.github.com/googleapis/java-storage/issues/779)) ([3c3d6b4](https://www.github.com/googleapis/java-storage/commit/3c3d6b487648fde4eb956ce8912cd680a4440f8d))
### Dependencies
* update dependency com.google.api-client:google-api-client to v1.31.4 ([#774](https://www.github.com/googleapis/java-storage/issues/774)) ([ad9ff7b](https://www.github.com/googleapis/java-storage/commit/ad9ff7b801d0c5fb39f72c7118c319f4e45084a0))
* update dependency com.google.cloud:google-cloud-shared-dependencies to v0.21.0 ([#771](https://www.github.com/googleapis/java-storage/issues/771)) ([5605095](https://www.github.com/googleapis/java-storage/commit/5605095ed796327879a930c12526b3c5b1409b17))
* update kms.version to v0.88.1 ([#758](https://www.github.com/googleapis/java-storage/issues/758)) ([3e57ea9](https://www.github.com/googleapis/java-storage/commit/3e57ea9a2f5f7013e997469c5ca32be8cef2a4a4))
* update kms.version to v0.88.2 ([#778](https://www.github.com/googleapis/java-storage/issues/778)) ([6edfc4c](https://www.github.com/googleapis/java-storage/commit/6edfc4ced2bdae9878ecdbc5ef636ac39bdb5881))
* update truth ([#767](https://www.github.com/googleapis/java-storage/issues/767)) ([4e5ee03](https://www.github.com/googleapis/java-storage/commit/4e5ee0398e700baf4f88224f66309e426f9532d7))
---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
BenWhitehead added a commit that referenced this pull request May 11, 2021
BenWhitehead added a commit that referenced this pull request May 11, 2021
gcf-merge-on-greenBot pushed a commit that referenced this pull request May 13, 2021
🤖 I have created a release \*beep\* \*boop\*
---
## [1.114.0](https://www.github.com/googleapis/java-storage/compare/v1.113.16...v1.114.0) (2021-05-13)
### Features
* Remove client side vaildation for lifecycle conditions ([#816](https://www.github.com/googleapis/java-storage/issues/816)) ([5ec84cc](https://www.github.com/googleapis/java-storage/commit/5ec84cc2935a4787dd14a207d27501878f5849d5))
### Bug Fixes
* **test:** update blob paths used in storage.it.ITStorageTest#testDownloadPublicBlobWithoutAuthentication ([#759](https://www.github.com/googleapis/java-storage/issues/759)) ([#817](https://www.github.com/googleapis/java-storage/issues/817)) ([1a576ca](https://www.github.com/googleapis/java-storage/commit/1a576ca3945b51d7a678aa2414be91b3c6b2d55e))
### Dependencies
* update dependency com.google.api-client:google-api-client to v1.31.5 ([#820](https://www.github.com/googleapis/java-storage/issues/820)) ([9e1bc0b](https://www.github.com/googleapis/java-storage/commit/9e1bc0b42abdaab0b11d761ecdbb92f6116aacd2))
* update dependency com.google.api.grpc:grpc-google-cloud-kms-v1 to v0.88.3 ([#797](https://www.github.com/googleapis/java-storage/issues/797)) ([747e7e4](https://www.github.com/googleapis/java-storage/commit/747e7e463c028b9cf8a406b7536b1916c1d52c01))
* update dependency com.google.cloud:google-cloud-shared-dependencies to v1.1.0 ([#815](https://www.github.com/googleapis/java-storage/issues/815)) ([e210de9](https://www.github.com/googleapis/java-storage/commit/e210de93452243242be7d3d719d00da723632335))
---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
danielduhh pushed a commit to danielduhh/java-storage that referenced this pull request Jun 9, 2021
danielduhh added a commit that referenced this pull request Jun 10, 2021
* build: add kokoro & ci changes
* add samples folder to pass ci
* build: pin fmt-maven-plugin version to 2.9
fmt-maven-plugin v2.10 is compiled for java 11, v2.9 is compiled for java 8
port of fix from googleapis/java-shared-config#90
* remove samples directory & ci
* add ci block
* remove snippet bot config
* sync ci changes
* fix(test): update blob paths used in storage.it.ITStorageTest#testDownloadPublicBlobWithoutAuthentication (#759)
Port of googleapis/google-cloud-go#3806Fixes#755
Co-authored-by: BenWhitehead <BenWhitehead@users.noreply.github.com>
danielduhh pushed a commit that referenced this pull request Jun 17, 2021
danielduhh added a commit that referenced this pull request Jun 17, 2021
* feat: configure initial sp version
* fix(test): update blob paths used in storage.it.ITStorageTest#testDownloadPublicBlobWithoutAuthentication (#759)
Port of googleapis/google-cloud-go#3806Fixes#755
* test: remove error string matching (#861)
It looks like the text for this error on the backend has changed
(sometimes) from "Precondition Failed" to "At least one of the
pre-conditions you specified did not hold". I don't think it's
really necessary to check the exact message in any case given
that we do check for a code of 412, which implies a precondition
failure. I added a check of the error Reason instead, which is more
standardized.
Fixes#853
Co-authored-by: BenWhitehead <BenWhitehead@users.noreply.github.com>
Co-authored-by: Chris Cotter <cjcotter@google.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storageIssues related to the googleapis/java-storage API.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

storage.it.ITStorageTest: testDownloadPublicBlobWithoutAuthentication failed

2 participants

@BenWhitehead@tritone