Code Review / yangtools.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | review | tree
raw | inline | side by side
Fix eclipse/checkstyle warnings
[yangtools.git] / yang / yang-common / src / test / java / org / opendaylight / yangtools / yang / common / QNameTest.java
diff --git a/yang/yang-common/src/test/java/org/opendaylight/yangtools/yang/common/QNameTest.java b/yang/yang-common/src/test/java/org/opendaylight/yangtools/yang/common/QNameTest.java
index e6a279b32925dd536b07d09173330d74b7127379..56bc0bc629541807022210859796243b8c22b202 100644 (file)
--- a/yang/yang-common/src/test/java/org/opendaylight/yangtools/yang/common/QNameTest.java
+++ b/yang/yang-common/src/test/java/org/opendaylight/yangtools/yang/common/QNameTest.java
@@ -11,41 +11,40 @@ import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assert.fail;
+
 import java.net.URI;
 import java.util.Date;
 import org.junit.Test;
 
 public class QNameTest {
-    private final String namespace = "urn:foo", revision = "2013-12-24", localName = "bar";
-    private final URI ns;
-
-    public QNameTest() throws Exception {
-        this.ns = new URI(namespace);
-    }
+    private final String namespace = "urn:foo";
+    private final String revision = "2013-12-24";
+    private final String localName = "bar";
+    private final URI ns = URI.create(namespace);
 
     @Test
     public void testStringSerialization() throws Exception {
         {
-            QName qName = QName.create(namespace, revision, localName);
+            QName qname = QName.create(namespace, revision, localName);
             assertEquals(QName.QNAME_LEFT_PARENTHESIS + namespace + QName.QNAME_REVISION_DELIMITER
-                    + revision + QName.QNAME_RIGHT_PARENTHESIS + localName, qName.toString());
-            QName copied = QName.create(qName.toString());
-            assertEquals(qName, copied);
+                    + revision + QName.QNAME_RIGHT_PARENTHESIS + localName, qname.toString());
+            QName copied = QName.create(qname.toString());
+            assertEquals(qname, copied);
         }
         // no revision
         {
-            QName qName = new QName(ns, localName);
+            QName qname = new QName(ns, localName);
             assertEquals(QName.QNAME_LEFT_PARENTHESIS + namespace + QName.QNAME_RIGHT_PARENTHESIS
-                    + localName, qName.toString());
-            QName copied = QName.create(qName.toString());
-            assertEquals(qName, copied);
+                    + localName, qname.toString());
+            QName copied = QName.create(qname.toString());
+            assertEquals(qname, copied);
         }
         // no namespace nor revision
         {
-            QName qName = new QName(null, localName);
-            assertEquals(localName, qName.toString());
-            QName copied = QName.create(qName.toString());
-            assertEquals(qName, copied);
+            QName qname = new QName(null, localName);
+            assertEquals(localName, qname.toString());
+            QName copied = QName.create(qname.toString());
+            assertEquals(qname, copied);
         }
     }
 
@@ -61,73 +60,73 @@ public class QNameTest {
 
     @Test
     public void testCompareTo() throws Exception {
-        String A = "a";
-        String B = "b";
+        final String A = "a";
+        final String B = "b";
 
-        QName a = QName.create(A);
-        QName b = QName.create(A);
-        assertTrue(a.compareTo(b) == 0);
-        assertTrue(b.compareTo(a) == 0);
+        QName qa = QName.create(A);
+        QName qb = QName.create(A);
+        assertTrue(qa.compareTo(qb) == 0);
+        assertTrue(qb.compareTo(qa) == 0);
 
         // compare with localName
-        a = QName.create(A);
-        b = QName.create(B);
-        assertTrue(a.compareTo(b) < 0);
-        assertTrue(b.compareTo(a) > 0);
+        qa = QName.create(A);
+        qb = QName.create(B);
+        assertTrue(qa.compareTo(qb) < 0);
+        assertTrue(qb.compareTo(qa) > 0);
 
         // compare with namespace
-        a = QName.create(A, revision, A);
-        b = QName.create(B, revision, A);
-        assertTrue(a.compareTo(b) < 0);
-        assertTrue(b.compareTo(a) > 0);
+        qa = QName.create(A, revision, A);
+        qb = QName.create(B, revision, A);
+        assertTrue(qa.compareTo(qb) < 0);
+        assertTrue(qb.compareTo(qa) > 0);
 
         // compare with 1 null namespace
-        a = QName.create(null, QName.parseRevision(revision), A);
-        b = QName.create(URI.create(A), QName.parseRevision(revision), A);
-        assertTrue(a.compareTo(b) < 0);
-        assertTrue(b.compareTo(a) > 0);
+        qa = QName.create(null, QName.parseRevision(revision), A);
+        qb = QName.create(URI.create(A), QName.parseRevision(revision), A);
+        assertTrue(qa.compareTo(qb) < 0);
+        assertTrue(qb.compareTo(qa) > 0);
 
         // compare with both null namespace
-        b = QName.create(null, QName.parseRevision(revision), A);
-        assertTrue(a.compareTo(b) == 0);
-        assertTrue(b.compareTo(a) == 0);
+        qb = QName.create(null, QName.parseRevision(revision), A);
+        assertTrue(qa.compareTo(qb) == 0);
+        assertTrue(qb.compareTo(qa) == 0);
 
         // compare with revision
-        a = QName.create(A, "2013-12-24", A);
-        b = QName.create(A, "2013-12-25", A);
-        assertTrue(a.compareTo(b) < 0);
-        assertTrue(b.compareTo(a) > 0);
+        qa = QName.create(A, "2013-12-24", A);
+        qb = QName.create(A, "2013-12-25", A);
+        assertTrue(qa.compareTo(qb) < 0);
+        assertTrue(qb.compareTo(qa) > 0);
 
         // compare with 1 null revision
-        a = QName.create(URI.create(A), null, A);
-        b = QName.create(URI.create(A), QName.parseRevision(revision), A);
-        assertTrue(a.compareTo(b) < 0);
-        assertTrue(b.compareTo(a) > 0);
+        qa = QName.create(URI.create(A), null, A);
+        qb = QName.create(URI.create(A), QName.parseRevision(revision), A);
+        assertTrue(qa.compareTo(qb) < 0);
+        assertTrue(qb.compareTo(qa) > 0);
 
         // compare with both null revision
-        b = QName.create(URI.create(A), null, A);
-        assertTrue(a.compareTo(b) == 0);
-        assertTrue(b.compareTo(a) == 0);
+        qb = QName.create(URI.create(A), null, A);
+        assertTrue(qa.compareTo(qb) == 0);
+        assertTrue(qb.compareTo(qa) == 0);
     }
 
     @Test
     public void testQName() {
-        final QName qName = QName.create(namespace, revision, localName);
-        final QName qName1 = QName.create(namespace, localName);
-        final QName qName2 = QName.create(qName1, localName);
-        assertEquals(qName1, qName.withoutRevision());
-        assertEquals(qName1, qName2);
-        assertTrue(qName.isEqualWithoutRevision(qName1));
+        final QName qname = QName.create(namespace, revision, localName);
+        final QName qname1 = QName.create(namespace, localName);
+        final QName qname2 = QName.create(qname1, localName);
+        assertEquals(qname1, qname.withoutRevision());
+        assertEquals(qname1, qname2);
+        assertTrue(qname.isEqualWithoutRevision(qname1));
         assertNotNull(QName.formattedRevision(new Date()));
-        assertNotNull(qName.hashCode());
-        assertEquals(qName, qName.intern());
+        assertNotNull(qname.hashCode());
+        assertEquals(qname, qname.intern());
     }
 
     @Test
     public void testQNameModule() {
-        final QNameModule qNameModule = QNameModule.create(ns, new Date());
-        assertNotNull(qNameModule.toString());
-        assertNotNull(qNameModule.getRevisionNamespace());
+        final QNameModule qnameModule = QNameModule.create(ns, new Date());
+        assertNotNull(qnameModule.toString());
+        assertNotNull(qnameModule.getRevisionNamespace());
     }
 
     private static void assertLocalNameFails(final String localName) {
@@ -135,6 +134,7 @@ public class QNameTest {
             new QName(null, localName);
             fail("Local name should fail:" + localName);
         } catch (IllegalArgumentException e) {
+            // Expected
         }
     }
-}
\ No newline at end of file
+}
OpenDaylight YANG Tools