fix: remove unused method from testing (appengine jdk 21 sample) (GoogleCloudPlatform#9584)

mpeddada1 · web-flow · commit dd24a4933092 · 2024-10-15T15:06:43.000-07:00
diff --git a/appengine-java21/helloworld/README.md b/appengine-java21/helloworld/README.md
@@ -49,13 +49,13 @@ version. Change this field to specify a specific version name.
 
     mvn package appengine:run
 
-To use vist: http://localhost:8080/
+To use visit: http://localhost:8080/
 
 ### Deploying
 
     mvn package appengine:deploy
 
-To use vist:  https://YOUR-PROJECT-ID.appspot.com
+To use visit:  https://YOUR-PROJECT-ID.appspot.com
 
 ### Testing
 
diff --git a/appengine-java21/helloworld/src/main/java/com/example/appengine/java21/HelloAppEngine.java b/appengine-java21/helloworld/src/main/java/com/example/appengine/java21/HelloAppEngine.java
@@ -38,5 +38,4 @@ public void doGet(HttpServletRequest request, HttpServletResponse response)
             + SystemProperty.version.get() + " Java "
             + properties.get("java.specification.version"));
   }
-
 }
diff --git a/appengine-java21/helloworld/src/test/java/com/example/appengine/java21/HelloAppEngineTest.java b/appengine-java21/helloworld/src/test/java/com/example/appengine/java21/HelloAppEngineTest.java
@@ -77,11 +77,4 @@ public void doGetWritesResponse() throws Exception {
     assertThat(responseWriter.toString())
         .contains("Hello App Engine - Standard ");
   }
-
-  @Test
-  public void helloInfoTest() {
-    String result = HelloAppEngine.getInfo();
-    assertThat(result)
-        .containsMatch("^Version:\\s+.+OS:\\s+.+User:\\s");
-  }
 }

Original file line number	Diff line number	Diff line change
`@@ -38,5 +38,4 @@ public void doGet(HttpServletRequest request, HttpServletResponse response)`
`38`	`38`	`+ SystemProperty.version.get() + " Java "`
`39`	`39`	`+ properties.get("java.specification.version"));`
`40`	`40`	`}`
`41`		`-`
`42`	`41`	`}`
Original file line number	Diff line number	Diff line change
`@@ -77,11 +77,4 @@ public void doGetWritesResponse() throws Exception {`
`77`	`77`	`assertThat(responseWriter.toString())`
`78`	`78`	`.contains("Hello App Engine - Standard ");`
`79`	`79`	`}`
`80`		`-`
`81`		`- @Test`
`82`		`- public void helloInfoTest() {`
`83`		`- String result = HelloAppEngine.getInfo();`
`84`		`- assertThat(result)`
`85`		`- .containsMatch("^Version:\\s+.+OS:\\s+.+User:\\s");`
`86`		`- }`
`87`	`80`	`}`