Skip to content
This repository has been archived by the owner on Sep 16, 2023. It is now read-only.

fix: Update IT Test Case Results #976

Merged
merged 7 commits into from
Aug 30, 2022
Merged

fix: Update IT Test Case Results #976

merged 7 commits into from
Aug 30, 2022

Conversation

lqiu96
Copy link
Contributor

@lqiu96 lqiu96 commented Aug 29, 2022

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #968 and #967 ☕️

If you write sample code, please follow the samples format.

@lqiu96 lqiu96 requested a review from a team as a code owner August 29, 2022 20:36
@product-auto-label product-auto-label bot added size: xs Pull request size is extra small. api: vision Issues related to the googleapis/java-vision API. labels Aug 29, 2022
@lqiu96 lqiu96 changed the title chore: Update IT test case result fix: Update IT Test Case Results Aug 29, 2022
@product-auto-label product-auto-label bot added size: s Pull request size is small. and removed size: xs Pull request size is extra small. labels Aug 29, 2022
@lqiu96 lqiu96 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 29, 2022
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Aug 29, 2022
@lqiu96
Copy link
Contributor Author

lqiu96 commented Aug 29, 2022

IT logs with error:

[INFO] Running com.google.cloud.vision.it.ITSystemTest
WebEntitiesGeo SIZE
0
[ERROR] Tests run: 33, Failures: 2, Errors: 0, Skipped: 0, Time elapsed: 20.672 s <<< FAILURE! - in com.google.cloud.vision.it.ITSystemTest
[ERROR] com.google.cloud.vision.it.ITSystemTest.detectWebEntitiesGcsTest  Time elapsed: 3.052 s  <<< FAILURE!
expected to contain: Palace Of Fine Arts
but was            : [Ancient Rome, Ancient history, Triumphal arch, Ancient Roman architecture, Palace of Fine Arts, History, Architecture, Historic site, Façade, Arch, Tourism, Hematocrit]
	at com.google.cloud.vision.it.ITSystemTest.detectWebEntitiesGcsTest(ITSystemTest.java:495)

[ERROR] com.google.cloud.vision.it.ITSystemTest.detectWebEntitiesIncludeGeoResultsGcsTest  Time elapsed: 1.607 s  <<< FAILURE!
expected to contain: Palace Of Fine Arts
but was            : [Ancient Rome, Triumphal arch, Ancient history, Ancient Roman architecture, Palace of Fine Arts, History, Historic site, Architecture, Façade, Arch, Tourism, Hematocrit]
	at com.google.cloud.vision.it.ITSystemTest.detectWebEntitiesIncludeGeoResultsGcsTest(ITSystemTest.java:557)

@lqiu96
Copy link
Contributor Author

lqiu96 commented Aug 29, 2022

Issue was

Palace Of Fine Arts vs Palace of Fine Arts

Of was capitalized

@lqiu96 lqiu96 requested a review from suztomo August 29, 2022 22:16
@lqiu96 lqiu96 added the automerge Merge the pull request once unit tests and other checks pass. label Aug 30, 2022
@gcf-merge-on-green gcf-merge-on-green bot merged commit 3272c49 into main Aug 30, 2022
@gcf-merge-on-green gcf-merge-on-green bot deleted the update_IT branch August 30, 2022 13:42
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Aug 30, 2022
gcf-merge-on-green bot pushed a commit that referenced this pull request Sep 16, 2022
🤖 I have created a release *beep* *boop*
---


## [3.1.1](v3.1.0...v3.1.1) (2022-09-16)


### Bug Fixes

* Update IT Test Case Results ([#976](#976)) ([3272c49](3272c49))


### Dependencies

* Update dependency com.google.cloud:google-cloud-core to v2.8.10 ([#977](#977)) ([579608b](579608b))
* Update dependency com.google.cloud:google-cloud-core to v2.8.11 ([#980](#980)) ([fe12d4e](fe12d4e))
* Update dependency com.google.cloud:google-cloud-core to v2.8.9 ([#973](#973)) ([167bbab](167bbab))
* Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.2 ([#981](#981)) ([ad1e601](ad1e601))
* Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.3 ([#985](#985)) ([22b5933](22b5933))
* Update dependency com.google.cloud:google-cloud-storage to v2.11.3 ([#964](#964)) ([eefc2b7](eefc2b7))
* Update spring.version to v2.7.3 ([#974](#974)) ([2a370a9](2a370a9))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
github-actions bot pushed a commit to meltsufin/monorepo-experiments that referenced this pull request Sep 17, 2022
🤖 I have created a release *beep* *boop*
---


## [3.1.1](googleapis/java-vision@v3.1.0...v3.1.1) (2022-09-16)


### Bug Fixes

* Update IT Test Case Results ([#976](googleapis/java-vision#976)) ([eb7372d](googleapis/java-vision@eb7372d))


### Dependencies

* Update dependency com.google.cloud:google-cloud-core to v2.8.10 ([#977](googleapis/java-vision#977)) ([fe3cb82](googleapis/java-vision@fe3cb82))
* Update dependency com.google.cloud:google-cloud-core to v2.8.11 ([#980](googleapis/java-vision#980)) ([915af18](googleapis/java-vision@915af18))
* Update dependency com.google.cloud:google-cloud-core to v2.8.9 ([#973](googleapis/java-vision#973)) ([9b857a6](googleapis/java-vision@9b857a6))
* Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.2 ([#981](googleapis/java-vision#981)) ([8dd78c8](googleapis/java-vision@8dd78c8))
* Update dependency com.google.cloud:google-cloud-shared-dependencies to v3.0.3 ([#985](googleapis/java-vision#985)) ([7873226](googleapis/java-vision@7873226))
* Update dependency com.google.cloud:google-cloud-storage to v2.11.3 ([#964](googleapis/java-vision#964)) ([726c31e](googleapis/java-vision@726c31e))
* Update spring.version to v2.7.3 ([#974](googleapis/java-vision#974)) ([d3aac93](googleapis/java-vision@d3aac93))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: vision Issues related to the googleapis/java-vision API. size: s Pull request size is small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

vision.it.ITSystemTest: detectWebEntitiesIncludeGeoResultsGcsTest failed
3 participants