fix(runtime-core): ensure consistent behavior between dev/prod on inv…

…alid v-for range close vuejs#5867
iwusong · May 13, 2022 · e436be8 · e436be8
1 parent 312bf8f
commit e436be8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/packages/runtime-core/__tests__/helpers/renderList.spec.ts b/packages/runtime-core/__tests__/helpers/renderList.spec.ts
@@ -22,7 +22,9 @@ describe('renderList', () => {
   })
 
   it('should warn when given a non-integer N', () => {
-    renderList(3.1, () => {})
+    try {
+      renderList(3.1, () => {})
+    } catch (e) {}
     expect(
       `The v-for range expect an integer value but got 3.1.`
     ).toHaveBeenWarned()

diff --git a/packages/runtime-core/src/helpers/renderList.ts b/packages/runtime-core/src/helpers/renderList.ts
@@ -67,7 +67,6 @@ export function renderList(
   } else if (typeof source === 'number') {
     if (__DEV__ && !Number.isInteger(source)) {
       warn(`The v-for range expect an integer value but got ${source}.`)
-      return []
     }
     ret = new Array(source)
     for (let i = 0; i < source; i++) {