fix: Check all expressions in dialog

Composer/packages/client/__tests__/store/reducer/reducer.test.js

@@ -6,26 +6,26 @@ import { reducer } from '../../../src/store/reducer/index';
 
 const mockResponse = {
   data: {
-    dialogs: 'test dialogs',
-    lgFiles: 'test lgFiles',
+    dialogs: ['test dialogs'],
+    lgFiles: ['test lgFiles'],
     schemas: 'test schemas',
   },
 };
 
 describe('test all reducer handlers', () => {
   it('test getProjectSuccess reducer', () => {
     const result = reducer({}, { type: ActionTypes.GET_PROJECT_SUCCESS, payload: { response: mockResponse } });
-    expect(result.dialogs).toBe('test dialogs');
-    expect(result.lgFiles).toBe('test lgFiles');
+    expect(result.dialogs[0]).toBe('test dialogs');
+    expect(result.lgFiles[0]).toBe('test lgFiles');
     expect(result.schemas).toBe('test schemas');
   });
   it('test createDialogSuccess reducer', () => {
     const result = reducer({}, { type: ActionTypes.CREATE_DIALOG_SUCCESS, payload: { response: mockResponse } });
-    expect(result.dialogs).toBe('test dialogs');
+    expect(result.dialogs[0]).toBe('test dialogs');
   });
   it('test updateLgTemplate reducer', () => {
     const result = reducer({}, { type: ActionTypes.UPDATE_LG_SUCCESS, payload: { response: mockResponse } });
-    expect(result.lgFiles).toBe('test lgFiles');
+    expect(result.lgFiles[0]).toBe('test lgFiles');
   });
 
   it('test getStorageFileSuccess reducer', () => {

Composer/packages/client/__tests__/utils/convertUtils/parsePathToFocused.test.ts

@@ -0,0 +1,15 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License
+
+import { parsePathToFocused } from '../../../src/utils/convertUtils';
+
+describe('parsePathToFocused', () => {
+  it('should return focusedPath', () => {
+    expect(parsePathToFocused('')).toBe('');
+    expect(parsePathToFocused('main.triggers[0].actions[0]')).toBe('triggers[0].actions[0]');
+    expect(parsePathToFocused('main.triggers[0].actions[0].actions[1]')).toBe('triggers[0].actions[0].actions[1]');
+    expect(parsePathToFocused('main.triggers[0].actions[0].elseActions[1]')).toBe(
+      'triggers[0].actions[0].elseActions[1]'
+    );
+  });
+});

Composer/packages/lib/shared/__tests__/convertUtils/parsePathToSelected.test.ts → Composer/packages/client/__tests__/utils/convertUtils/parsePathToSelected.test.ts

@@ -1,7 +1,7 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License
 
-import { parsePathToSelected } from '../../src';
+import { parsePathToSelected } from '../../../src/utils/convertUtils';
 
 describe('parsePathToSelected', () => {
   it('should return selected path', () => {

Composer/packages/lib/shared/__tests__/convertUtils/parseTypeToFragment.test.ts → Composer/packages/client/__tests__/utils/convertUtils/parseTypeToFragment.test.ts

@@ -1,7 +1,8 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License
+import { PromptTab } from '@bfc/shared';
 
-import { parseTypeToFragment, PromptTab } from './../../src';
+import { parseTypeToFragment } from '../../../src/utils/convertUtils';
 
 describe('parseTypeToFragment', () => {
   it('should return corrent tab name', () => {

Composer/packages/client/src/utils/convertUtils/parsePathToFocused.ts

@@ -0,0 +1,21 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License.
+
+import { FieldNames } from '@bfc/shared';
+
+import { parsePathToSelected } from './parsePathToSelected';
+
+export function parsePathToFocused(path: string): string {
+  //path is like main.triggers[0].actions[0]
+
+  const trigger = parsePathToSelected(path);
+
+  const list = path.split('.');
+
+  const matchActions = list.filter(x => x.startsWith(FieldNames.Actions) || x.startsWith(FieldNames.ElseActions));
+
+  if (matchActions.length > 0) {
+    return `${trigger}.${matchActions.join('.')}`;
+  }
+  return '';
+}

Composer/packages/lib/shared/src/convertUtils/parsePathToSelected.ts → Composer/packages/client/src/utils/convertUtils/parsePathToSelected.ts

@@ -2,7 +2,7 @@
 // Licensed under the MIT License.
 
 export function parsePathToSelected(path: string): string {
-  //path is like main.trigers[0].actions[0]
+  //path is like main.triggers[0].actions[0]
 
   const triggerPattern = /triggers\[(\d+)\]/g;
   const matchTriggers = triggerPattern.exec(path);

Composer/packages/client/src/utils/convertUtils/parseTypeToFragment.ts

@@ -0,0 +1,22 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License.
+
+import { dialogGroups, DialogGroup, PromptTab } from '@bfc/shared';
+
+export function parseTypeToFragment(type: string, property: string): string {
+  const inputTypes = dialogGroups[DialogGroup.INPUT].types;
+  const index = inputTypes.findIndex(t => t === type);
+  if (index >= 0) {
+    switch (property) {
+      case 'prompt':
+        return PromptTab.BOT_ASKS;
+      case 'property':
+      case 'choices':
+      case 'outputFormat':
+        return PromptTab.USER_INPUT;
+      default:
+        return PromptTab.OTHER;
+    }
+  }
+  return '';
+}

Composer/packages/client/src/utils/navigation.ts

@@ -4,10 +4,10 @@
 import cloneDeep from 'lodash/cloneDeep';
 import { navigate, NavigateOptions } from '@reach/router';
 import { Diagnostic } from '@bfc/indexers';
-import { parsePathToFocused, parsePathToSelected, parseTypeToFragment } from '@bfc/shared';
 
 import { BreadcrumbItem, DesignPageLocation } from '../store/types';
 
+import { parsePathToFocused, parsePathToSelected, parseTypeToFragment } from './convertUtils';
 import { BASEPATH } from './../constants/index';
 import { resolveToBasePath } from './fileUtil';
 

Composer/packages/extensions/obiformeditor/src/Form/ArrayFieldTemplate/styles.ts

@@ -18,6 +18,7 @@ export const arrayItem = css`
 export const arrayItemField = css`
   flex: 1;
   display: flex;
+  min-width: 0;
 `;
 
 export const arrayItemInputFieldContainer = css`

Composer/packages/extensions/obiformeditor/src/Form/ObjectFieldTemplate/styles.css

@@ -1,5 +1,6 @@
 .ObjectItemInline {
   flex: 1;
+  min-width: 0;
 }
 
 .ObjectItemInline + .ObjectItemInline {