From bf32299c76325d54d5a0272edef3f109ed7bc72a Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Mon, 29 Aug 2022 20:34:57 +0000 Subject: [PATCH 1/6] chore: Update IT test case result --- .../src/test/java/com/google/cloud/vision/it/ITSystemTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java b/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java index 9ba1ee404..fe0a500e9 100644 --- a/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java +++ b/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java @@ -554,7 +554,7 @@ public void detectWebEntitiesIncludeGeoResultsGcsTest() { actual.add(entity.getDescription()); } } - assertThat(actual).contains("The Palace Of Fine Arts"); + assertThat(actual).contains("Palace Of Fine Arts"); } @Test From 22412e89ae3158b6500b37299ce2ea9cca321d08 Mon Sep 17 00:00:00 2001 From: Owl Bot Date: Mon, 29 Aug 2022 20:40:02 +0000 Subject: [PATCH 2/6] =?UTF-8?q?=F0=9F=A6=89=20Updates=20from=20OwlBot=20po?= =?UTF-8?q?st-processor?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md --- README.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/README.md b/README.md index 35199363b..a18aa250d 100644 --- a/README.md +++ b/README.md @@ -50,20 +50,20 @@ If you are using Maven without BOM, add this to your dependencies: If you are using Gradle 5.x or later, add this to your dependencies: ```Groovy -implementation platform('com.google.cloud:libraries-bom:26.0.0') +implementation platform('com.google.cloud:libraries-bom:26.1.0') implementation 'com.google.cloud:google-cloud-vision' ``` If you are using Gradle without BOM, add this to your dependencies: ```Groovy -implementation 'com.google.cloud:google-cloud-vision:3.0.1' +implementation 'com.google.cloud:google-cloud-vision:3.1.0' ``` If you are using SBT, add this to your dependencies: ```Scala -libraryDependencies += "com.google.cloud" % "google-cloud-vision" % "3.0.1" +libraryDependencies += "com.google.cloud" % "google-cloud-vision" % "3.1.0" ``` ## Authentication From 07e0b6c2749384e19cd2e40c91331103ccb8591b Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Mon, 29 Aug 2022 20:42:32 +0000 Subject: [PATCH 3/6] fix: Update detectWebEntitiesGcsTest test case --- .../src/test/java/com/google/cloud/vision/it/ITSystemTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java b/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java index fe0a500e9..6647b9c4d 100644 --- a/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java +++ b/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java @@ -492,7 +492,7 @@ public void detectWebEntitiesGcsTest() throws IOException { actual.add(entity.getDescription()); } } - assertThat(actual).contains("The Palace Of Fine Arts"); + assertThat(actual).contains("Palace Of Fine Arts"); } @Test From f435d51be9ade56bc7104aa8b5aa2584ab81b82b Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Mon, 29 Aug 2022 21:37:44 +0000 Subject: [PATCH 4/6] chore: Add logging for arrayList.contains: --- .../test/java/com/google/cloud/vision/it/ITSystemTest.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java b/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java index 6647b9c4d..fc5352716 100644 --- a/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java +++ b/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java @@ -473,7 +473,7 @@ public void detectSafeSearchGcsTest() throws IOException { } @Test - public void detectWebEntitiesGcsTest() throws IOException { + public void detectWebEntitiesGcsTest() { ImageSource imgSource = ImageSource.newBuilder().setGcsImageUri(SAMPLE_BUCKET + "landmark/pofa.jpg").build(); Image img = Image.newBuilder().setSource(imgSource).build(); @@ -492,6 +492,7 @@ public void detectWebEntitiesGcsTest() throws IOException { actual.add(entity.getDescription()); } } + System.out.println(actual.contains("Palace of Fine Arts")); assertThat(actual).contains("Palace Of Fine Arts"); } @@ -547,13 +548,12 @@ public void detectWebEntitiesIncludeGeoResultsGcsTest() { imageAnnotatorClient.batchAnnotateImages(ImmutableList.of(request)); List responses = response.getResponsesList(); List actual = new ArrayList<>(); - System.out.println("WebEntitiesGeo SIZE"); - System.out.println(actual.size()); for (AnnotateImageResponse imgResponse : responses) { for (WebDetection.WebEntity entity : imgResponse.getWebDetection().getWebEntitiesList()) { actual.add(entity.getDescription()); } } + System.out.println(actual.contains("Palace of Fine Arts")); assertThat(actual).contains("Palace Of Fine Arts"); } From 24ae224d1e5514468a7741e6ca3f8c6c34039c7e Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Mon, 29 Aug 2022 21:58:35 +0000 Subject: [PATCH 5/6] chore: Update text --- .../test/java/com/google/cloud/vision/it/ITSystemTest.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java b/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java index fc5352716..760d46b0c 100644 --- a/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java +++ b/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java @@ -493,7 +493,7 @@ public void detectWebEntitiesGcsTest() { } } System.out.println(actual.contains("Palace of Fine Arts")); - assertThat(actual).contains("Palace Of Fine Arts"); + assertThat(actual).contains("Palace of Fine Arts"); } @Test @@ -554,7 +554,7 @@ public void detectWebEntitiesIncludeGeoResultsGcsTest() { } } System.out.println(actual.contains("Palace of Fine Arts")); - assertThat(actual).contains("Palace Of Fine Arts"); + assertThat(actual).contains("Palace of Fine Arts"); } @Test From 0f0913c9f3873c0b15f78a79808dd569f785e5ad Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Mon, 29 Aug 2022 22:07:06 +0000 Subject: [PATCH 6/6] chore: Remomve logging for arraylist contains --- .../src/test/java/com/google/cloud/vision/it/ITSystemTest.java | 2 -- 1 file changed, 2 deletions(-) diff --git a/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java b/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java index 760d46b0c..517fc799a 100644 --- a/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java +++ b/google-cloud-vision/src/test/java/com/google/cloud/vision/it/ITSystemTest.java @@ -492,7 +492,6 @@ public void detectWebEntitiesGcsTest() { actual.add(entity.getDescription()); } } - System.out.println(actual.contains("Palace of Fine Arts")); assertThat(actual).contains("Palace of Fine Arts"); } @@ -553,7 +552,6 @@ public void detectWebEntitiesIncludeGeoResultsGcsTest() { actual.add(entity.getDescription()); } } - System.out.println(actual.contains("Palace of Fine Arts")); assertThat(actual).contains("Palace of Fine Arts"); }