diff options
author | Dominik Stadler <centic@apache.org> | 2016-07-17 21:18:07 +0000 |
---|---|---|
committer | Dominik Stadler <centic@apache.org> | 2016-07-17 21:18:07 +0000 |
commit | 8924e1260bf1d1f7f8c086c0fa046fc261d2951c (patch) | |
tree | dbb0364ab25e4153a012022f0cc349b55e0ef71d /src/testcases/org/apache/poi/ss | |
parent | 22a9d458cfaa275a810c7e81c7e4a8496c132758 (diff) | |
download | poi-8924e1260bf1d1f7f8c086c0fa046fc261d2951c.tar.gz poi-8924e1260bf1d1f7f8c086c0fa046fc261d2951c.zip |
Fix some IntelliJ and Findbugs warnings: StringBuilder, foreach, append(), ...
test-updates
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1753122 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'src/testcases/org/apache/poi/ss')
-rw-r--r-- | src/testcases/org/apache/poi/ss/usermodel/TestDataFormatter.java | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/testcases/org/apache/poi/ss/usermodel/TestDataFormatter.java b/src/testcases/org/apache/poi/ss/usermodel/TestDataFormatter.java index f5e79c8c90..649bddf9b4 100644 --- a/src/testcases/org/apache/poi/ss/usermodel/TestDataFormatter.java +++ b/src/testcases/org/apache/poi/ss/usermodel/TestDataFormatter.java @@ -140,7 +140,7 @@ public class TestDataFormatter { public void testColours() { DataFormatter dfUS = new DataFormatter(Locale.US); - String[] formats = new String[] { + String[] formats = { "##.##", "[WHITE]##.##", "[BLACK]##.##;[RED]-##.##", @@ -169,7 +169,7 @@ public class TestDataFormatter { DataFormatter dfUS = new DataFormatter(Locale.US); // Without currency symbols - String[] formats = new String[] { "#,##0.00;[Blue](#,##0.00)" }; + String[] formats = { "#,##0.00;[Blue](#,##0.00)" }; for (String format : formats) { assertEquals( "Wrong format for: " + format, @@ -304,7 +304,7 @@ public class TestDataFormatter { assertEquals("321 1/3", dfUS.formatRawCellContents(321.321, -1, "# ?/? ?/?")); assertEquals("321 1/3", dfUS.formatRawCellContents(321.321, -1, "# ?/? #/# #/#")); - // Where +ve has a fraction, but -ve doesnt, we currently show both + // Where +ve has a fraction, but -ve doesn't, we currently show both assertEquals("123 1/3", dfUS.formatRawCellContents( 123.321, -1, "0 ?/?;0")); //assertEquals("123", dfUS.formatRawCellContents(-123.321, -1, "0 ?/?;0")); @@ -567,12 +567,12 @@ public class TestDataFormatter { assertEquals(" 0.10 ", dfUS.formatRawCellContents( 0.1, -1, "_-* #,##0.00_-;-* #,##0.00_-;_-* \"-\"??_-;_-@_-")); assertEquals("- 0.10 ", dfUS.formatRawCellContents(-0.1, -1, "_-* #,##0.00_-;-* #,##0.00_-;_-* \"-\"??_-;_-@_-")); - // TODO Fix this, we are randomly adding a 0 at the end that souldn't be there + // TODO Fix this, we are randomly adding a 0 at the end that shouldn't be there //assertEquals(" - ", dfUS.formatRawCellContents(0.0, -1, "_-* #,##0.00_-;-* #,##0.00_-;_-* \"-\"??_-;_-@_-")); assertEquals(" $ 1.10 ", dfUS.formatRawCellContents( 1.1, -1, "_-$* #,##0.00_-;-$* #,##0.00_-;_-$* \"-\"??_-;_-@_-")); assertEquals("-$ 1.10 ", dfUS.formatRawCellContents(-1.1, -1, "_-$* #,##0.00_-;-$* #,##0.00_-;_-$* \"-\"??_-;_-@_-")); - // TODO Fix this, we are randomly adding a 0 at the end that souldn't be there + // TODO Fix this, we are randomly adding a 0 at the end that shouldn't be there //assertEquals(" $ - ", dfUS.formatRawCellContents( 0.0, -1, "_-$* #,##0.00_-;-$* #,##0.00_-;_-$* \"-\"??_-;_-@_-")); } |