Fix sum function so that it doesn't accept errors.

anmcgrath · Mar 2, 2024 · 03a38a5 · 03a38a5
1 parent f8f4871
commit 03a38a5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/BlazorDatasheet.Formula.Functions/Math/SumFunction.cs b/src/BlazorDatasheet.Formula.Functions/Math/SumFunction.cs
@@ -32,5 +32,5 @@ public CellValue Call(CellValue[] args, FunctionCallMetaData metaData)
         return CellValue.Number(sum);
     }
 
-    public bool AcceptsErrors => true;
+    public bool AcceptsErrors => false;
 }
diff --git a/test/BlazorDatasheet.Test/Functions/MathFunctionTests.cs b/test/BlazorDatasheet.Test/Functions/MathFunctionTests.cs
@@ -45,6 +45,7 @@ public void Sum_Function_Tests()
         _env.RegisterFunction("sum", new SumFunction());
         var res1 = Eval("=sum(1, 2)");
         res1.Should().Be(3);
+        Eval("=sum(1/0)").Should().BeOfType<FormulaError>();
         Eval("=sum(5)").Should().Be(5);
         Eval("=sum(true,true)").Should().Be(2);
         Eval("=sum(\"ab\",true)").Should().BeOfType<FormulaError>();