Inverted the logic - don't know how I did that. Anyway, fixed now. An…

…d added a test.
ISA-tools · Jan 20, 2014 · bedae16 · bedae16
1 parent 0aa79bb
commit bedae16
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 1 deletion.
diff --git a/src/main/java/org/isatools/isacreator/io/importisa/InvestigationImport.java b/src/main/java/org/isatools/isacreator/io/importisa/InvestigationImport.java
@@ -127,9 +127,10 @@ public Pair<Boolean, OrderedMap<String, OrderedMap<InvestigationFileSection, Ord
 
 
                     String valueToTitleCase = lineLabel;
-                    if (!lineLabel.contains("Comment"))
+                    if (lineLabel.contains("Comment"))
                         valueToTitleCase = StringProcessing.removeSpaceFromQualifiedField(StringProcessing.convertStringToTitleCase(lineLabel));
 
+                    System.out.println(valueToTitleCase);
                     if (!importedInvestigationFile.get(currentMajorSection).get(currentMinorSection).containsKey(valueToTitleCase)) {
                         importedInvestigationFile.get(currentMajorSection).get(currentMinorSection).put(valueToTitleCase, new ArrayList<String>());
                     }

diff --git a/src/test/java/org/isatools/isacreator/utils/StringProcessingTest.java b/src/test/java/org/isatools/isacreator/utils/StringProcessingTest.java
@@ -0,0 +1,23 @@
+package org.isatools.isacreator.utils;
+
+import org.junit.Test;
+
+import static junit.framework.Assert.assertEquals;
+
+/**
+ * Created with IntelliJ IDEA.
+ * User: eamonnmaguire
+ * Date: 20/01/2014
+ * Time: 13:41
+ * To change this template use File | Settings | File Templates.
+ */
+public class StringProcessingTest {
+
+    @Test
+    public void testSpaceRemovalInQualifier() {
+        String comment = "Comment [Technical Validation]";
+        comment = StringProcessing.removeSpaceFromQualifiedField(comment);
+        System.out.println(comment);
+        assertEquals("Comment should have no spaces...", comment, "Comment[Technical Validation]");
+    }
+}