Issue jcabi#174 - Implementing pinaf's suggestions

pecko · Sep 30, 2015 · 68e8a4a · 68e8a4a
1 parent 7b4769b
commit 68e8a4a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/test/java/com/jcabi/aspects/ImmutableTest.java b/src/test/java/com/jcabi/aspects/ImmutableTest.java
@@ -99,10 +99,10 @@ public void catchesTypesMutableByClassInheritance() {
     }
 
     /**
-     * Test Version.
+     * Informs version on error.
      */
     @Test
-    public void testVersion() {
+    public void informsVersionOnError() {
         this.thrown.expect(IllegalStateException.class);
         this.thrown.expectMessage(Version.CURRENT.projectVersion());
         this.thrown.expectMessage(Version.CURRENT.buildNumber());

diff --git a/src/test/java/com/jcabi/aspects/aj/NamedThreadsTest.java b/src/test/java/com/jcabi/aspects/aj/NamedThreadsTest.java
@@ -54,7 +54,7 @@ public final class NamedThreadsTest {
     @SuppressWarnings("PMD.DoNotUseThreads")
     public void testVersion() {
         final org.apache.log4j.Logger root = LogManager.getRootLogger();
-        final Level oldLevel = root.getLevel();
+        final Level level = root.getLevel();
         root.setLevel(Level.INFO);
         final StringWriter writer = new StringWriter();
         final WriterAppender appender =
@@ -84,7 +84,7 @@ public void run() {
             );
         } finally {
             root.removeAppender(appender);
-            root.setLevel(oldLevel);
+            root.setLevel(level);
         }
     }