From: BobHanson Date: Mon, 23 Mar 2020 02:00:53 +0000 (-0500) Subject: JAL-3563 Bamboo test for range 3 instead of 13 X-Git-Url: http://source.jalview.org/gitweb/?a=commitdiff_plain;h=0902f5e06ac56bc3ce2fbd8ced84870134479e36;p=jalview.git JAL-3563 Bamboo test for range 3 instead of 13 --- diff --git a/test/jalview/gui/SeqCanvasTest.java b/test/jalview/gui/SeqCanvasTest.java index ce4ff59..3915559 100644 --- a/test/jalview/gui/SeqCanvasTest.java +++ b/test/jalview/gui/SeqCanvasTest.java @@ -27,7 +27,6 @@ import jalview.bin.Jalview; import jalview.datamodel.AlignmentI; import jalview.io.DataSourceType; import jalview.io.FileLoader; -import jalview.util.Platform; import java.awt.Font; import java.awt.FontMetrics; @@ -223,7 +222,10 @@ public class SeqCanvasTest canvasWidth += 2; wrappedWidth = testee.calculateWrappedGeometry(canvasWidth, canvasHeight); - assertEquals(wrappedWidth, Platform.isMac() ? 24 : 25); // 2px not enough + Assert.assertTrue(wrappedWidth == 24 || wrappedWidth == 25, + "WrappedWidth [" + wrappedWidth + "] should be 24 or 25"); // 2px + // not + // enough canvasWidth += 1; wrappedWidth = testee.calculateWrappedGeometry(canvasWidth, canvasHeight); @@ -376,12 +378,15 @@ public class SeqCanvasTest AlignmentI al = av.getAlignment(); assertEquals(al.getWidth(), 157); assertEquals(al.getHeight(), 15); + String ss = ""; + ss += " 1:" + av.getRanges(); av.getRanges().setStartEndSeq(0, 3); + ss += " 2:" + av.getRanges(); av.setShowAnnotation(false); av.setScaleAboveWrapped(true); - SeqCanvas testee = af.alignPanel.getSeqPanel().seqCanvas; av.setWrapAlignment(true); + ss += " 4:" + av.getRanges(); av.setFont(new Font("SansSerif", Font.PLAIN, 14), true); int charHeight = av.getCharHeight(); int charWidth = av.getCharWidth(); @@ -392,10 +397,13 @@ public class SeqCanvasTest int canvasWidth = 400; int canvasHeight = 300; testee.calculateWrappedGeometry(canvasWidth, canvasHeight); + + ss += " 5:" + av.getRanges(); + int repeatingHeight = (int) PA.getValue(testee, "wrappedRepeatHeightPx"); assertEquals(av.getRanges().getEndSeq(), 3, - "endSeq should be 3; av.getRanges=" + av.getRanges()); // unchanged + "endSeq should be 3; av.getRanges=" + ss); // unchanged assertEquals(repeatingHeight, charHeight * (2 + al.getHeight())); } }