From 0902f5e06ac56bc3ce2fbd8ced84870134479e36 Mon Sep 17 00:00:00 2001 From: BobHanson Date: Sun, 22 Mar 2020 21:00:53 -0500 Subject: [PATCH] JAL-3563 Bamboo test for range 3 instead of 13 --- test/jalview/gui/SeqCanvasTest.java | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) 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())); } } -- 1.7.10.2