s434730/RandomSec
3
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Update toDate and toNumber tests to handle null and empty strings correctly

Browse Source
This commit is contained in:
Owen Stephens 2018-02-21 23:10:38 +00:00
parent e0d87c3d94
commit 2758633f32
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
main/tests/server/src/com/google/refine/tests/expr/functions/strings/ToFromConversionTests.java
View File

@ -98,10 +98,9 @@ public class ToFromConversionTests extends RefineTest {
@Test
public void testToNumber() {
// Assert.assertTrue(invoke("toNumber") instanceof EvalError);
Assert.assertNull(invoke("toNumber"));
// Assert.assertTrue(invoke("toNumber", (Object) null) instanceof EvalError);
Assert.assertNull(invoke("toNumber", (Object) null));
Assert.assertTrue(invoke("toNumber") instanceof EvalError);
Assert.assertTrue(invoke("toNumber", (Object) null) instanceof EvalError);
Assert.assertTrue(invoke("toNumber", "") instanceof EvalError);
Assert.assertTrue(invoke("toNumber", "string") instanceof EvalError);
Assert.assertEquals(invoke("toNumber", "0.0"), 0.0);
Assert.assertEquals(invoke("toNumber", "123"), Long.valueOf(123));
@ -128,9 +127,9 @@ public class ToFromConversionTests extends RefineTest {
@Test
public void testToDate() throws CalendarParserException {
// Assert.assertTrue(invoke("toDate") instanceof EvalError);
Assert.assertNull(invoke("toDate"));
Assert.assertTrue(invoke("toDate") instanceof EvalError);
Assert.assertTrue(invoke("toDate", (Object) null) instanceof EvalError);
Assert.assertTrue(invoke("toDate", "") instanceof EvalError);
Assert.assertTrue(invoke("toDate", 1.0) instanceof EvalError);
Assert.assertTrue(invoke("toDate", "2012-03-01","xxx") instanceof EvalError); // bad format string
Assert.assertTrue(invoke("toDate", "2012-03-01") instanceof GregorianCalendar);