fm453/nocobase
1
0
Fork 0
mirror of https://gitee.com/nocobase/nocobase.git synced 2025-07-01 18:52:20 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: tests error

Browse Source
This commit is contained in:
gchust 2025-06-30 18:05:47 +08:00
parent 5068fba8a5
commit 3286db836f
1 changed files with 21 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

42
packages/core/flow-engine/src/models/__tests__/flowModel.test.ts
View File

@ -185,13 +185,10 @@ describe('FlowModel', () => {
expect(model.forks.size).toBe(0); expect(model.forks.size).toBe(0);
}); });
test('should handle missing flowEngine gracefully', () => { test('should throw error when flowEngine is missing', () => {
expect(() => { expect(() => {
new FlowModel({} as any); new FlowModel({} as any);
}).not.toThrow(); }).toThrow('FlowModel must be initialized with a FlowEngine instance.');
const model = new FlowModel({} as any);
expect(model.flowEngine).toBeUndefined();
}); });
test('should initialize emitter', () => { test('should initialize emitter', () => {
@ -426,11 +423,10 @@ describe('FlowModel', () => {
}); });
test('should throw error when FlowEngine not available', async () => { test('should throw error when FlowEngine not available', async () => {
const modelWithoutEngine = new TestFlowModel({ uid: 'test' } as any); // Since FlowModel constructor now requires flowEngine, we test the error at construction time
const flow = createBasicFlowDefinition(); expect(() => {
TestFlowModel.registerFlow(flow); new TestFlowModel({ uid: 'test' } as any);
}).toThrow('FlowModel must be initialized with a FlowEngine instance.');
await expect(modelWithoutEngine.applyFlow(flow.key)).rejects.toThrow('FlowEngine not available');
}); });
test('should handle FlowExitException correctly', async () => { test('should handle FlowExitException correctly', async () => {
@ -1145,11 +1141,10 @@ describe('FlowModel', () => {
}); });
test('should throw error when FlowEngine not set', async () => { test('should throw error when FlowEngine not set', async () => {
const modelWithoutEngine = new FlowModel({ uid: 'test' } as any); // Since FlowModel constructor now requires flowEngine, we test the error at construction time
expect(() => {
await expect(modelWithoutEngine.save()).rejects.toThrow( new FlowModel({ uid: 'test' } as any);
'FlowEngine is not set on this model. Please set flowEngine before saving', }).toThrow('FlowModel must be initialized with a FlowEngine instance.');
);
}); });
test('should handle save operation failures', async () => { test('should handle save operation failures', async () => {
@ -1172,9 +1167,10 @@ describe('FlowModel', () => {
}); });
test('should throw error when FlowEngine not available for destroy', async () => { test('should throw error when FlowEngine not available for destroy', async () => {
const modelWithoutEngine = new FlowModel({ uid: 'test' } as any); // Since FlowModel constructor now requires flowEngine, we test the error at construction time
expect(() => {
await expect(modelWithoutEngine.destroy()).rejects.toThrow('FlowEngine is not set on this model'); new FlowModel({ uid: 'test' } as any);
}).toThrow('FlowModel must be initialized with a FlowEngine instance.');
}); });
test('should clean up resources on remove', () => { test('should clean up resources on remove', () => {
@ -1225,7 +1221,7 @@ describe('FlowModel', () => {
}); });
describe('serialization', () => { describe('serialization', () => {
test('should serialize basic model data, only saving option props', () => { test('should serialize basic model data, excluding props and flowEngine', () => {
model.sortIndex = 5; model.sortIndex = 5;
model.setProps({ name: 'Test Model', value: 42 }); model.setProps({ name: 'Test Model', value: 42 });
model.setStepParams({ model.setStepParams({
@ -1236,11 +1232,13 @@ describe('FlowModel', () => {
expect(serialized).toEqual({ expect(serialized).toEqual({
uid: model.uid, uid: model.uid,
props: expect.objectContaining({ testProp: 'value' }),
stepParams: expect.objectContaining({ flow1: { step1: { param1: 'value1' } } }), stepParams: expect.objectContaining({ flow1: { step1: { param1: 'value1' } } }),
sortIndex: 5, sortIndex: 5,
subModels: expect.any(Object), subModels: expect.any(Object),
}); });
// props should be excluded from serialization
expect(serialized.props).toBeUndefined();
expect(serialized.flowEngine).toBeUndefined();
}); });
test('should serialize empty model correctly', () => { test('should serialize empty model correctly', () => {
@ -1256,11 +1254,13 @@ describe('FlowModel', () => {
expect(serialized).toEqual({ expect(serialized).toEqual({
uid: 'empty-model', uid: 'empty-model',
props: expect.any(Object),
stepParams: expect.any(Object), stepParams: expect.any(Object),
sortIndex: expect.any(Number), sortIndex: expect.any(Number),
subModels: expect.any(Object), subModels: expect.any(Object),
}); });
// props should be excluded from serialization
expect(serialized.props).toBeUndefined();
expect(serialized.flowEngine).toBeUndefined();
}); });
}); });
}); });