Skip to content

Commit 685d26f

Browse files
committed
Fix checkstyle issues with landmarkdetection.
1 parent 7b69a2d commit 685d26f

File tree

2 files changed

+12
-14
lines changed

2 files changed

+12
-14
lines changed

vision/landmark-detection/src/main/java/com/google/cloud/vision/samples/landmarkdetection/DetectLandmark.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
package com.google.cloud.vision.samples.landmarkdetection;
1818

1919
// [START import_libraries]
20+
2021
import com.google.api.client.googleapis.auth.oauth2.GoogleCredential;
2122
import com.google.api.client.googleapis.javanet.GoogleNetHttpTransport;
2223
import com.google.api.client.json.JsonFactory;
@@ -32,7 +33,6 @@
3233
import com.google.api.services.vision.v1.model.Image;
3334
import com.google.api.services.vision.v1.model.ImageSource;
3435
import com.google.common.collect.ImmutableList;
35-
3636
import java.io.IOException;
3737
import java.security.GeneralSecurityException;
3838
import java.util.List;

vision/landmark-detection/src/test/java/com/google/cloud/vision/samples/landmarkdetection/DetectLandmarkIT.java

+11-13
Original file line numberDiff line numberDiff line change
@@ -20,15 +20,13 @@
2020
import static org.junit.Assert.fail;
2121

2222
import com.google.api.services.vision.v1.model.EntityAnnotation;
23-
23+
import java.io.IOException;
24+
import java.util.List;
2425
import org.junit.Before;
2526
import org.junit.Test;
2627
import org.junit.runner.RunWith;
2728
import org.junit.runners.JUnit4;
2829

29-
import java.io.IOException;
30-
import java.util.List;
31-
3230
/**
3331
* Integration (system) tests for {@link DetectLandmark}.
3432
**/
@@ -66,13 +64,13 @@ public class DetectLandmarkIT {
6664
}
6765
}
6866

69-
// TODO(lesv): Currently we can access it, need to set better attributes.
70-
// @Test public void identifyLandmark_noImage_throwsForbidden() throws Exception {
71-
// try {
72-
// appUnderTest.identifyLandmark(PRIVATE_LANDMARK_URI, MAX_RESULTS);
73-
// fail("Expected IOException");
74-
// } catch (IOException expected) {
75-
// assertThat(expected.getMessage()).named("IOException message").contains("permission");
76-
// }
77-
// }
67+
// TODO(lesv): Currently we can access it, need to set better attributes.
68+
// @Test public void identifyLandmark_noImage_throwsForbidden() throws Exception {
69+
// try {
70+
// appUnderTest.identifyLandmark(PRIVATE_LANDMARK_URI, MAX_RESULTS);
71+
// fail("Expected IOException");
72+
// } catch (IOException expected) {
73+
// assertThat(expected.getMessage()).named("IOException message").contains("permission");
74+
// }
75+
// }
7876
}

0 commit comments

Comments
 (0)