From e21b19711c139773f6cfb72f7526c5b5154b5b6c Mon Sep 17 00:00:00 2001 From: cgivre Date: Sun, 25 Aug 2024 09:45:27 -0400 Subject: [PATCH] Updated Unit Tests --- .../store/easy/json/loader/TestBasics.java | 26 +++++++++++++------ 1 file changed, 18 insertions(+), 8 deletions(-) diff --git a/exec/java-exec/src/test/java/org/apache/drill/exec/store/easy/json/loader/TestBasics.java b/exec/java-exec/src/test/java/org/apache/drill/exec/store/easy/json/loader/TestBasics.java index 10fcbace9e9..0ab5c2b3c3c 100644 --- a/exec/java-exec/src/test/java/org/apache/drill/exec/store/easy/json/loader/TestBasics.java +++ b/exec/java-exec/src/test/java/org/apache/drill/exec/store/easy/json/loader/TestBasics.java @@ -17,12 +17,6 @@ */ package org.apache.drill.exec.store.easy.json.loader; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertNotNull; -import static org.junit.Assert.assertNull; -import static org.junit.Assert.assertTrue; -import static org.junit.Assert.fail; - import org.apache.drill.categories.JsonTest; import org.apache.drill.common.exceptions.UserException; import org.apache.drill.common.types.TypeProtos.MinorType; @@ -35,6 +29,12 @@ import org.junit.Test; import org.junit.experimental.categories.Category; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertNotNull; +import static org.junit.Assert.assertNull; +import static org.junit.Assert.assertTrue; +import static org.junit.Assert.fail; + @Category(JsonTest.class) public class TestBasics extends BaseJsonLoaderTest { @@ -176,12 +176,22 @@ public void testMissingEndOuterArray() { @Test public void testEmptyKey() { - expectError("{\"\": 10}", "does not allow empty keys"); + JsonLoaderFixture loader = new JsonLoaderFixture(); + loader.jsonOptions.skipMalformedRecords = true; + loader.open("{\"\": 10}"); + RowSet results = loader.next(); + assertNotNull(results); + assertEquals(1, results.rowCount()); } @Test public void testBlankKey() { - expectError("{\" \": 10}", "does not allow empty keys"); + JsonLoaderFixture loader = new JsonLoaderFixture(); + loader.jsonOptions.skipMalformedRecords = true; + loader.open("{\" \": 10}"); + RowSet results = loader.next(); + assertNotNull(results); + assertEquals(1, results.rowCount()); } @Test