fix(render-core): fix when designMode is false & loop is null, isUseLoop should return true

This commit is contained in:
liujuping 2023-06-12 14:45:10 +08:00 committed by 林熠
parent c838dc70eb
commit 11b929b42f
3 changed files with 11 additions and 4 deletions

View File

@ -13,6 +13,7 @@ const jestConfig = {
// testMatch: ['**/*/base.test.tsx'], // testMatch: ['**/*/base.test.tsx'],
// testMatch: ['**/utils/common.test.ts'], // testMatch: ['**/utils/common.test.ts'],
// testMatch: ['**/*/leaf.test.tsx'], // testMatch: ['**/*/leaf.test.tsx'],
// testMatch: ['**/*/is-use-loop.test.ts'],
transformIgnorePatterns: [ transformIgnorePatterns: [
`/node_modules/(?!${esModules})/`, `/node_modules/(?!${esModules})/`,
], ],

View File

@ -8,13 +8,13 @@ export default function isUseLoop(loop: null | any[] | IPublicTypeJSExpression,
return true; return true;
} }
if (!Array.isArray(loop)) {
return false;
}
if (!isDesignMode) { if (!isDesignMode) {
return true; return true;
} }
if (!Array.isArray(loop)) {
return false;
}
return loop.length > 0; return loop.length > 0;
} }

View File

@ -5,6 +5,9 @@ describe('base test', () => {
it('designMode is true', () => { it('designMode is true', () => {
expect(isUseLoop([], true)).toBeFalsy(); expect(isUseLoop([], true)).toBeFalsy();
expect(isUseLoop([{}], true)).toBeTruthy(); expect(isUseLoop([{}], true)).toBeTruthy();
expect(isUseLoop(null, true)).toBeFalsy();
expect(isUseLoop(undefined, true)).toBeFalsy();
expect(isUseLoop(0, true)).toBeFalsy();
}); });
it('loop is expression', () => { it('loop is expression', () => {
@ -21,5 +24,8 @@ describe('base test', () => {
it('designMode is false', () => { it('designMode is false', () => {
expect(isUseLoop([], false)).toBeTruthy(); expect(isUseLoop([], false)).toBeTruthy();
expect(isUseLoop([{}], false)).toBeTruthy(); expect(isUseLoop([{}], false)).toBeTruthy();
expect(isUseLoop(null, false)).toBeTruthy();
expect(isUseLoop(undefined, false)).toBeTruthy();
expect(isUseLoop(0, false)).toBeTruthy();
}); });
}); });