@nocobase/flow-engine 2.0.22 → 2.1.0-alpha.10
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/lib/components/MobilePopup.js +6 -5
- package/lib/components/subModel/AddSubModelButton.js +1 -1
- package/lib/components/subModel/utils.js +2 -2
- package/lib/flowEngine.d.ts +120 -1
- package/lib/flowEngine.js +301 -14
- package/lib/flowSettings.d.ts +14 -6
- package/lib/flowSettings.js +34 -6
- package/lib/lazy-helper.d.ts +14 -0
- package/lib/lazy-helper.js +71 -0
- package/lib/models/flowModel.js +17 -7
- package/lib/types.d.ts +35 -0
- package/lib/utils/runjsTemplateCompat.js +1 -1
- package/package.json +4 -4
- package/src/__tests__/flowEngine.modelLoaders.test.ts +245 -0
- package/src/__tests__/flowSettings.test.ts +94 -15
- package/src/__tests__/renderHiddenInConfig.test.tsx +6 -6
- package/src/__tests__/viewScopedFlowEngine.test.ts +3 -3
- package/src/components/MobilePopup.tsx +4 -2
- package/src/components/__tests__/flow-model-render-error-fallback.test.tsx +3 -3
- package/src/components/subModel/AddSubModelButton.tsx +1 -1
- package/src/components/subModel/__tests__/AddSubModelButton.test.tsx +93 -33
- package/src/components/subModel/utils.ts +1 -1
- package/src/flowEngine.ts +338 -10
- package/src/flowSettings.ts +40 -6
- package/src/lazy-helper.tsx +57 -0
- package/src/models/flowModel.tsx +18 -6
- package/src/types.ts +47 -0
- package/src/utils/runjsTemplateCompat.ts +1 -1
|
@@ -8,6 +8,10 @@
|
|
|
8
8
|
*/
|
|
9
9
|
|
|
10
10
|
import { describe, test, expect, beforeEach, afterEach, vi } from 'vitest';
|
|
11
|
+
import React from 'react';
|
|
12
|
+
import { createForm } from '@formily/core';
|
|
13
|
+
import { createSchemaField, FormProvider } from '@formily/react';
|
|
14
|
+
import { render, screen } from '@testing-library/react';
|
|
11
15
|
import { FlowSettings } from '../flowSettings';
|
|
12
16
|
import { DefaultSettingsIcon } from '../components/settings/wrappers/contextual/DefaultSettingsIcon';
|
|
13
17
|
import { FlowModel } from '../models';
|
|
@@ -142,10 +146,10 @@ describe('FlowSettings', () => {
|
|
|
142
146
|
expect(settingsItem?.sort).toBe(0);
|
|
143
147
|
});
|
|
144
148
|
|
|
145
|
-
test('should set up observable properties', () => {
|
|
149
|
+
test('should set up observable properties', async () => {
|
|
146
150
|
// Test that enabled property is reactive
|
|
147
151
|
const initialEnabled = flowSettings.enabled;
|
|
148
|
-
flowSettings.enable();
|
|
152
|
+
await flowSettings.enable();
|
|
149
153
|
expect(flowSettings.enabled).not.toBe(initialEnabled);
|
|
150
154
|
expect(flowSettings.enabled).toBe(true);
|
|
151
155
|
});
|
|
@@ -186,6 +190,43 @@ describe('FlowSettings', () => {
|
|
|
186
190
|
flowSettings.registerComponents({});
|
|
187
191
|
expect(Object.keys(flowSettings.components)).toHaveLength(0);
|
|
188
192
|
});
|
|
193
|
+
|
|
194
|
+
test('should register component loaders and load component on render', async () => {
|
|
195
|
+
const loader = vi.fn(async () => ({
|
|
196
|
+
default: () => React.createElement('div', null, 'Lazy Flow Settings Component'),
|
|
197
|
+
}));
|
|
198
|
+
|
|
199
|
+
flowSettings.registerComponentLoaders({
|
|
200
|
+
DemoFlowSettingsLazyField: loader,
|
|
201
|
+
});
|
|
202
|
+
|
|
203
|
+
expect(loader).not.toHaveBeenCalled();
|
|
204
|
+
|
|
205
|
+
const SchemaField = createSchemaField();
|
|
206
|
+
const form = createForm();
|
|
207
|
+
|
|
208
|
+
render(
|
|
209
|
+
React.createElement(
|
|
210
|
+
FormProvider,
|
|
211
|
+
{ form },
|
|
212
|
+
React.createElement(SchemaField, {
|
|
213
|
+
schema: {
|
|
214
|
+
type: 'object',
|
|
215
|
+
properties: {
|
|
216
|
+
demo: {
|
|
217
|
+
type: 'void',
|
|
218
|
+
'x-component': 'DemoFlowSettingsLazyField',
|
|
219
|
+
},
|
|
220
|
+
},
|
|
221
|
+
},
|
|
222
|
+
components: flowSettings.components,
|
|
223
|
+
}),
|
|
224
|
+
),
|
|
225
|
+
);
|
|
226
|
+
|
|
227
|
+
expect(await screen.findByText('Lazy Flow Settings Component')).toBeInTheDocument();
|
|
228
|
+
expect(loader).toHaveBeenCalledTimes(1);
|
|
229
|
+
});
|
|
189
230
|
});
|
|
190
231
|
|
|
191
232
|
describe('Scope Registration', () => {
|
|
@@ -228,30 +269,68 @@ describe('FlowSettings', () => {
|
|
|
228
269
|
});
|
|
229
270
|
|
|
230
271
|
describe('Enable/Disable Functionality', () => {
|
|
231
|
-
test('should enable flow settings', () => {
|
|
272
|
+
test('should enable flow settings', async () => {
|
|
232
273
|
expect(flowSettings.enabled).toBe(false);
|
|
233
274
|
|
|
234
|
-
flowSettings.enable();
|
|
275
|
+
await flowSettings.enable();
|
|
235
276
|
|
|
236
277
|
expect(flowSettings.enabled).toBe(true);
|
|
237
278
|
});
|
|
238
279
|
|
|
239
|
-
test('should
|
|
240
|
-
|
|
280
|
+
test('should preload model loaders before enabling flow settings', async () => {
|
|
281
|
+
const preloadSpy = vi.spyOn(engine, 'preloadModelLoaders').mockResolvedValue({
|
|
282
|
+
requested: [],
|
|
283
|
+
loaded: [],
|
|
284
|
+
failed: [],
|
|
285
|
+
});
|
|
286
|
+
|
|
287
|
+
await flowSettings.enable();
|
|
288
|
+
|
|
289
|
+
expect(preloadSpy).toHaveBeenCalledTimes(1);
|
|
241
290
|
expect(flowSettings.enabled).toBe(true);
|
|
291
|
+
});
|
|
242
292
|
|
|
243
|
-
|
|
293
|
+
test('should preload model loaders before force enabling flow settings', async () => {
|
|
294
|
+
const preloadSpy = vi.spyOn(engine, 'preloadModelLoaders').mockResolvedValue({
|
|
295
|
+
requested: [],
|
|
296
|
+
loaded: [],
|
|
297
|
+
failed: [],
|
|
298
|
+
});
|
|
299
|
+
|
|
300
|
+
await flowSettings.forceEnable();
|
|
301
|
+
|
|
302
|
+
expect(preloadSpy).toHaveBeenCalledTimes(1);
|
|
303
|
+
expect(flowSettings.enabled).toBe(true);
|
|
304
|
+
});
|
|
305
|
+
|
|
306
|
+
test('should disable flow settings', async () => {
|
|
307
|
+
await flowSettings.enable();
|
|
308
|
+
expect(flowSettings.enabled).toBe(true);
|
|
309
|
+
|
|
310
|
+
await flowSettings.disable();
|
|
244
311
|
|
|
245
312
|
expect(flowSettings.enabled).toBe(false);
|
|
246
313
|
});
|
|
247
314
|
|
|
248
|
-
test('should handle multiple enable/disable calls', () => {
|
|
249
|
-
flowSettings.enable();
|
|
250
|
-
flowSettings.enable();
|
|
315
|
+
test('should handle multiple enable/disable calls', async () => {
|
|
316
|
+
await flowSettings.enable();
|
|
317
|
+
await flowSettings.enable();
|
|
251
318
|
expect(flowSettings.enabled).toBe(true);
|
|
252
319
|
|
|
253
|
-
flowSettings.disable();
|
|
254
|
-
flowSettings.disable();
|
|
320
|
+
await flowSettings.disable();
|
|
321
|
+
await flowSettings.disable();
|
|
322
|
+
expect(flowSettings.enabled).toBe(false);
|
|
323
|
+
});
|
|
324
|
+
|
|
325
|
+
test('forceDisable should clear force-enabled state and disable flow settings', async () => {
|
|
326
|
+
await flowSettings.forceEnable();
|
|
327
|
+
expect(flowSettings.enabled).toBe(true);
|
|
328
|
+
|
|
329
|
+
await flowSettings.forceDisable();
|
|
330
|
+
|
|
331
|
+
expect(flowSettings.enabled).toBe(false);
|
|
332
|
+
|
|
333
|
+
await flowSettings.disable();
|
|
255
334
|
expect(flowSettings.enabled).toBe(false);
|
|
256
335
|
});
|
|
257
336
|
});
|
|
@@ -512,7 +591,7 @@ describe('FlowSettings', () => {
|
|
|
512
591
|
});
|
|
513
592
|
|
|
514
593
|
describe('Complex Integration Scenarios', () => {
|
|
515
|
-
test('should maintain state consistency during multiple operations', () => {
|
|
594
|
+
test('should maintain state consistency during multiple operations', async () => {
|
|
516
595
|
// Initialize with components and scopes
|
|
517
596
|
const TestComponent = () => 'TestComponent';
|
|
518
597
|
const testScope = () => 'testScope';
|
|
@@ -528,7 +607,7 @@ describe('FlowSettings', () => {
|
|
|
528
607
|
});
|
|
529
608
|
|
|
530
609
|
// Enable/disable
|
|
531
|
-
flowSettings.enable();
|
|
610
|
+
await flowSettings.enable();
|
|
532
611
|
expect(flowSettings.enabled).toBe(true);
|
|
533
612
|
|
|
534
613
|
// Verify all state is maintained
|
|
@@ -536,7 +615,7 @@ describe('FlowSettings', () => {
|
|
|
536
615
|
expect(flowSettings.scopes.testScope).toBe(testScope);
|
|
537
616
|
expect(flowSettings.getToolbarItems().find((item) => item.key === 'integration-test')).toBeDefined();
|
|
538
617
|
|
|
539
|
-
flowSettings.disable();
|
|
618
|
+
await flowSettings.disable();
|
|
540
619
|
expect(flowSettings.enabled).toBe(false);
|
|
541
620
|
|
|
542
621
|
// State should still be maintained after disable
|
|
@@ -14,7 +14,7 @@ import { FlowEngine } from '../flowEngine';
|
|
|
14
14
|
import { FlowModel, ModelRenderMode } from '../models/flowModel';
|
|
15
15
|
|
|
16
16
|
describe('FlowModel.renderHiddenInConfig', () => {
|
|
17
|
-
it('renders via renderHiddenInConfig when hidden and config enabled (React element mode, mounted)', () => {
|
|
17
|
+
it('renders via renderHiddenInConfig when hidden and config enabled (React element mode, mounted)', async () => {
|
|
18
18
|
class ElemModel extends FlowModel {
|
|
19
19
|
render() {
|
|
20
20
|
return <div data-testid="content">Content</div>;
|
|
@@ -28,14 +28,14 @@ describe('FlowModel.renderHiddenInConfig', () => {
|
|
|
28
28
|
const model = new ElemModel({ uid: 'elem-1', flowEngine: engine });
|
|
29
29
|
|
|
30
30
|
// runtime hidden => mounted result should be empty (no content/hidden)
|
|
31
|
-
engine.flowSettings.disable();
|
|
31
|
+
await engine.flowSettings.disable();
|
|
32
32
|
model.setHidden(true);
|
|
33
33
|
const { container, unmount, rerender } = render(model.render() as React.ReactElement);
|
|
34
34
|
expect(screen.queryByTestId('content')).toBeNull();
|
|
35
35
|
expect(screen.queryByTestId('hidden')).toBeNull();
|
|
36
36
|
|
|
37
37
|
// config enabled + hidden => should show renderHiddenInConfig result
|
|
38
|
-
engine.flowSettings.enable();
|
|
38
|
+
await engine.flowSettings.enable();
|
|
39
39
|
rerender(model.render() as React.ReactElement);
|
|
40
40
|
expect(screen.getByTestId('hidden').textContent).toBe('HiddenViaAPI');
|
|
41
41
|
|
|
@@ -46,7 +46,7 @@ describe('FlowModel.renderHiddenInConfig', () => {
|
|
|
46
46
|
unmount();
|
|
47
47
|
cleanup();
|
|
48
48
|
});
|
|
49
|
-
it('returns a render function when hidden and config enabled (RenderFunction mode)', () => {
|
|
49
|
+
it('returns a render function when hidden and config enabled (RenderFunction mode)', async () => {
|
|
50
50
|
class FuncModel extends FlowModel {
|
|
51
51
|
static override renderMode = ModelRenderMode.RenderFunction;
|
|
52
52
|
render() {
|
|
@@ -63,13 +63,13 @@ describe('FlowModel.renderHiddenInConfig', () => {
|
|
|
63
63
|
const model = engine.createModel({ use: 'FuncModel' }) as FuncModel;
|
|
64
64
|
|
|
65
65
|
// runtime hidden => null
|
|
66
|
-
engine.flowSettings.disable();
|
|
66
|
+
await engine.flowSettings.disable();
|
|
67
67
|
model.setHidden(true);
|
|
68
68
|
const runtimeHidden = model.render();
|
|
69
69
|
expect(runtimeHidden).toBeNull();
|
|
70
70
|
|
|
71
71
|
// config enabled + hidden => renderHiddenInConfig (function)
|
|
72
|
-
engine.flowSettings.enable();
|
|
72
|
+
await engine.flowSettings.enable();
|
|
73
73
|
const cfgHidden = model.render();
|
|
74
74
|
expect(typeof cfgHidden).toBe('function');
|
|
75
75
|
const cellNode = (cfgHidden as any)();
|
|
@@ -256,11 +256,11 @@ describe('ViewScopedFlowEngine', () => {
|
|
|
256
256
|
|
|
257
257
|
// Both children should return null from hydration because parent has flowSettingsEnabled
|
|
258
258
|
// This is the bug fix: previously only children with their own flowSettingsEnabled would return null
|
|
259
|
-
const result1 = (scoped as any).hydrateModelFromPreviousEngines({
|
|
259
|
+
const result1 = await (scoped as any).hydrateModelFromPreviousEngines({
|
|
260
260
|
parentId: 'parent-with-settings',
|
|
261
261
|
subKey: 'popup',
|
|
262
262
|
});
|
|
263
|
-
const result2 = (scoped as any).hydrateModelFromPreviousEngines({
|
|
263
|
+
const result2 = await (scoped as any).hydrateModelFromPreviousEngines({
|
|
264
264
|
parentId: 'parent-with-settings',
|
|
265
265
|
subKey: 'items',
|
|
266
266
|
});
|
|
@@ -298,7 +298,7 @@ describe('ViewScopedFlowEngine', () => {
|
|
|
298
298
|
const scoped = createViewScopedEngine(root);
|
|
299
299
|
|
|
300
300
|
// Call the private method hydrateModelFromPreviousEngines directly
|
|
301
|
-
const result = (scoped as any).hydrateModelFromPreviousEngines({
|
|
301
|
+
const result = await (scoped as any).hydrateModelFromPreviousEngines({
|
|
302
302
|
parentId: 'parent-normal',
|
|
303
303
|
subKey: 'content',
|
|
304
304
|
});
|
|
@@ -8,11 +8,13 @@
|
|
|
8
8
|
*/
|
|
9
9
|
|
|
10
10
|
import { ConfigProvider } from 'antd';
|
|
11
|
-
import { Popup } from 'antd-mobile';
|
|
12
11
|
import React, { FC, ReactNode, useMemo } from 'react';
|
|
13
|
-
import { CloseOutline } from 'antd-mobile-icons';
|
|
14
12
|
import { useMobileActionDrawerStyle } from './MobilePopup.style';
|
|
15
13
|
import { useTranslation } from 'react-i18next';
|
|
14
|
+
import { lazy } from '../lazy-helper';
|
|
15
|
+
|
|
16
|
+
const { Popup } = lazy(() => import('antd-mobile'), 'Popup');
|
|
17
|
+
const { CloseOutline } = lazy(() => import('antd-mobile-icons'), 'CloseOutline');
|
|
16
18
|
|
|
17
19
|
interface MobilePopupProps {
|
|
18
20
|
title?: string;
|
|
@@ -114,7 +114,7 @@ describe('Delete problematic model via FlowSettings menu', () => {
|
|
|
114
114
|
}
|
|
115
115
|
|
|
116
116
|
const engine = new FlowEngine();
|
|
117
|
-
engine.flowSettings.forceEnable();
|
|
117
|
+
await engine.flowSettings.forceEnable();
|
|
118
118
|
engine.registerModels({ BrokenModel });
|
|
119
119
|
const model = engine.createModel({ use: 'BrokenModel', uid: 'broken-top-2' }) as BrokenModel;
|
|
120
120
|
// satisfy FlowsFloatContextMenu styles
|
|
@@ -154,7 +154,7 @@ describe('Delete problematic model via FlowSettings menu', () => {
|
|
|
154
154
|
}
|
|
155
155
|
|
|
156
156
|
const engine = new FlowEngine();
|
|
157
|
-
engine.flowSettings.forceEnable();
|
|
157
|
+
await engine.flowSettings.forceEnable();
|
|
158
158
|
engine.registerModels({ ParentModel, BrokenChild });
|
|
159
159
|
const parent = engine.createModel({ use: 'ParentModel', uid: 'parent-3' }) as ParentModel;
|
|
160
160
|
const child = engine.createModel({ use: 'BrokenChild', uid: 'child-3' }) as BrokenChild;
|
|
@@ -200,7 +200,7 @@ describe('Delete problematic model via FlowSettings menu', () => {
|
|
|
200
200
|
}
|
|
201
201
|
|
|
202
202
|
const engine = new FlowEngine();
|
|
203
|
-
engine.flowSettings.forceEnable();
|
|
203
|
+
await engine.flowSettings.forceEnable();
|
|
204
204
|
engine.registerModels({ ParentModel, RenderFnChild });
|
|
205
205
|
const parent = engine.createModel({ use: 'ParentModel', uid: 'parent-4' }) as ParentModel;
|
|
206
206
|
const child = engine.createModel({ use: 'RenderFnChild', uid: 'cell-4' }) as RenderFnChild;
|
|
@@ -610,7 +610,7 @@ const AddSubModelButtonCore = function AddSubModelButton({
|
|
|
610
610
|
let addedModel: FlowModel | undefined;
|
|
611
611
|
|
|
612
612
|
try {
|
|
613
|
-
addedModel = model.flowEngine.
|
|
613
|
+
addedModel = await model.flowEngine.createModelAsync({
|
|
614
614
|
..._.cloneDeep(createOpts),
|
|
615
615
|
parentId: model.uid,
|
|
616
616
|
subKey: subModelKey,
|
|
@@ -25,7 +25,7 @@ describe('AddSubModelButton - preset settings open on add', () => {
|
|
|
25
25
|
test('calls openFlowSettings with preset=true for subModel with preset steps', async () => {
|
|
26
26
|
// Arrange: set up engine and models
|
|
27
27
|
const engine = new FlowEngine();
|
|
28
|
-
engine.flowSettings.forceEnable();
|
|
28
|
+
await engine.flowSettings.forceEnable();
|
|
29
29
|
|
|
30
30
|
class ParentModel extends FlowModel {}
|
|
31
31
|
|
|
@@ -99,12 +99,70 @@ describe('AddSubModelButton - preset settings open on add', () => {
|
|
|
99
99
|
});
|
|
100
100
|
});
|
|
101
101
|
|
|
102
|
+
describe('AddSubModelButton - model loader integration', () => {
|
|
103
|
+
test('resolves model loaders before creating sub models', async () => {
|
|
104
|
+
const engine = new FlowEngine();
|
|
105
|
+
await engine.flowSettings.forceEnable();
|
|
106
|
+
|
|
107
|
+
class ParentModel extends FlowModel {}
|
|
108
|
+
class ChildModel extends FlowModel {}
|
|
109
|
+
|
|
110
|
+
const childLoader = vi.fn(async () => ({ ChildModel }));
|
|
111
|
+
|
|
112
|
+
engine.registerModels({ ParentModel });
|
|
113
|
+
engine.registerModelLoaders({
|
|
114
|
+
ChildModel: {
|
|
115
|
+
loader: childLoader,
|
|
116
|
+
},
|
|
117
|
+
});
|
|
118
|
+
|
|
119
|
+
const parent = engine.createModel<ParentModel>({ use: 'ParentModel', uid: 'parent-loader' });
|
|
120
|
+
|
|
121
|
+
render(
|
|
122
|
+
<FlowEngineProvider engine={engine}>
|
|
123
|
+
<ConfigProvider>
|
|
124
|
+
<App>
|
|
125
|
+
<AddSubModelButton
|
|
126
|
+
model={parent}
|
|
127
|
+
subModelKey="items"
|
|
128
|
+
items={[
|
|
129
|
+
{
|
|
130
|
+
key: 'child',
|
|
131
|
+
label: 'Add Child',
|
|
132
|
+
createModelOptions: { use: 'ChildModel' },
|
|
133
|
+
},
|
|
134
|
+
]}
|
|
135
|
+
>
|
|
136
|
+
Add SubModel
|
|
137
|
+
</AddSubModelButton>
|
|
138
|
+
</App>
|
|
139
|
+
</ConfigProvider>
|
|
140
|
+
</FlowEngineProvider>,
|
|
141
|
+
);
|
|
142
|
+
|
|
143
|
+
await act(async () => {
|
|
144
|
+
await userEvent.click(screen.getByText('Add SubModel'));
|
|
145
|
+
});
|
|
146
|
+
|
|
147
|
+
await waitFor(() => expect(screen.getByText('Add Child')).toBeInTheDocument());
|
|
148
|
+
|
|
149
|
+
await act(async () => {
|
|
150
|
+
await userEvent.click(screen.getByText('Add Child'));
|
|
151
|
+
});
|
|
152
|
+
|
|
153
|
+
await waitFor(() => expect(childLoader).toHaveBeenCalledTimes(1));
|
|
154
|
+
const items = parent.subModels.items as FlowModel[];
|
|
155
|
+
expect(Array.isArray(items)).toBe(true);
|
|
156
|
+
expect(items[0]).toBeInstanceOf(ChildModel);
|
|
157
|
+
});
|
|
158
|
+
});
|
|
159
|
+
|
|
102
160
|
describe('AddSubModelButton - async group children (nested)', () => {
|
|
103
161
|
const sleep = (ms: number) => new Promise((resolve) => setTimeout(resolve, ms));
|
|
104
162
|
|
|
105
163
|
it('renders group and nested async group leaf items', async () => {
|
|
106
164
|
const engine = new FlowEngine();
|
|
107
|
-
engine.flowSettings.forceEnable();
|
|
165
|
+
await engine.flowSettings.forceEnable();
|
|
108
166
|
class Parent extends FlowModel {}
|
|
109
167
|
engine.registerModels({ Parent });
|
|
110
168
|
const parent = engine.createModel<FlowModel>({ use: 'Parent', uid: 'p1' });
|
|
@@ -162,7 +220,7 @@ describe('AddSubModelButton - async group children (nested)', () => {
|
|
|
162
220
|
describe('transformItems - searchable flags', () => {
|
|
163
221
|
it('preserves searchable + placeholder on non-group submenu items', async () => {
|
|
164
222
|
const engine = new FlowEngine();
|
|
165
|
-
engine.flowSettings.forceEnable();
|
|
223
|
+
await engine.flowSettings.forceEnable();
|
|
166
224
|
class Parent extends FlowModel {}
|
|
167
225
|
engine.registerModels({ Parent });
|
|
168
226
|
const parent = engine.createModel<FlowModel>({ use: 'Parent' });
|
|
@@ -193,7 +251,7 @@ describe('transformItems - searchable flags', () => {
|
|
|
193
251
|
describe('transformItems - hide', () => {
|
|
194
252
|
it('filters items by hide flag/function recursively', async () => {
|
|
195
253
|
const engine = new FlowEngine();
|
|
196
|
-
engine.flowSettings.forceEnable();
|
|
254
|
+
await engine.flowSettings.forceEnable();
|
|
197
255
|
class Parent extends FlowModel {}
|
|
198
256
|
engine.registerModels({ Parent });
|
|
199
257
|
const parent = engine.createModel<FlowModel>({ use: 'Parent', uid: 'p-hide' });
|
|
@@ -239,7 +297,7 @@ describe('transformItems - hide', () => {
|
|
|
239
297
|
|
|
240
298
|
it('removes group when all children are hidden (even with async hide)', async () => {
|
|
241
299
|
const engine = new FlowEngine();
|
|
242
|
-
engine.flowSettings.forceEnable();
|
|
300
|
+
await engine.flowSettings.forceEnable();
|
|
243
301
|
class Parent extends FlowModel {}
|
|
244
302
|
engine.registerModels({ Parent });
|
|
245
303
|
const parent = engine.createModel<FlowModel>({ use: 'Parent', uid: 'p-empty-group' });
|
|
@@ -272,7 +330,7 @@ describe('transformItems - hide', () => {
|
|
|
272
330
|
|
|
273
331
|
it('supports async hide functions and disables cache', async () => {
|
|
274
332
|
const engine = new FlowEngine();
|
|
275
|
-
engine.flowSettings.forceEnable();
|
|
333
|
+
await engine.flowSettings.forceEnable();
|
|
276
334
|
class Parent extends FlowModel {}
|
|
277
335
|
engine.registerModels({ Parent });
|
|
278
336
|
const parent = engine.createModel<FlowModel>({ use: 'Parent', uid: 'p-async-hide' });
|
|
@@ -300,7 +358,7 @@ describe('transformItems - hide', () => {
|
|
|
300
358
|
|
|
301
359
|
it('shows items when hide function throws (conservative fallback)', async () => {
|
|
302
360
|
const engine = new FlowEngine();
|
|
303
|
-
engine.flowSettings.forceEnable();
|
|
361
|
+
await engine.flowSettings.forceEnable();
|
|
304
362
|
class Parent extends FlowModel {}
|
|
305
363
|
engine.registerModels({ Parent });
|
|
306
364
|
const parent = engine.createModel<FlowModel>({ use: 'Parent', uid: 'p-hide-throws' });
|
|
@@ -331,15 +389,15 @@ describe('transformItems - toggleable items', () => {
|
|
|
331
389
|
class ToggleParent extends FlowModel {}
|
|
332
390
|
class ToggleChild extends FlowModel {}
|
|
333
391
|
|
|
334
|
-
const setupEngine = () => {
|
|
392
|
+
const setupEngine = async () => {
|
|
335
393
|
const engine = new FlowEngine();
|
|
336
|
-
engine.flowSettings.forceEnable();
|
|
394
|
+
await engine.flowSettings.forceEnable();
|
|
337
395
|
engine.registerModels({ ToggleParent, ToggleChild });
|
|
338
396
|
return engine;
|
|
339
397
|
};
|
|
340
398
|
|
|
341
399
|
it('marks toggleable item as active when matching sub model exists', async () => {
|
|
342
|
-
const engine = setupEngine();
|
|
400
|
+
const engine = await setupEngine();
|
|
343
401
|
const parent = engine.createModel<ToggleParent>({ use: 'ToggleParent', uid: 'toggle-parent-on' });
|
|
344
402
|
const child = engine.createModel<ToggleChild>({ use: 'ToggleChild', uid: 'toggle-child-on' });
|
|
345
403
|
parent.addSubModel('items', child);
|
|
@@ -371,7 +429,7 @@ describe('transformItems - toggleable items', () => {
|
|
|
371
429
|
});
|
|
372
430
|
|
|
373
431
|
it('infers useModel from createModelOptions when toggleable is enabled', async () => {
|
|
374
|
-
const engine = setupEngine();
|
|
432
|
+
const engine = await setupEngine();
|
|
375
433
|
const parent = engine.createModel<ToggleParent>({ use: 'ToggleParent', uid: 'toggle-parent-infer' });
|
|
376
434
|
const child = engine.createModel<ToggleChild>({ use: 'ToggleChild', uid: 'toggle-child-infer' });
|
|
377
435
|
parent.addSubModel('items', child);
|
|
@@ -397,7 +455,7 @@ describe('transformItems - toggleable items', () => {
|
|
|
397
455
|
});
|
|
398
456
|
|
|
399
457
|
it('keeps toggleable item off when sub model missing', async () => {
|
|
400
|
-
const engine = setupEngine();
|
|
458
|
+
const engine = await setupEngine();
|
|
401
459
|
const parent = engine.createModel<ToggleParent>({ use: 'ToggleParent', uid: 'toggle-parent-off' });
|
|
402
460
|
|
|
403
461
|
const definition: SubModelItem[] = [
|
|
@@ -420,7 +478,7 @@ describe('transformItems - toggleable items', () => {
|
|
|
420
478
|
});
|
|
421
479
|
|
|
422
480
|
it('respects keepDropdownOpen override on toggleable items', async () => {
|
|
423
|
-
const engine = setupEngine();
|
|
481
|
+
const engine = await setupEngine();
|
|
424
482
|
const parent = engine.createModel<ToggleParent>({ use: 'ToggleParent', uid: 'toggle-parent-keep' });
|
|
425
483
|
|
|
426
484
|
const definition: SubModelItem[] = [
|
|
@@ -443,7 +501,7 @@ describe('transformItems - toggleable items', () => {
|
|
|
443
501
|
|
|
444
502
|
it('removes object sub model via default remove handler when toggleDetector provided', async () => {
|
|
445
503
|
const engine = new FlowEngine();
|
|
446
|
-
engine.flowSettings.forceEnable();
|
|
504
|
+
await engine.flowSettings.forceEnable();
|
|
447
505
|
|
|
448
506
|
class ObjectParent extends FlowModel {}
|
|
449
507
|
class ObjectChild extends FlowModel {}
|
|
@@ -481,6 +539,8 @@ describe('transformItems - toggleable items', () => {
|
|
|
481
539
|
</FlowEngineProvider>,
|
|
482
540
|
);
|
|
483
541
|
|
|
542
|
+
await waitFor(() => expect(screen.getByText('Toggle Menu')).toBeInTheDocument());
|
|
543
|
+
|
|
484
544
|
await act(async () => {
|
|
485
545
|
await userEvent.click(screen.getByText('Toggle Menu'));
|
|
486
546
|
});
|
|
@@ -519,16 +579,16 @@ describe('transformItems - caching behaviour', () => {
|
|
|
519
579
|
class CacheParent extends FlowModel {}
|
|
520
580
|
class CacheChild extends FlowModel {}
|
|
521
581
|
|
|
522
|
-
const setupEngine = () => {
|
|
582
|
+
const setupEngine = async () => {
|
|
523
583
|
const engine = new FlowEngine();
|
|
524
|
-
engine.flowSettings.forceEnable();
|
|
584
|
+
await engine.flowSettings.forceEnable();
|
|
525
585
|
engine.registerModels({ CacheParent, CacheChild });
|
|
526
586
|
const parent = engine.createModel<CacheParent>({ use: 'CacheParent', uid: 'cache-parent' });
|
|
527
587
|
return { engine, parent };
|
|
528
588
|
};
|
|
529
589
|
|
|
530
590
|
it('reuses cached result when no toggleable items exist', async () => {
|
|
531
|
-
const { parent } = setupEngine();
|
|
591
|
+
const { parent } = await setupEngine();
|
|
532
592
|
const definition: SubModelItem[] = [{ key: 'basic', label: 'Basic', createModelOptions: { use: 'CacheChild' } }];
|
|
533
593
|
|
|
534
594
|
const factory = transformItems(definition, parent, 'items', 'array');
|
|
@@ -541,7 +601,7 @@ describe('transformItems - caching behaviour', () => {
|
|
|
541
601
|
});
|
|
542
602
|
|
|
543
603
|
it('refreshes toggle state after new sub model is added', async () => {
|
|
544
|
-
const { parent, engine } = setupEngine();
|
|
604
|
+
const { parent, engine } = await setupEngine();
|
|
545
605
|
const createDefinition = (): SubModelItem[] => [
|
|
546
606
|
{
|
|
547
607
|
key: 'toggleable',
|
|
@@ -570,7 +630,7 @@ describe('transformItems - caching behaviour', () => {
|
|
|
570
630
|
describe('AddSubModelButton - refreshTargets linkage', () => {
|
|
571
631
|
it('clicking an item with refreshTargets triggers toggle recomputation on target branch', async () => {
|
|
572
632
|
const engine = new FlowEngine();
|
|
573
|
-
engine.flowSettings.forceEnable();
|
|
633
|
+
await engine.flowSettings.forceEnable();
|
|
574
634
|
|
|
575
635
|
class Parent extends FlowModel {}
|
|
576
636
|
class ToggleModel extends FlowModel {}
|
|
@@ -642,7 +702,7 @@ describe('AddSubModelButton - base class menu groups', () => {
|
|
|
642
702
|
|
|
643
703
|
it('renders async children provided by subModelBaseClasses', async () => {
|
|
644
704
|
const engine = new FlowEngine();
|
|
645
|
-
engine.flowSettings.forceEnable();
|
|
705
|
+
await engine.flowSettings.forceEnable();
|
|
646
706
|
|
|
647
707
|
class Parent extends FlowModel {}
|
|
648
708
|
class AsyncLeaf extends FlowModel {}
|
|
@@ -687,7 +747,7 @@ describe('AddSubModelButton - base class menu groups', () => {
|
|
|
687
747
|
|
|
688
748
|
it('skips base class groups whose children resolve to empty', async () => {
|
|
689
749
|
const engine = new FlowEngine();
|
|
690
|
-
engine.flowSettings.forceEnable();
|
|
750
|
+
await engine.flowSettings.forceEnable();
|
|
691
751
|
|
|
692
752
|
class Parent extends FlowModel {}
|
|
693
753
|
class EmptyLeaf extends FlowModel {}
|
|
@@ -739,7 +799,7 @@ describe('AddSubModelButton - base class menu groups', () => {
|
|
|
739
799
|
|
|
740
800
|
it('renders submenu base class with children and respects meta.sort', async () => {
|
|
741
801
|
const engine = new FlowEngine();
|
|
742
|
-
engine.flowSettings.forceEnable();
|
|
802
|
+
await engine.flowSettings.forceEnable();
|
|
743
803
|
|
|
744
804
|
class Parent extends FlowModel {}
|
|
745
805
|
class Leaf extends FlowModel {}
|
|
@@ -803,7 +863,7 @@ describe('AddSubModelButton - base class menu groups', () => {
|
|
|
803
863
|
|
|
804
864
|
it('merges explicit items with base class and grouped sources', async () => {
|
|
805
865
|
const engine = new FlowEngine();
|
|
806
|
-
engine.flowSettings.forceEnable();
|
|
866
|
+
await engine.flowSettings.forceEnable();
|
|
807
867
|
|
|
808
868
|
class Parent extends FlowModel {}
|
|
809
869
|
class BaseChild extends FlowModel {}
|
|
@@ -862,7 +922,7 @@ describe('AddSubModelButton - base class menu groups', () => {
|
|
|
862
922
|
describe('AddSubModelButton - toggle interactions', () => {
|
|
863
923
|
it('removes existing toggleable sub model and triggers callbacks', async () => {
|
|
864
924
|
const engine = new FlowEngine();
|
|
865
|
-
engine.flowSettings.forceEnable();
|
|
925
|
+
await engine.flowSettings.forceEnable();
|
|
866
926
|
|
|
867
927
|
class ToggleParent extends FlowModel {}
|
|
868
928
|
const destroySpy = vi.fn();
|
|
@@ -926,7 +986,7 @@ describe('AddSubModelButton - toggle interactions', () => {
|
|
|
926
986
|
|
|
927
987
|
it('creates toggleable sub model and runs lifecycle callbacks', async () => {
|
|
928
988
|
const engine = new FlowEngine();
|
|
929
|
-
engine.flowSettings.forceEnable();
|
|
989
|
+
await engine.flowSettings.forceEnable();
|
|
930
990
|
|
|
931
991
|
class ToggleParent extends FlowModel {}
|
|
932
992
|
const saveSpy = vi.fn();
|
|
@@ -998,7 +1058,7 @@ describe('AddSubModelButton - toggle interactions', () => {
|
|
|
998
1058
|
|
|
999
1059
|
it('updates toggle state after external sub model removal', async () => {
|
|
1000
1060
|
const engine = new FlowEngine();
|
|
1001
|
-
engine.flowSettings.forceEnable();
|
|
1061
|
+
await engine.flowSettings.forceEnable();
|
|
1002
1062
|
|
|
1003
1063
|
class ToggleParent extends FlowModel {}
|
|
1004
1064
|
class ToggleChild extends FlowModel {}
|
|
@@ -1063,9 +1123,9 @@ describe('AddSubModelButton toggleable behavior', () => {
|
|
|
1063
1123
|
duplicate = vi.fn().mockResolvedValue(null);
|
|
1064
1124
|
}
|
|
1065
1125
|
|
|
1066
|
-
function setup() {
|
|
1126
|
+
async function setup() {
|
|
1067
1127
|
const engine = new FlowEngine();
|
|
1068
|
-
engine.flowSettings.forceEnable();
|
|
1128
|
+
await engine.flowSettings.forceEnable();
|
|
1069
1129
|
engine.registerModels({ ToggleModel });
|
|
1070
1130
|
engine.setModelRepository(new FakeRepo());
|
|
1071
1131
|
|
|
@@ -1118,7 +1178,7 @@ describe('AddSubModelButton toggleable behavior', () => {
|
|
|
1118
1178
|
});
|
|
1119
1179
|
|
|
1120
1180
|
test('keeps dropdown open and preserves loaded children on toggle add/remove', async () => {
|
|
1121
|
-
const { engine, ui } = setup();
|
|
1181
|
+
const { engine, ui } = await setup();
|
|
1122
1182
|
const user = userEvent.setup();
|
|
1123
1183
|
|
|
1124
1184
|
render(ui);
|
|
@@ -1163,7 +1223,7 @@ describe('AddSubModelButton toggleable behavior', () => {
|
|
|
1163
1223
|
});
|
|
1164
1224
|
|
|
1165
1225
|
test('toggle state updates without menu closing', async () => {
|
|
1166
|
-
const { ui } = setup();
|
|
1226
|
+
const { ui } = await setup();
|
|
1167
1227
|
const user = userEvent.setup();
|
|
1168
1228
|
|
|
1169
1229
|
render(ui);
|
|
@@ -1181,7 +1241,7 @@ describe('AddSubModelButton toggleable behavior', () => {
|
|
|
1181
1241
|
|
|
1182
1242
|
test('nested submenu (static items) toggle keeps menu open and reflects state', async () => {
|
|
1183
1243
|
const engine = new FlowEngine();
|
|
1184
|
-
engine.flowSettings.forceEnable();
|
|
1244
|
+
await engine.flowSettings.forceEnable();
|
|
1185
1245
|
engine.registerModels({ ToggleModel });
|
|
1186
1246
|
const parent = engine.createModel<FlowModel>({ use: FlowModel });
|
|
1187
1247
|
|
|
@@ -1263,7 +1323,7 @@ describe('AddSubModelButton toggleable behavior', () => {
|
|
|
1263
1323
|
|
|
1264
1324
|
test('submenu (second-level) toggleable stays open and updates state', async () => {
|
|
1265
1325
|
const engine = new FlowEngine();
|
|
1266
|
-
engine.flowSettings.forceEnable();
|
|
1326
|
+
await engine.flowSettings.forceEnable();
|
|
1267
1327
|
engine.registerModels({ ToggleModel });
|
|
1268
1328
|
engine.setModelRepository(new FakeRepo());
|
|
1269
1329
|
vi.spyOn(engine.flowSettings, 'open').mockResolvedValue(false as any);
|
|
@@ -1322,7 +1382,7 @@ describe('AddSubModelButton toggleable behavior', () => {
|
|
|
1322
1382
|
|
|
1323
1383
|
test('top-level toggle updates after opening a second-level branch', async () => {
|
|
1324
1384
|
const engine = new FlowEngine();
|
|
1325
|
-
engine.flowSettings.forceEnable();
|
|
1385
|
+
await engine.flowSettings.forceEnable();
|
|
1326
1386
|
engine.registerModels({ ToggleModel });
|
|
1327
1387
|
engine.setModelRepository(new FakeRepo());
|
|
1328
1388
|
vi.spyOn(engine.flowSettings, 'open').mockResolvedValue(false as any);
|
|
@@ -7,7 +7,7 @@
|
|
|
7
7
|
* For more information, please refer to: https://www.nocobase.com/agreement.
|
|
8
8
|
*/
|
|
9
9
|
|
|
10
|
-
import
|
|
10
|
+
import _ from 'lodash';
|
|
11
11
|
import type { Collection } from '../../data-source';
|
|
12
12
|
import { FlowModelContext } from '../../flowContext';
|
|
13
13
|
import { FlowModelMeta, ModelConstructor } from '../../types';
|