SupressWarnings correctly (#824)

GoogleCloudPlatform · Aug 23, 2017 · e6d84cd · e6d84cd
1 parent 0e81fc5
commit e6d84cd
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/appengine-java8/datastore/src/test/java/com/example/appengine/EntitiesTest.java b/appengine-java8/datastore/src/test/java/com/example/appengine/EntitiesTest.java
@@ -218,7 +218,7 @@ public void repeatedProperties_storesList() throws Exception {
     assertThat(retrievedFruits).containsExactlyElementsIn(favoriteFruit).inOrder();
   }
 
-  @SuppressWarnings(VariableDeclarationUsageDistance)
+  @SuppressWarnings("VariableDeclarationUsageDistance")
   @Test
   public void embeddedEntity_fromEmbedded_embedsProperties() throws Exception {
     Entity employee = new Entity("Employee");

diff --git a/appengine-java8/datastore/src/test/java/com/example/appengine/TransactionsTest.java b/appengine-java8/datastore/src/test/java/com/example/appengine/TransactionsTest.java
@@ -140,7 +140,6 @@ public void entityGroups() throws Exception {
     }
   }
 
-  @SuppressWarnings(VariableDeclarationUsageDistance)
   @Test
   public void creatingAnEntityInASpecificEntityGroup() throws Exception {
     String boardName = "my-message-board";
@@ -152,14 +151,14 @@ public void creatingAnEntityInASpecificEntityGroup() throws Exception {
     String messageText = "Some message.";
     Date postDate = new Date();
 
-    Transaction txn = datastore.beginTransaction();
-
     Key messageBoardKey = KeyFactory.createKey("MessageBoard", boardName);
 
     Entity message = new Entity("Message", messageBoardKey);
     message.setProperty("message_title", messageTitle);
     message.setProperty("message_text", messageText);
     message.setProperty("post_date", postDate);
+
+    Transaction txn = datastore.beginTransaction();
     datastore.put(txn, message);
 
     txn.commit();