JAL-2586 Clarification to URL link code bug/JAL-2586
authorkiramt <k.mourao@dundee.ac.uk>
Thu, 8 Jun 2017 14:01:02 +0000 (15:01 +0100)
committerkiramt <k.mourao@dundee.ac.uk>
Thu, 8 Jun 2017 14:01:02 +0000 (15:01 +0100)
src/jalview/urls/UrlLinkDisplay.java
test/jalview/urls/UrlLinkDisplayTest.java

index 5b376a2..09566c3 100644 (file)
@@ -177,7 +177,7 @@ public class UrlLinkDisplay
       break;
     case NAME:
       setDescription((String) value);
-      break;
+      // deliberate fall through
     case DATABASE:
       setDBName((String) value);
       break;
index ff7d957..9eeac9f 100644 (file)
@@ -131,7 +131,7 @@ public class UrlLinkDisplayTest {
 
     u.setValue(UrlLinkDisplay.NAME, "New Desc");
     Assert.assertEquals(u.getValue(UrlLinkDisplay.NAME), "New Desc");
-    Assert.assertEquals(u.getValue(UrlLinkDisplay.DATABASE), "TestDB");
+    Assert.assertEquals(u.getValue(UrlLinkDisplay.DATABASE), "New Desc");
 
     u.setValue(UrlLinkDisplay.DATABASE, "NewName");
     Assert.assertEquals(u.getValue(UrlLinkDisplay.DATABASE), "NewName");