fix: remove unused method from testing (appengine jdk 21 sample) (Goo…

…gleCloudPlatform#9584)
invertase · Oct 15, 2024 · dd24a49 · dd24a49
1 parent 7792ddb
commit dd24a49
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 10 deletions.
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/...gine-java21/helloworld/src/test/java/com/example/appengine/java21/HelloAppEngineTest.java b/...gine-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 change
@@ Expand Up @@
                 + SystemProperty.version.get() + " Java "
                 + properties.get("java.specification.version"));
       }
     }