tests for advanced async component features

yyx990803 · yyx990803 · commit d2b7142cf557 · 2017-04-05T17:04:58.000+08:00
diff --git a/src/core/vdom/helpers/resolve-async-component.js b/src/core/vdom/helpers/resolve-async-component.js
@@ -75,30 +75,42 @@ export function resolveAsyncComponent (
           res.then(resolve, reject)
         }
       } else if (isDef(res.component) && typeof res.component.then === 'function') {
+        res.component.then(resolve, reject)
+
         if (isDef(res.error)) {
           factory.errorComp = ensureCtor(res.error, baseCtor)
         }
 
         if (isDef(res.loading)) {
           factory.loadingComp = ensureCtor(res.loading, baseCtor)
-          setTimeout(() => {
-            if (isUndef(factory.resolved) && isUndef(factory.error)) {
-              factory.loading = true
-              forceRender()
-            }
-          }, res.delay || 200)
+          if (res.delay === 0) {
+            factory.loading = true
+          } else {
+            setTimeout(() => {
+              if (isUndef(factory.resolved) && isUndef(factory.error)) {
+                factory.loading = true
+                forceRender()
+              }
+            }, res.delay || 200)
+          }
         }
 
         if (isDef(res.timeout)) {
-          setTimeout(reject, res.timeout)
+          setTimeout(() => {
+            reject(
+              process.env.NODE_ENV !== 'production'
+                ? `timeout (${res.timeout}ms)`
+                : null
+            )
+          }, res.timeout)
         }
-
-        res.component.then(resolve, reject)
       }
     }
 
     sync = false
     // return in case resolved synchronously
-    return factory.resolved
+    return factory.loading
+      ? factory.loadingComp
+      : factory.resolved
   }
 }
diff --git a/test/unit/features/component/component-async.spec.js b/test/unit/features/component/component-async.spec.js
@@ -159,4 +159,138 @@ describe('Component async', () => {
       done()
     }
   })
+
+  describe('loading/error/timeout', () => {
+    it('with loading component', done => {
+      const vm = new Vue({
+        template: `<div><test/></div>`,
+        components: {
+          test: () => ({
+            component: new Promise(resolve => {
+              setTimeout(() => {
+                resolve({ template: '<div>hi</div>' })
+                // wait for promise resolve and then parent update
+                Promise.resolve().then(() => {
+                  Vue.nextTick(next)
+                })
+              }, 5)
+            }),
+            loading: { template: `<div>loading</div>` },
+            delay: 1
+          })
+        }
+      }).$mount()
+
+      expect(vm.$el.innerHTML).toBe('<!---->')
+
+      let loadingAsserted = false
+      setTimeout(() => {
+        Vue.nextTick(() => {
+          loadingAsserted = true
+          expect(vm.$el.textContent).toBe('loading')
+        })
+      }, 1)
+
+      function next () {
+        expect(loadingAsserted).toBe(true)
+        expect(vm.$el.textContent).toBe('hi')
+        done()
+      }
+    })
+
+    it('with loading component (0 delay)', done => {
+      const vm = new Vue({
+        template: `<div><test/></div>`,
+        components: {
+          test: () => ({
+            component: new Promise(resolve => {
+              setTimeout(() => {
+                resolve({ template: '<div>hi</div>' })
+                // wait for promise resolve and then parent update
+                Promise.resolve().then(() => {
+                  Vue.nextTick(next)
+                })
+              }, 0)
+            }),
+            loading: { template: `<div>loading</div>` },
+            delay: 0
+          })
+        }
+      }).$mount()
+
+      expect(vm.$el.textContent).toBe('loading')
+
+      function next () {
+        expect(vm.$el.textContent).toBe('hi')
+        done()
+      }
+    })
+
+    it('with error component', done => {
+      const vm = new Vue({
+        template: `<div><test/></div>`,
+        components: {
+          test: () => ({
+            component: new Promise((resolve, reject) => {
+              setTimeout(() => {
+                reject()
+                // wait for promise resolve and then parent update
+                Promise.resolve().then(() => {
+                  Vue.nextTick(next)
+                })
+              }, 0)
+            }),
+            loading: { template: `<div>loading</div>` },
+            error: { template: `<div>error</div>` },
+            delay: 0
+          })
+        }
+      }).$mount()
+
+      expect(vm.$el.textContent).toBe('loading')
+
+      function next () {
+        expect(`Failed to resolve async component`).toHaveBeenWarned()
+        expect(vm.$el.textContent).toBe('error')
+        done()
+      }
+    })
+
+    it('with error component + timeout', done => {
+      const vm = new Vue({
+        template: `<div><test/></div>`,
+        components: {
+          test: () => ({
+            component: new Promise((resolve, reject) => {
+              setTimeout(() => {
+                resolve({ template: '<div>hi</div>' })
+                // wait for promise resolve and then parent update
+                Promise.resolve().then(() => {
+                  Vue.nextTick(next)
+                })
+              }, 5)
+            }),
+            loading: { template: `<div>loading</div>` },
+            error: { template: `<div>error</div>` },
+            delay: 0,
+            timeout: 1
+          })
+        }
+      }).$mount()
+
+      expect(vm.$el.textContent).toBe('loading')
+
+      setTimeout(() => {
+        Vue.nextTick(() => {
+          expect(`Failed to resolve async component`).toHaveBeenWarned()
+          expect(vm.$el.textContent).toBe('error')
+        })
+      }, 1)
+
+      function next () {
+        expect(vm.$el.textContent).toBe('error') // late resolve ignored
+        done()
+      }
+    })
+  })
 })
