diff options
author | PJ Fanning <fanningpj@apache.org> | 2017-09-19 21:04:30 +0000 |
---|---|---|
committer | PJ Fanning <fanningpj@apache.org> | 2017-09-19 21:04:30 +0000 |
commit | de664ee97a70af0ddb3094fd74cc98a43d2036e3 (patch) | |
tree | 2daba4d73cc18789639fde01f6b33de8dba7cc26 /src/testcases/org | |
parent | cf2265d0b0b490c18383e28c18021304e02a6ece (diff) | |
download | poi-de664ee97a70af0ddb3094fd74cc98a43d2036e3.tar.gz poi-de664ee97a70af0ddb3094fd74cc98a43d2036e3.zip |
tidy up some test code
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1808944 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'src/testcases/org')
-rw-r--r-- | src/testcases/org/apache/poi/hssf/usermodel/TestHSSFCell.java | 10 | ||||
-rw-r--r-- | src/testcases/org/apache/poi/hssf/util/TestAreaReference.java | 6 |
2 files changed, 8 insertions, 8 deletions
diff --git a/src/testcases/org/apache/poi/hssf/usermodel/TestHSSFCell.java b/src/testcases/org/apache/poi/hssf/usermodel/TestHSSFCell.java index a9cc347bef..7fe5f09224 100644 --- a/src/testcases/org/apache/poi/hssf/usermodel/TestHSSFCell.java +++ b/src/testcases/org/apache/poi/hssf/usermodel/TestHSSFCell.java @@ -301,13 +301,13 @@ public final class TestHSSFCell extends BaseTestCell { styB.verifyBelongsToWorkbook(wbB); try { styA.verifyBelongsToWorkbook(wbB); - fail(); + fail("expected IllegalArgumentException"); } catch (IllegalArgumentException e) { // expected during successful test } try { styB.verifyBelongsToWorkbook(wbA); - fail(); + fail("expected IllegalArgumentException"); } catch (IllegalArgumentException e) { // expected during successful test } @@ -319,13 +319,13 @@ public final class TestHSSFCell extends BaseTestCell { cellB.setCellStyle(styB); try { cellA.setCellStyle(styB); - fail(); + fail("expected IllegalArgumentException"); } catch (IllegalArgumentException e) { // expected during successful test } try { cellB.setCellStyle(styA); - fail(); + fail("expected IllegalArgumentException"); } catch (IllegalArgumentException e) { // expected during successful test } @@ -352,7 +352,7 @@ public final class TestHSSFCell extends BaseTestCell { Record[] recs = RecordInspector.getRecords(sheet, 0); if (recs.length == 28 && recs[23] instanceof StringRecord) { wb.close(); - throw new AssertionFailedError("Identified bug - leftover StringRecord"); + fail("Identified bug - leftover StringRecord"); } confirmStringRecord(sheet, false); diff --git a/src/testcases/org/apache/poi/hssf/util/TestAreaReference.java b/src/testcases/org/apache/poi/hssf/util/TestAreaReference.java index 2819289ea9..c14ca44e42 100644 --- a/src/testcases/org/apache/poi/hssf/util/TestAreaReference.java +++ b/src/testcases/org/apache/poi/hssf/util/TestAreaReference.java @@ -126,19 +126,19 @@ public final class TestAreaReference extends TestCase { new AreaReference(ref2D, SpreadsheetVersion.EXCEL97); try { new AreaReference(refDCSimple, SpreadsheetVersion.EXCEL97); - fail(); + fail("expected IllegalArgumentException"); } catch(IllegalArgumentException e) { // expected during successful test } try { new AreaReference(refDC2D, SpreadsheetVersion.EXCEL97); - fail(); + fail("expected IllegalArgumentException"); } catch(IllegalArgumentException e) { // expected during successful test } try { new AreaReference(refDC3D, SpreadsheetVersion.EXCEL97); - fail(); + fail("expected IllegalArgumentException"); } catch(IllegalArgumentException e) { // expected during successful test } |