fix(wren-ui): fix tests. Remove nil values in view properties when building mdl

wren-ui/src/apollo/server/mdl/mdlBuilder.ts

@@ -1,4 +1,4 @@
-import { isEmpty } from 'lodash';
+import { isEmpty, isNil, pick, pickBy } from 'lodash';
 import {
   Model,
   ModelColumn,
@@ -114,14 +114,21 @@ export class MDLBuilder implements IMDLBuilder {
       // if putting properties not string, it will throw error
       // filter out properties that have string value
       const properties = JSON.parse(view.properties) || {};
+
+      // filter out properties that are not null or undefined
+      // and are in the list of properties that are allowed
+      const viewProperties = pickBy(properties, (value, key) => {
+        return (
+          !isNil(value) &&
+          ['displayName', 'description', 'question', 'summary'].includes(key)
+        );
+      });
+
       return {
         name: view.name,
         statement: view.statement,
         properties: {
-          displayName: properties.displayName,
-          description: properties.description,
-          question: properties.question,
-          summary: properties.summary,
+          ...viewProperties,
 
           // viewId will be passed back in other APIs
           // to identify the view

wren-ui/src/apollo/server/mdl/test/mdlBuilder.test.ts

@@ -392,7 +392,11 @@ describe('MDLBuilder', () => {
       {
         name: 'view',
         statement: 'select * from order',
-        properties: { description: 'foo view', displayName: 'view' },
+        properties: {
+          description: 'foo view',
+          displayName: 'view',
+          viewId: '1',
+        },
       },
     ] as ViewMDL[];