From: Josh Micich <josh@apache.org>
Date: Wed, 1 Oct 2008 20:56:21 +0000 (+0000)
Subject: Fixed bug in CellCacheEntry (support for caching blank evaluation results)
X-Git-Tag: REL_3_2_FINAL~16
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=3d3c4958d23bfde21eaf1df1f34fcfa8f87f90fc;p=poi.git

Fixed bug in CellCacheEntry (support for caching blank evaluation results)

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

diff --git a/src/java/org/apache/poi/ss/formula/CellCacheEntry.java b/src/java/org/apache/poi/ss/formula/CellCacheEntry.java
index 861874c25b..bd7500023c 100644
--- a/src/java/org/apache/poi/ss/formula/CellCacheEntry.java
+++ b/src/java/org/apache/poi/ss/formula/CellCacheEntry.java
@@ -20,6 +20,7 @@ package org.apache.poi.ss.formula;
 import java.util.HashSet;
 import java.util.Set;
 
+import org.apache.poi.hssf.record.formula.eval.BlankEval;
 import org.apache.poi.hssf.record.formula.eval.BoolEval;
 import org.apache.poi.hssf.record.formula.eval.ErrorEval;
 import org.apache.poi.hssf.record.formula.eval.NumberEval;
@@ -60,6 +61,9 @@ final class CellCacheEntry {
 			// value type is changing
 			return false;
 		}
+		if (a == BlankEval.INSTANCE) {
+			return b == a;
+		}
 		if (cls == NumberEval.class) {
 			return ((NumberEval)a).getNumberValue() == ((NumberEval)b).getNumberValue();
 		}
diff --git a/src/testcases/org/apache/poi/ss/formula/AllSSFormulaTests.java b/src/testcases/org/apache/poi/ss/formula/AllSSFormulaTests.java
index dd1360f769..13e2cfecdf 100644
--- a/src/testcases/org/apache/poi/ss/formula/AllSSFormulaTests.java
+++ b/src/testcases/org/apache/poi/ss/formula/AllSSFormulaTests.java
@@ -26,9 +26,10 @@ import junit.framework.TestSuite;
  */
 public final class AllSSFormulaTests {
     public static Test suite() {
-        TestSuite result = new TestSuite(AllSSFormulaTests.class.getName());
-        result.addTestSuite(TestEvaluationCache.class);
-        result.addTestSuite(TestWorkbookEvaluator.class);
-        return result;
-    }
+		TestSuite result = new TestSuite(AllSSFormulaTests.class.getName());
+		result.addTestSuite(TestCellCacheEntry.class);
+		result.addTestSuite(TestEvaluationCache.class);
+		result.addTestSuite(TestWorkbookEvaluator.class);
+		return result;
+	}
 }
diff --git a/src/testcases/org/apache/poi/ss/formula/TestCellCacheEntry.java b/src/testcases/org/apache/poi/ss/formula/TestCellCacheEntry.java
new file mode 100644
index 0000000000..e8ff03f8b7
--- /dev/null
+++ b/src/testcases/org/apache/poi/ss/formula/TestCellCacheEntry.java
@@ -0,0 +1,53 @@
+/* ====================================================================
+   Licensed to the Apache Software Foundation (ASF) under one or more
+   contributor license agreements.  See the NOTICE file distributed with
+   this work for additional information regarding copyright ownership.
+   The ASF licenses this file to You under the Apache License, Version 2.0
+   (the "License"); you may not use this file except in compliance with
+   the License.  You may obtain a copy of the License at
+
+       http://www.apache.org/licenses/LICENSE-2.0
+
+   Unless required by applicable law or agreed to in writing, software
+   distributed under the License is distributed on an "AS IS" BASIS,
+   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+   See the License for the specific language governing permissions and
+   limitations under the License.
+==================================================================== */
+
+package org.apache.poi.ss.formula;
+
+import org.apache.poi.hssf.record.formula.eval.BlankEval;
+import org.apache.poi.hssf.record.formula.eval.NumberEval;
+import org.apache.poi.hssf.record.formula.eval.ValueEval;
+
+import junit.framework.AssertionFailedError;
+import junit.framework.TestCase;
+
+/**
+ * Tests {@link CellCacheEntry}.
+ *
+ * @author Josh Micich
+ */
+public class TestCellCacheEntry extends TestCase {
+
+	public void testBasic() {
+		CellCacheEntry cce = new CellCacheEntry();
+		cce.updatePlainValue(new NumberEval(42.0));
+		ValueEval ve = cce.getValue();
+		assertEquals(42, ((NumberEval)ve).getNumberValue(), 0.0);
+		
+		cce.setFormulaResult(new NumberEval(10.0), new CellLocation[] { });
+	}
+
+	public void testBlank() {
+		CellCacheEntry cce = new CellCacheEntry();
+		cce.updatePlainValue(BlankEval.INSTANCE);
+		try {
+			cce.updatePlainValue(BlankEval.INSTANCE);
+		} catch (IllegalStateException e) {
+			// bug was visible around svn r700356
+			throw new AssertionFailedError("cache entry does not handle blank values properly");
+		}
+	}
+}