From: gmungoc <g.m.carstairs@dundee.ac.uk>
Date: Mon, 4 Jun 2018 19:42:53 +0000 (+0100)
Subject: JAL-2808 update test for revised treatment of 'mixed type' attribute values
X-Git-Tag: Release_2_11_0~31
X-Git-Url: http://source.jalview.org/gitweb/?a=commitdiff_plain;h=fc9edfca9cec3e0a10d3585bb856e0dcd6cff999;p=jalview.git

JAL-2808 update test for revised treatment of 'mixed type' attribute values
---

diff --git a/test/jalview/datamodel/features/FeatureAttributesTest.java b/test/jalview/datamodel/features/FeatureAttributesTest.java
index 686cd2f..0846ec2 100644
--- a/test/jalview/datamodel/features/FeatureAttributesTest.java
+++ b/test/jalview/datamodel/features/FeatureAttributesTest.java
@@ -76,12 +76,14 @@ public class FeatureAttributesTest
     assertNull(fa.getMinMax("Pfam", "kd"));
     sf.setValue("domain", "xyz");
     assertNull(fa.getMinMax("Pfam", "kd"));
-    sf.setValue("kd", "some text");
-    assertNull(fa.getMinMax("Pfam", "kd"));
     sf.setValue("kd", "1.3");
     assertEquals(fa.getMinMax("Pfam", "kd"), new float[] { 1.3f, 1.3f });
     sf.setValue("kd", "-2.6");
     assertEquals(fa.getMinMax("Pfam", "kd"), new float[] { -2.6f, 1.3f });
+    // setting 'mixed' character and numeric values wipes the min/max value
+    sf.setValue("kd", "some text");
+    assertNull(fa.getMinMax("Pfam", "kd"));
+
     Map<String, String> csq = new HashMap<>();
     csq.put("AF", "-3");
     sf.setValue("CSQ", csq);