ci: fix empty view spec fields (#9)

Author: bocembocem

src/stories/components/InputPreview/constants.ts

@@ -288,7 +288,7 @@ const order: ArraySpec = {
     viewSpec: {type: 'base', layout: 'accordeon', layoutTitle: 'Order'},
 };
 
-export const arrayOptions: ObjectSpec = {
+export const getArrayOptions = (): ObjectSpec => ({
     type: SpecTypes.Object,
     required: true,
     properties: {
@@ -338,9 +338,9 @@ export const arrayOptions: ObjectSpec = {
             'viewSpec',
         ],
     },
-};
+});
 
-export const booleanOptions: ObjectSpec = {
+export const getBooleanOptions = (): ObjectSpec => ({
     type: SpecTypes.Object,
     required: true,
     properties: {
@@ -363,9 +363,9 @@ export const booleanOptions: ObjectSpec = {
         layout: 'section',
         order: ['required', 'validator', 'viewSpec'],
     },
-};
+});
 
-export const numberOptions: ObjectSpec = {
+export const getNumberOptions = (): ObjectSpec => ({
     type: SpecTypes.Object,
     required: true,
     properties: {
@@ -400,9 +400,9 @@ export const numberOptions: ObjectSpec = {
         layout: 'section',
         order: ['required', 'maximum', 'minimum', 'format', 'validator', 'viewSpec'],
     },
-};
+});
 
-export const objectOptions: ObjectSpec = {
+export const getObjectOptions = (): ObjectSpec => ({
     type: SpecTypes.Object,
     required: true,
     properties: {
@@ -436,9 +436,9 @@ export const objectOptions: ObjectSpec = {
         layout: 'section',
         order: ['required', 'properties', 'description', 'validator', 'viewSpec'],
     },
-};
+});
 
-export const stringOptions: ObjectSpec = {
+export const getStringOptions = (): ObjectSpec => ({
     type: SpecTypes.Object,
     required: true,
     properties: {
@@ -498,4 +498,4 @@ export const stringOptions: ObjectSpec = {
             'viewSpec',
         ],
     },
-};
+});

src/stories/components/InputPreview/utils.ts

@@ -12,11 +12,11 @@ import {
 } from '../../../lib';
 
 import {
-    arrayOptions,
-    booleanOptions,
-    numberOptions,
-    objectOptions,
-    stringOptions,
+    getArrayOptions,
+    getBooleanOptions,
+    getNumberOptions,
+    getObjectOptions,
+    getStringOptions,
 } from './constants';
 
 export const transformCorrect = (spec: Spec) => {
@@ -137,51 +137,41 @@ export const getOptionsSpec = (spec: Spec, excludeOptions?: string[]) => {
     let result: ObjectSpec | null = null;
 
     if (isArraySpec(spec)) {
-        result = arrayOptions;
+        result = getArrayOptions();
     }
 
     if (isBooleanSpec(spec)) {
-        result = booleanOptions;
+        result = getBooleanOptions();
     }
 
     if (isNumberSpec(spec)) {
-        result = numberOptions;
+        result = getNumberOptions();
     }
 
     if (isObjectSpec(spec)) {
-        result = objectOptions;
+        result = getObjectOptions();
     }
 
     if (isStringSpec(spec)) {
-        result = stringOptions;
+        result = getStringOptions();
     }
 
     if (result) {
-        let viewSpec: ObjectSpec | undefined;
-
         if (result.properties?.viewSpec) {
-            viewSpec = result.properties?.viewSpec as ObjectSpec;
-            viewSpec.viewSpec = {
-                ...viewSpec.viewSpec,
-                order: viewSpec.viewSpec.order?.filter(
-                    (key) => !excludeOptions?.find((k) => k === `viewSpec.${key}`),
-                ),
-            };
+            const viewSpecOrder = (result.properties.viewSpec as ObjectSpec).viewSpec.order?.filter(
+                (key) => !excludeOptions?.find((k) => k === `viewSpec.${key}`),
+            );
+
+            (result.properties.viewSpec as ObjectSpec).viewSpec.order = viewSpecOrder?.length
+                ? viewSpecOrder
+                : undefined;
         }
 
-        result = {
-            ...result,
-            properties: {
-                ...(result.properties || {}),
-                ...(viewSpec ? {viewSpec} : {}),
-            },
-            viewSpec: {
-                ...result.viewSpec,
-                order: result.viewSpec.order?.filter(
-                    (key) => !excludeOptions?.find((k) => k === key),
-                ),
-            },
-        };
+        const order = result.viewSpec.order?.filter(
+            (key) => !excludeOptions?.find((k) => k === key),
+        );
+
+        result.viewSpec.order = order?.length ? order : undefined;
     }
 
     return result;