From: Josh Micich <josh@apache.org>
Date: Wed, 21 Jan 2009 02:16:01 +0000 (+0000)
Subject: Removed System.out.println and compiler warnings from test code
X-Git-Tag: REL_3_5_BETA5~26
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=bd48c3cf1e0af94b46bd70cc49ff691bf212bd3b;p=poi.git

Removed System.out.println and compiler warnings from test code

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@736193 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/src/testcases/org/apache/poi/hssf/usermodel/TestEscherGraphics2d.java b/src/testcases/org/apache/poi/hssf/usermodel/TestEscherGraphics2d.java
index 6906cfdcb8..ea098aff71 100644
--- a/src/testcases/org/apache/poi/hssf/usermodel/TestEscherGraphics2d.java
+++ b/src/testcases/org/apache/poi/hssf/usermodel/TestEscherGraphics2d.java
@@ -27,43 +27,38 @@ import java.awt.geom.Line2D;
  *
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestEscherGraphics2d extends TestCase
-{
-    private HSSFShapeGroup escherGroup;
-    private EscherGraphics2d graphics;
-
-    protected void setUp() throws Exception
-    {
-        super.setUp();
-
-        HSSFWorkbook workbook = new HSSFWorkbook();
-        HSSFSheet sheet = workbook.createSheet("test");
-        escherGroup = sheet.createDrawingPatriarch().createGroup(new HSSFClientAnchor(0,0,1023,255,(short)0,0,(short) 0,0));
-        escherGroup = new HSSFShapeGroup(null, new HSSFChildAnchor());
-        EscherGraphics g = new EscherGraphics(this.escherGroup, workbook, Color.black, 1.0f);
-        graphics = new EscherGraphics2d(g);
-
-    }
-
-    public void testDrawString() throws Exception
-    {
-        graphics.drawString("This is a test", 10, 10);
-        HSSFTextbox t = (HSSFTextbox) escherGroup.getChildren().get(0);
-        assertEquals("This is a test", t.getString().getString().toString());
+public final class TestEscherGraphics2d extends TestCase {
+	private HSSFShapeGroup escherGroup;
+	private EscherGraphics2d graphics;
+
+	protected void setUp() {
+
+		HSSFWorkbook workbook = new HSSFWorkbook();
+		HSSFSheet sheet = workbook.createSheet("test");
+		escherGroup = sheet.createDrawingPatriarch().createGroup(new HSSFClientAnchor(0,0,1023,255,(short)0,0,(short) 0,0));
+		escherGroup = new HSSFShapeGroup(null, new HSSFChildAnchor());
+		EscherGraphics g = new EscherGraphics(this.escherGroup, workbook, Color.black, 1.0f);
+		graphics = new EscherGraphics2d(g);
+	}
+
+	public void testDrawString() {
+		graphics.drawString("This is a test", 10, 10);
+		HSSFTextbox t = (HSSFTextbox) escherGroup.getChildren().get(0);
+		assertEquals("This is a test", t.getString().getString().toString());
 
 		// Check that with a valid font, it's still ok
 		Font font = new Font("Forte", Font.PLAIN, 12);
 		graphics.setFont(font);
-        graphics.drawString("This is another test", 10, 10);
+		graphics.drawString("This is another test", 10, 10);
 
 		// And test with ones that need the style appending
 		font = new Font("dialog", Font.PLAIN, 12);
 		graphics.setFont(font);
-        graphics.drawString("This is another test", 10, 10);
+		graphics.drawString("This is another test", 10, 10);
 
 		font = new Font("dialog", Font.BOLD, 12);
 		graphics.setFont(font);
-        graphics.drawString("This is another test", 10, 10);
+		graphics.drawString("This is another test", 10, 10);
 
 		// But with an invalid font, we get an exception
 		font = new Font("IamAmadeUPfont", Font.PLAIN, 22);
@@ -72,60 +67,59 @@ public class TestEscherGraphics2d extends TestCase
 			graphics.drawString("This is another test", 10, 10);
 			fail();
 		} catch(IllegalArgumentException e) {
+			// expected
 		}
-    }
-
-    public void testFillRect() throws Exception
-    {
-        graphics.fillRect( 10, 10, 20, 20 );
-        HSSFSimpleShape s = (HSSFSimpleShape) escherGroup.getChildren().get(0);
-        assertEquals(HSSFSimpleShape.OBJECT_TYPE_RECTANGLE, s.getShapeType());
-        assertEquals(10, s.getAnchor().getDx1());
-        assertEquals(10, s.getAnchor().getDy1());
-        assertEquals(30, s.getAnchor().getDy2());
-        assertEquals(30, s.getAnchor().getDx2());
-    }
-
-    public void testGetFontMetrics() throws Exception
-    {
-        FontMetrics fontMetrics = graphics.getFontMetrics(graphics.getFont());
-    if (graphics.getFont().toString().indexOf("dialog") != -1 || graphics.getFont().toString().indexOf("Dialog") != -1) // if dialog is returned we can't run the test properly.
-            return;
-        assertEquals(7, fontMetrics.charWidth('X'));
-        assertEquals("java.awt.Font[family=Arial,name=Arial,style=plain,size=10]", fontMetrics.getFont().toString());
-    }
-
-    public void testSetFont() throws Exception
-    {
-        Font f = new Font("Helvetica", 0, 12);
-        graphics.setFont(f);
-        assertEquals(f, graphics.getFont());
-    }
-
-    public void testSetColor() throws Exception
-    {
-        graphics.setColor(Color.red);
-        assertEquals(Color.red, graphics.getColor());
-    }
-
-    public void testGetFont() throws Exception
-    {
-        Font f = graphics.getFont();
-        if (graphics.getFont().toString().indexOf("dialog") != -1 || graphics.getFont().toString().indexOf("Dialog") != -1) // if dialog is returned we can't run the test properly.
-            return;
-
-        assertEquals("java.awt.Font[family=Arial,name=Arial,style=plain,size=10]", f.toString());
-    }
-
-    public void testDraw() throws Exception
-    {
-        graphics.draw(new Line2D.Double(10,10,20,20));
-        HSSFSimpleShape s = (HSSFSimpleShape) escherGroup.getChildren().get(0);
-        assertTrue(s.getShapeType() == HSSFSimpleShape.OBJECT_TYPE_LINE);
-        assertEquals(10, s.getAnchor().getDx1());
-        assertEquals(10, s.getAnchor().getDy1());
-        assertEquals(20, s.getAnchor().getDx2());
-        assertEquals(20, s.getAnchor().getDy2());
-        System.out.println("s = " + s);
-    }
+	}
+
+	public void testFillRect() {
+		graphics.fillRect( 10, 10, 20, 20 );
+		HSSFSimpleShape s = (HSSFSimpleShape) escherGroup.getChildren().get(0);
+		assertEquals(HSSFSimpleShape.OBJECT_TYPE_RECTANGLE, s.getShapeType());
+		assertEquals(10, s.getAnchor().getDx1());
+		assertEquals(10, s.getAnchor().getDy1());
+		assertEquals(30, s.getAnchor().getDy2());
+		assertEquals(30, s.getAnchor().getDx2());
+	}
+
+	public void testGetFontMetrics() {
+		FontMetrics fontMetrics = graphics.getFontMetrics(graphics.getFont());
+		if (isDialogPresent()) // if dialog is returned we can't run the test properly.
+			return;
+		assertEquals(7, fontMetrics.charWidth('X'));
+		assertEquals("java.awt.Font[family=Arial,name=Arial,style=plain,size=10]", fontMetrics.getFont().toString());
+	}
+
+	public void testSetFont() {
+		Font f = new Font("Helvetica", 0, 12);
+		graphics.setFont(f);
+		assertEquals(f, graphics.getFont());
+	}
+
+	public void testSetColor() {
+		graphics.setColor(Color.red);
+		assertEquals(Color.red, graphics.getColor());
+	}
+
+	public void testGetFont() {
+		Font f = graphics.getFont();
+		if (isDialogPresent()) // if dialog is returned we can't run the test properly.
+			return;
+
+		assertEquals("java.awt.Font[family=Arial,name=Arial,style=plain,size=10]", f.toString());
+	}
+
+	private boolean isDialogPresent() {
+		String fontDebugStr = graphics.getFont().toString();
+		return fontDebugStr.indexOf("dialog") != -1 || fontDebugStr.indexOf("Dialog") != -1;
+	}
+
+	public void testDraw() {
+		graphics.draw(new Line2D.Double(10,10,20,20));
+		HSSFSimpleShape s = (HSSFSimpleShape) escherGroup.getChildren().get(0);
+		assertTrue(s.getShapeType() == HSSFSimpleShape.OBJECT_TYPE_LINE);
+		assertEquals(10, s.getAnchor().getDx1());
+		assertEquals(10, s.getAnchor().getDy1());
+		assertEquals(20, s.getAnchor().getDx2());
+		assertEquals(20, s.getAnchor().getDy2());
+	}
 }