Removed duplicated literals

yegor256 · Jun 26, 2017 · 9a7c524 · 9a7c524
1 parent 3af1273
commit 9a7c524
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/src/test/java/org/cactoos/list/MapEntryTest.java b/src/test/java/org/cactoos/list/MapEntryTest.java
@@ -38,22 +38,24 @@
 public final class MapEntryTest {
 
     @Test
-    @SuppressWarnings("PMD.AvoidDuplicateLiterals")
     public void getKey() {
+        final String key = "hello";
+        final String value = "world";
         MatcherAssert.assertThat(
             "Can't get key in the map entry",
-            new MapEntry<>("hello", "world").getKey(),
-            Matchers.equalTo("hello")
+            new MapEntry<>(key, value).getKey(),
+            Matchers.equalTo(key)
         );
     }
 
     @Test
-    @SuppressWarnings("PMD.AvoidDuplicateLiterals")
     public void getValue() {
+        final String key = "foo";
+        final String value = "bar";
         MatcherAssert.assertThat(
             "Can't get value in the map entry",
-            new MapEntry<>("foo", "bar").getValue(),
-            Matchers.equalTo("bar")
+            new MapEntry<>(key, value).getValue(),
+            Matchers.equalTo(value)
         );
     }
 
@@ -63,11 +65,12 @@ public void cantSetValue() {
     }
 
     @Test
-    @SuppressWarnings("PMD.AvoidDuplicateLiterals")
     public void equalsTo() {
+        final String key = "eo";
+        final String value = "book";
         MatcherAssert.assertThat(
             "MapEntries are not equals",
-            new MapEntry<>("eo", "book").equals(new MapEntry<>("eo", "book")),
+            new MapEntry<>(key, value).equals(new MapEntry<>(key, value)),
             Matchers.equalTo(true)
         );
     }