X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=test%2Fjalview%2Fio%2FFeaturesFileTest.java;h=04ddeb08802d9260daeb950247cfdbfc84b12388;hb=a29326254cf21aa7633d1485c1621ffd0fee1097;hp=77c18dbd054cf19f2367ca9f241026d6bac8a854;hpb=3c8a25936a2d805e7e3d7ab82f83b13135406d18;p=jalview.git

diff --git a/test/jalview/io/FeaturesFileTest.java b/test/jalview/io/FeaturesFileTest.java
index 77c18db..04ddeb0 100644
--- a/test/jalview/io/FeaturesFileTest.java
+++ b/test/jalview/io/FeaturesFileTest.java
@@ -43,7 +43,6 @@ import jalview.gui.AlignFrame;
 import jalview.gui.Desktop;
 import jalview.gui.JvOptionPane;
 import jalview.schemes.FeatureColour;
-import jalview.structure.StructureSelectionManager;
 import jalview.util.matcher.Condition;
 import jalview.viewmodel.seqfeatures.FeatureRendererModel;
 import jalview.viewmodel.seqfeatures.FeatureRendererModel.FeatureSettingsBean;
@@ -62,6 +61,7 @@ import org.testng.annotations.Test;
 
 public class FeaturesFileTest
 {
+  private static final String LINE_SEPARATOR = System.getProperty("line.separator");
   private static String simpleGffFile = "examples/testdata/simpleGff3.gff";
 
   @AfterClass(alwaysRun = true)
@@ -70,9 +70,7 @@ public class FeaturesFileTest
     /*
      * remove any sequence mappings created so they don't pollute other tests
      */
-    StructureSelectionManager ssm = StructureSelectionManager
-            .getStructureSelectionManager(Desktop.instance);
-    ssm.resetAll();
+    Desktop.getStructureSelectionManager().resetAll();
   }
 
   @BeforeClass(alwaysRun = true)
@@ -491,7 +489,9 @@ public class FeaturesFileTest
             + "ENDGROUP\tuniprot\n\n"
             + "desc1\tFER_CAPAN\t-1\t0\t0\tPfam\t1.3\n\n"
             + "desc3\tFER1_SOLLC\t-1\t0\t0\tPfam\n"; // NaN is not output
-    assertEquals(expected, exported);
+    assertEquals(
+            expected.replace("\n", LINE_SEPARATOR),
+            exported);
 
     /*
      * set METAL (in uniprot group) and GAMMA-TURN visible, but not Pfam
@@ -506,7 +506,7 @@ public class FeaturesFileTest
             + "Turn\tFER_CAPAA\t-1\t36\t38\tGAMMA-TURN\t0.0\n"
             + "Iron\tFER_CAPAA\t-1\t39\t39\tMETAL\t0.0\n"
             + "ENDGROUP\tuniprot\n";
-    assertEquals(expected, exported);
+    assertEquals(fixLineEnd(expected), exported);
 
     /*
      * now set Pfam visible
@@ -528,7 +528,7 @@ public class FeaturesFileTest
             // null / empty group features are output after named groups
             + "\ndesc2\tFER_CAPAN\t-1\t4\t9\tPfam\n"
             + "\ndesc4\tFER1_SOLLC\t-1\t5\t8\tPfam\t-2.6\n";
-    assertEquals(expected, exported);
+    assertEquals(fixLineEnd(expected), exported);
 
     /*
      * hide uniprot group
@@ -540,7 +540,7 @@ public class FeaturesFileTest
             + "\ndesc4\tFER1_SOLLC\t-1\t5\t8\tPfam\t-2.6\n";
     exported = featuresFile.printJalviewFormat(al.getSequencesArray(), fr,
             false);
-    assertEquals(expected, exported);
+    assertEquals(fixLineEnd(expected), exported);
 
     /*
      * include non-positional (overrides group not shown)
@@ -556,7 +556,7 @@ public class FeaturesFileTest
             + "desc2\tFER_CAPAN\t-1\t4\t9\tPfam\n"
             + "\ndesc3\tFER1_SOLLC\t-1\t0\t0\tPfam\n"
             + "desc4\tFER1_SOLLC\t-1\t5\t8\tPfam\t-2.6\n";
-    assertEquals(expected, exported);
+    assertEquals(fixLineEnd(expected), exported);
   }
 
   @Test(groups = { "Functional" })
@@ -575,10 +575,14 @@ public class FeaturesFileTest
     String exported = featuresFile.printGffFormat(al.getSequencesArray(),
             fr, false);
     String gffHeader = "##gff-version 2\n";
-    assertEquals(gffHeader, exported);
+    assertEquals(
+            fixLineEnd(gffHeader),
+            exported);
     exported = featuresFile.printGffFormat(al.getSequencesArray(), fr,
             true);
-    assertEquals(gffHeader, exported);
+    assertEquals(
+            fixLineEnd(gffHeader),
+            exported);
 
     /*
      * add some features
@@ -615,7 +619,9 @@ public class FeaturesFileTest
      */
     exported = featuresFile.printGffFormat(al.getSequencesArray(), fr,
             false);
-    assertEquals(gffHeader, exported);
+    assertEquals(
+            fixLineEnd(gffHeader),
+            exported);
 
     /*
      * include non-positional features
@@ -626,7 +632,9 @@ public class FeaturesFileTest
             true);
     String expected = gffHeader
             + "FER_CAPAA\tUniprot\tDomain\t0\t0\t0.0\t.\t.\n";
-    assertEquals(expected, exported);
+    assertEquals(
+            fixLineEnd(expected),
+            exported);
 
     /*
      * set METAL (in uniprot group) and GAMMA-TURN visible, but not Pfam
@@ -638,7 +646,9 @@ public class FeaturesFileTest
             false);
     // METAL feature has null group: description used for column 2
     expected = gffHeader + "FER_CAPAA\tCath\tMETAL\t39\t39\t1.2\t.\t.\n";
-    assertEquals(expected, exported);
+    assertEquals(
+            expected.replace("\n", LINE_SEPARATOR),
+            exported);
 
     /*
      * set s3dm group visible
@@ -649,7 +659,9 @@ public class FeaturesFileTest
     // METAL feature has null group: description used for column 2
     expected = gffHeader + "FER_CAPAA\tCath\tMETAL\t39\t39\t1.2\t.\t.\n"
             + "FER_CAPAN\ts3dm\tGAMMA-TURN\t36\t38\t2.1\t.\t.\n";
-    assertEquals(expected, exported);
+    assertEquals(
+            expected.replace("\n", LINE_SEPARATOR),
+            exported);
 
     /*
      * now set Pfam visible
@@ -662,7 +674,12 @@ public class FeaturesFileTest
             + "FER_CAPAA\tCath\tMETAL\t39\t39\t1.2\t.\t.\n"
             + "FER_CAPAN\ts3dm\tGAMMA-TURN\t36\t38\t2.1\t.\t.\n"
             + "FER_CAPAN\tUniprot\tPfam\t20\t20\t0.0\t+\t2\tx=y;black=white\n";
-    assertEquals(expected, exported);
+    assertEquals(fixLineEnd(expected), exported);
+  }
+
+  private String fixLineEnd(String s)
+  {
+    return s.replace("\n", LINE_SEPARATOR);
   }
 
   /**
@@ -737,7 +754,7 @@ public class FeaturesFileTest
     visible.put("foobar", new FeatureColour(Color.blue));
     ff.outputFeatureFilters(sb, visible, featureFilters);
     String expected = "\nSTARTFILTERS\nfoobar\tLabel Present\npfam\t(CSQ:PolyPhen Present) AND (Score LE -2.4)\nENDFILTERS\n";
-    assertEquals(expected, sb.toString());
+    assertEquals(fixLineEnd(expected), sb.toString());
   }
 
   /**
@@ -774,7 +791,8 @@ public class FeaturesFileTest
     String expected = gffHeader
             + "FER_CAPAA\tCath\tMETAL\t39\t39\t1.2\t.\t.\n"
             + "FER_CAPAA\tCath\tMETAL\t41\t41\t0.6\t.\t.\n";
-    assertEquals(expected, exported);
+    assertEquals(
+            fixLineEnd(expected), exported);
 
     /*
      * now threshold to Score > 1.1 - should exclude sf2
@@ -787,7 +805,7 @@ public class FeaturesFileTest
     exported = featuresFile.printGffFormat(al.getSequencesArray(), fr,
             false);
     expected = gffHeader + "FER_CAPAA\tCath\tMETAL\t39\t39\t1.2\t.\t.\n";
-    assertEquals(expected, exported);
+    assertEquals(fixLineEnd(expected), exported);
 
     /*
      * remove threshold and check sf2 is exported
@@ -797,7 +815,7 @@ public class FeaturesFileTest
             false);
     expected = gffHeader + "FER_CAPAA\tCath\tMETAL\t39\t39\t1.2\t.\t.\n"
             + "FER_CAPAA\tCath\tMETAL\t41\t41\t0.6\t.\t.\n";
-    assertEquals(expected, exported);
+    assertEquals(fixLineEnd(expected), exported);
 
     /*
      * filter on (clin_sig contains Benign) - should include sf2 and exclude sf1
@@ -809,7 +827,7 @@ public class FeaturesFileTest
     exported = featuresFile.printGffFormat(al.getSequencesArray(), fr,
             false);
     expected = gffHeader + "FER_CAPAA\tCath\tMETAL\t41\t41\t0.6\t.\t.\n";
-    assertEquals(expected, exported);
+    assertEquals(fixLineEnd(expected), exported);
   }
 
   /**
@@ -849,7 +867,9 @@ public class FeaturesFileTest
             + "ENDGROUP\tgrp1\n\nSTARTGROUP\tgrp2\n"
             + "Cath\tFER_CAPAA\t-1\t41\t41\tMETAL\t0.6\n"
             + "ENDGROUP\tgrp2\n";
-    assertEquals(expected, exported);
+    assertEquals(
+            fixLineEnd(expected),
+            exported);
   
     /*
      * now threshold to Score > 1.1 - should exclude sf2
@@ -866,7 +886,9 @@ public class FeaturesFileTest
             + "STARTGROUP\tgrp1\n"
             + "Cath\tFER_CAPAA\t-1\t39\t39\tMETAL\t1.2\n"
             + "ENDGROUP\tgrp1\n";
-    assertEquals(expected, exported);
+    assertEquals(
+            expected.replace("\n", LINE_SEPARATOR),
+            exported);
   
     /*
      * remove threshold and check sf2 is exported
@@ -880,7 +902,7 @@ public class FeaturesFileTest
             + "ENDGROUP\tgrp1\n\nSTARTGROUP\tgrp2\n"
             + "Cath\tFER_CAPAA\t-1\t41\t41\tMETAL\t0.6\n"
             + "ENDGROUP\tgrp2\n";
-    assertEquals(expected, exported);
+    assertEquals(fixLineEnd(expected), exported);
   
     /*
      * filter on (clin_sig contains Benign) - should include sf2 and exclude sf1
@@ -897,6 +919,6 @@ public class FeaturesFileTest
             + "STARTGROUP\tgrp2\n"
             + "Cath\tFER_CAPAA\t-1\t41\t41\tMETAL\t0.6\n"
             + "ENDGROUP\tgrp2\n";
-    assertEquals(expected, exported);
+    assertEquals(fixLineEnd(expected), exported);
   }
 }