fix #280

Author: mProjectsCode

packages/core/src/utils/Literal.ts

@@ -71,7 +71,7 @@ export function isLiteral(literal: unknown): literal is MBLiteral {
  * @param literal
  */
 export function parseUnknownToLiteralArray(literal: unknown): MBLiteral[] | undefined {
-	if (literal === undefined) {
+	if (literal === undefined || literal === null) {
 		return undefined;
 	}
 

tests/fields/IPF.test.ts

@@ -152,9 +152,12 @@ let TEST_CONFIG: IPFTest[] = [
 		testCases: [
 			{
 				declaration: `INPUT[${InputFieldType.MULTI_SELECT}(${STANDARD_OPTION_ARGUMENTS}):${TEST_PROP}]`,
-				validValues: [...STANDARD_OPTION_VALUES_ARRAY, [TEST_VALUES.STRING], [TEST_VALUES.EMPTY_STRING]],
-				validMappedValues: [[TEST_VALUES.NULL, [TEST_VALUES.NULL]]],
-				invalidValues: [TEST_VALUES.UNDEFINED, TEST_VALUES.OBJECT],
+				validValues: STANDARD_OPTION_VALUES_ARRAY,
+				validMappedValues: [
+					[TEST_VALUES.STRING, [TEST_VALUES.STRING]],
+					[TEST_VALUES.EMPTY_STRING, [TEST_VALUES.EMPTY_STRING]],
+				],
+				invalidValues: [TEST_VALUES.NULL, TEST_VALUES.UNDEFINED, TEST_VALUES.OBJECT],
 			},
 		],
 	},
@@ -275,11 +278,10 @@ let TEST_CONFIG: IPFTest[] = [
 				declaration: `INPUT[${InputFieldType.LIST}:${TEST_PROP}]`,
 				validValues: STANDARD_OPTION_VALUES_ARRAY,
 				validMappedValues: [
-					[TEST_VALUES.NULL, [TEST_VALUES.NULL]],
 					[TEST_VALUES.STRING, [TEST_VALUES.STRING]],
 					[TEST_VALUES.EMPTY_STRING, [TEST_VALUES.EMPTY_STRING]],
 				],
-				invalidValues: [TEST_VALUES.UNDEFINED, TEST_VALUES.OBJECT],
+				invalidValues: [TEST_VALUES.NULL, TEST_VALUES.UNDEFINED, TEST_VALUES.OBJECT],
 			},
 		],
 	},
@@ -290,11 +292,10 @@ let TEST_CONFIG: IPFTest[] = [
 				declaration: `INPUT[${InputFieldType.LIST_SUGGESTER}(${STANDARD_OPTION_ARGUMENTS}):${TEST_PROP}]`,
 				validValues: STANDARD_OPTION_VALUES_ARRAY,
 				validMappedValues: [
-					[TEST_VALUES.NULL, [TEST_VALUES.NULL]],
 					[TEST_VALUES.STRING, [TEST_VALUES.STRING]],
 					[TEST_VALUES.EMPTY_STRING, [TEST_VALUES.EMPTY_STRING]],
 				],
-				invalidValues: [TEST_VALUES.UNDEFINED, TEST_VALUES.OBJECT],
+				invalidValues: [TEST_VALUES.NULL, TEST_VALUES.UNDEFINED, TEST_VALUES.OBJECT],
 			},
 		],
 	},
@@ -343,11 +344,10 @@ let TEST_CONFIG: IPFTest[] = [
 				declaration: `INPUT[${InputFieldType.INLINE_LIST_SUGGESTER}(${STANDARD_OPTION_ARGUMENTS}):${TEST_PROP}]`,
 				validValues: STANDARD_OPTION_VALUES_ARRAY,
 				validMappedValues: [
-					[TEST_VALUES.NULL, [TEST_VALUES.NULL]],
 					[TEST_VALUES.STRING, [TEST_VALUES.STRING]],
 					[TEST_VALUES.EMPTY_STRING, [TEST_VALUES.EMPTY_STRING]],
 				],
-				invalidValues: [TEST_VALUES.UNDEFINED, TEST_VALUES.OBJECT],
+				invalidValues: [TEST_VALUES.NULL, TEST_VALUES.UNDEFINED, TEST_VALUES.OBJECT],
 			},
 		],
 	},
@@ -358,11 +358,10 @@ let TEST_CONFIG: IPFTest[] = [
 				declaration: `INPUT[${InputFieldType.INLINE_LIST}:${TEST_PROP}]`,
 				validValues: STANDARD_OPTION_VALUES_ARRAY,
 				validMappedValues: [
-					[TEST_VALUES.NULL, [TEST_VALUES.NULL]],
 					[TEST_VALUES.STRING, [TEST_VALUES.STRING]],
 					[TEST_VALUES.EMPTY_STRING, [TEST_VALUES.EMPTY_STRING]],
 				],
-				invalidValues: [TEST_VALUES.UNDEFINED, TEST_VALUES.OBJECT],
+				invalidValues: [TEST_VALUES.NULL, TEST_VALUES.UNDEFINED, TEST_VALUES.OBJECT],
 			},
 		],
 	},
@@ -373,11 +372,10 @@ let TEST_CONFIG: IPFTest[] = [
 				declaration: `INPUT[${InputFieldType.INLINE_LIST_SUGGESTER}:${TEST_PROP}]`,
 				validValues: STANDARD_OPTION_VALUES_ARRAY,
 				validMappedValues: [
-					[TEST_VALUES.NULL, [TEST_VALUES.NULL]],
 					[TEST_VALUES.STRING, [TEST_VALUES.STRING]],
 					[TEST_VALUES.EMPTY_STRING, [TEST_VALUES.EMPTY_STRING]],
 				],
-				invalidValues: [TEST_VALUES.UNDEFINED, TEST_VALUES.OBJECT],
+				invalidValues: [TEST_VALUES.NULL, TEST_VALUES.UNDEFINED, TEST_VALUES.OBJECT],
 			},
 		],
 	},

tests/utils/Literal.test.ts

@@ -108,6 +108,7 @@ describe('isLiteral function', () => {
 describe('parseUnknownToLiteralArray function', () => {
 	test('should return undefined when given undefined', () => {
 		expect(parseUnknownToLiteralArray(undefined)).toBeUndefined();
+		expect(parseUnknownToLiteralArray(null)).toBeUndefined();
 	});
 
 	test('should return an array with one element when given a literal', () => {
@@ -116,7 +117,6 @@ describe('parseUnknownToLiteralArray function', () => {
 		expect(parseUnknownToLiteralArray(123)).toEqual([123]);
 		expect(parseUnknownToLiteralArray(-123.45)).toEqual([-123.45]);
 		expect(parseUnknownToLiteralArray('hello')).toEqual(['hello']);
-		expect(parseUnknownToLiteralArray(null)).toEqual([null]);
 	});
 
 	test('should return an array with all literals when given an array with literals', () => {