From 1dbcef88f9195c77d11308123889544191088113 Mon Sep 17 00:00:00 2001 From: Greg Woolsey Date: Mon, 18 Mar 2019 22:05:34 +0000 Subject: [PATCH] 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 --- .classpath | 6 +++--- .../usermodel/ConditionalFormattingEvalTest.java | 14 ++------------ 2 files changed, 5 insertions(+), 15 deletions(-) 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()); } -- 2.39.5