fix: deal with a corner case

Author: Haixing-Hu

src/utils.js

@@ -61,7 +61,9 @@ function collectMethod(obj, key, options) {
     return;
   }
   if (VUE_LIFECYCLE_HOOKS.includes(key) || VUE_SPECIAL_FUNCTIONS.includes(key)) {
-    // obj[key] must be a function
+    if (typeof obj[key] !== 'function') {
+      throw new Error(`The property "${key}" of the class must be a Vue component function.`);
+    }
     options[key] = obj[key];
   } else {
     const descriptor = Object.getOwnPropertyDescriptor(obj, key);

test/bug-5.test.js

@@ -10,7 +10,7 @@ import { mount } from '@vue/test-utils';
 import { nextTick } from 'vue';
 import { Component, toVue } from '../src';
 
-function AddMessage(target, context) {
+function AddMessage(target) {
   // add a new field to the target class
   target.prototype.message = 'Hello';
 }
@@ -24,7 +24,6 @@ class Test {
 }
 const TestComponent = toVue(Test);
 
-
 describe('bug #5', () => {
   test('Should handle the class field added to the prototype of the class', async () => {
     const w1 = mount(TestComponent);

test/component.test.js

@@ -214,4 +214,18 @@ describe('@Component decorator', () => {
       + 'as the class method and decorated by @Watch.',
     );
   });
+  test('@Component on class with invalid Vue method `created`', () => {
+    expect(() => {
+      function AddCreated(target) {
+        target.prototype.created = 'Hello';
+      }
+      @Component
+      @AddCreated
+      class F1 {}
+      new F1();
+    }).toThrowWithMessage(
+      Error,
+      'The property "created" of the class must be a Vue component function.',
+    );
+  });
 });

test/create-decorator.test.js

@@ -25,7 +25,7 @@ describe('createDecorator() function', () => {
       const methodName = context.name;
       const originalMethod = options.methods[methodName];
       // note that the following function can NOT be an arrow function
-      options.methods[methodName] = function (...args) {
+      options.methods[methodName] = function log(...args) {
         messages.push(`Calling ${Class.name}.${methodName}() with arguments: ${args.join(', ')}`);
         console.log(`${Class.name}.${methodName}: ${args.join(', ')}`);
         return originalMethod.apply(this, args);