Skip to content

Commit 63bcef0

Browse files
committed
Fixing review comments
Fixes #184
1 parent 3dffa18 commit 63bcef0

File tree

1 file changed

+0
-11
lines changed

1 file changed

+0
-11
lines changed

tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_graphics_TextLayout.java

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1277,11 +1277,8 @@ public void test_Bug579335_win32_StyledText_LongLine() {
12771277
public void test_getLineCount() {
12781278
//Skipping this test for GTK and Cocoa as this pr #1320 applies only for windows platform.
12791279
//Without pr #1320(master) : Number of lines: 11 (for Cocoa it is : 10)
1280-
//Without pr #1320(master) : Bounds of the text layout: Rectangle {0, 0, 30, 220} (for Cocoa it is : {0, 0, 27, 176})
12811280
//Expected : Number of lines: 7 (for Cocoa it is : 10)
1282-
//Expected : Bounds of the text layout: Rectangle {0, 0, 43, 140} (for Cocoa it is : {0, 0, 27, 176})
12831281
if (SwtTestUtil.isGTK || SwtTestUtil.isCocoa) {
1284-
//TODO Fix not applicable for GTK and Cocoa.
12851282
if (SwtTestUtil.verbose) {
12861283
System.out.println("Excluded test_getLineCount(org.eclipse.swt.tests.junit.Test_org_eclipse_swt_graphics_TextLayout)");
12871284
}
@@ -1293,15 +1290,7 @@ public void test_getLineCount() {
12931290
String text = "\rabc\ndef\r\rghi\n\njkl\r\nmno\r\n\r\npqr\r";
12941291
layout.setText(text);
12951292
int lineCount = layout.getLineCount();
1296-
System.out.println("lineCount : "+lineCount);
12971293
assertEquals(7, lineCount);
1298-
1299-
Rectangle bounds = layout.getBounds();
1300-
System.out.println("bounds.width : "+bounds.width);
1301-
System.out.println("bounds.height : "+bounds.height);
1302-
assertEquals(layout.getBounds().toString(), 43, bounds.width);
1303-
//assertEquals(43, bounds.width);
1304-
assertEquals(140, bounds.height);
13051294
} finally {
13061295
layout.dispose();
13071296
}

0 commit comments

Comments
 (0)