From: Greg Woolsey Date: Mon, 18 Mar 2019 22:05:34 +0000 (+0000) Subject: Fix a unit test that changed with the implementation of another function. X-Git-Tag: REL_4_1_0~40 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=1dbcef88f9195c77d11308123889544191088113;p=poi.git Fix a unit test that changed with the implementation of another function. Didn't fail for me because I hadn't noticed the Eclipse classpath was out of date wrt some dependency versions, and therefore my local code was not compiling properly, meaning my test run ran old code and passed this one when it should have failed. git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1855792 13f79535-47bb-0310-9956-ffa450edef68 --- diff --git a/.classpath b/.classpath index 247c5000e2..f17074e3e5 100644 --- a/.classpath +++ b/.classpath @@ -39,10 +39,10 @@ - - - + + + diff --git a/src/ooxml/testcases/org/apache/poi/ss/usermodel/ConditionalFormattingEvalTest.java b/src/ooxml/testcases/org/apache/poi/ss/usermodel/ConditionalFormattingEvalTest.java index 9d83635215..c889eb8471 100644 --- a/src/ooxml/testcases/org/apache/poi/ss/usermodel/ConditionalFormattingEvalTest.java +++ b/src/ooxml/testcases/org/apache/poi/ss/usermodel/ConditionalFormattingEvalTest.java @@ -161,19 +161,9 @@ public class ConditionalFormattingEvalTest { cfe = new ConditionalFormattingEvaluator(wb, formulaEval); sheet = wb.getSheetAt(0); - try { - getRulesFor(2, 1); - fail("Got rules when an unsupported function error was expected."); - } catch (NotImplementedException e) { - // expected - } + assertEquals("no rules should apply", 0, getRulesFor(2, 1).size()); - try { - getRulesFor(2, 1); - fail("Got rules the second time when an unsupported function error was expected."); - } catch (NotImplementedException e) { - // expected - } + assertEquals("no rules should apply", 0, getRulesFor(2, 1).size()); }