Add new tests

jupyterlab · May 19, 2024 · a227e96 · a227e96
1 parent a10d181
commit a227e96
Show file tree

Hide file tree

Showing 2 changed files with 91 additions and 1 deletion.
diff --git a/packages/application/src/plugins.ts b/packages/application/src/plugins.ts
@@ -117,7 +117,7 @@ export interface IPlugin<T, U> {
 }
 
 /**
- * Abstract plugin registry.
+ * Plugin registry.
  */
 export class PluginRegistry<T = any> {
   constructor(options: PluginRegistry.IOptions = {}) {

diff --git a/packages/application/tests/src/index.spec.ts b/packages/application/tests/src/index.spec.ts
@@ -28,6 +28,35 @@ describe('@lumino/application', () => {
         expect(app.contextMenu).to.be.instanceOf(ContextMenu);
         expect(app.shell).to.equal(shell);
       });
+
+      it('should accept an external plugin registry', async () => {
+        const shell = new Widget();
+        const pluginRegistry = new PluginRegistry();
+        const id1 = 'plugin1';
+        pluginRegistry.registerPlugin({
+          id: id1,
+          activate: () => {
+            // no-op
+          }
+        });
+        const id2 = 'plugin2';
+        pluginRegistry.registerPlugin({
+          id: id2,
+          activate: () => {
+            // no-op
+          }
+        });
+
+        const app = new Application({
+          shell,
+          pluginRegistry
+        });
+
+        await pluginRegistry.activatePlugin(id2);
+
+        expect(app.hasPlugin(id1)).to.be.true;
+        expect(app.isPluginActivated(id2)).to.be.true;
+      });
     });
 
     describe('#getPluginDescription', () => {
@@ -869,6 +898,67 @@ describe('@lumino/application', () => {
 
         expect(plugins.hasPlugin(id)).to.be.true;
       });
+
+      it('should refuse to register not allowed plugins', async () => {
+        const plugins = new PluginRegistry({
+          allowedPlugins: new Set(['id1'])
+        });
+        expect(function () {
+          plugins.registerPlugin({
+            id: 'id',
+            activate: () => {
+              /* no-op */
+            }
+          });
+        }).to.throw();
+        plugins.registerPlugin({
+          id: 'id1',
+          activate: () => {
+            /* no-op */
+          }
+        });
+      });
+
+      it('should refuse to register blocked plugins', async () => {
+        const plugins = new PluginRegistry({
+          blockedPlugins: new Set(['id1'])
+        });
+        expect(function () {
+          plugins.registerPlugin({
+            id: 'id1',
+            activate: () => {
+              /* no-op */
+            }
+          });
+        }).to.throw();
+        plugins.registerPlugin({
+          id: 'id2',
+          activate: () => {
+            /* no-op */
+          }
+        });
+      });
+
+      it('should use allowed list over blocked list of plugins', async () => {
+        const plugins = new PluginRegistry({
+          allowedPlugins: new Set(['id1', 'id2']),
+          blockedPlugins: new Set(['id2'])
+        });
+        expect(function () {
+          plugins.registerPlugin({
+            id: 'id',
+            activate: () => {
+              /* no-op */
+            }
+          });
+        }).to.throw();
+        plugins.registerPlugin({
+          id: 'id2',
+          activate: () => {
+            /* no-op */
+          }
+        });
+      });
     });
 
     describe('#deregisterPlugin', () => {