Merge pull request #834 from gemini-testing/fix/load.plugins

fix: load plugins only once
gemini-testing · Nov 2, 2017 · 0224b70 · 0224b70
2 parents 05d10da + a56e39b
commit 0224b70
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/lib/gemini.js b/lib/gemini.js
@@ -103,8 +103,12 @@ module.exports = class Gemini extends PassthroughEmitter {
     }
 
     _exec(fn) {
-        const plugins = pluginsLoader.load(this, this.config.system.plugins, PREFIX);
-        return bluebirdQ(Promise.all(plugins).then(() => fn()));
+        return bluebirdQ(this._loadPlugins().then(() => fn()));
+    }
+
+    _loadPlugins() {
+        this._loadPlugins = () => Promise.resolve(); // load plugins only once
+        return Promise.all(pluginsLoader.load(this, this.config.system.plugins, PREFIX));
     }
 
     _readTests(paths, options) {

diff --git a/test/unit/gemini.js b/test/unit/gemini.js
@@ -183,6 +183,14 @@ describe('gemini', () => {
             return assert.isRejected(result, err)
                 .then(() => assert.notCalled(Runner.prototype.run));
         });
+
+        it('should load plugins only once', () => {
+            const gemini = initGemini();
+
+            return gemini.readTests()
+                .then((collection) => gemini.test(collection))
+                .then(() => assert.calledOnce(pluginsLoader.load));
+        });
     });
 
     describe('readTests', () => {