[update] throw plugin name if you install ilegal plugin

What: Why: How:
Chimeejs · Aug 2, 2017 · a9d833d · a9d833d
1 parent 07ced66
commit a9d833d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/__tests__/dispatcher/index.js b/__tests__/dispatcher/index.js
@@ -35,7 +35,7 @@ describe('dispatcher', () => {
     expect(() => Dispatcher.install({})).toThrow("plugin's config must be an Object");
     expect(() => Dispatcher.install({name: ''})).toThrow('plugin must have a legal name');
     expect(() => Dispatcher.install({name: 1})).toThrow('plugin must have a legal name');
-    expect(() => Dispatcher.install(class Illegal {})).toThrow('If you pass a function as plugin config, this class must extends from Chimee.plugin');
+    expect(() => Dispatcher.install(class Illegal {})).toThrow("Your are trying to install plugin Illegal, but it's not extends from Chimee.plugin.");
     expect(Dispatcher.install(normalInstall)).toBe('normalInstall');
     expect(Dispatcher.install(NormalFunctionInstall)).toBe('normalFunctionInstall');
     expect(Dispatcher.install(GrandSon)).toBe('grandSon');

diff --git a/src/dispatcher/index.js b/src/dispatcher/index.js
@@ -14,7 +14,7 @@ function convertNameIntoId (name: string): string {
 function checkPluginConfig (config: any) {
   if(isFunction(config)) {
     if(!(config.prototype instanceof Plugin)) {
-      throw new TypeError('If you pass a function as plugin config, this class must extends from Chimee.plugin');
+      throw new TypeError(`Your are trying to install plugin ${config.name}, but it's not extends from Chimee.plugin.`);
     }
     return;
   }