add IIntegrationManager interface

Author: jankuca

src/notebooks/deepnote/deepnoteActivationService.ts

@@ -5,7 +5,7 @@ import { IExtensionContext } from '../../platform/common/types';
 import { IDeepnoteNotebookManager } from '../types';
 import { DeepnoteNotebookSerializer } from './deepnoteSerializer';
 import { DeepnoteExplorerView } from './deepnoteExplorerView';
-import { IntegrationManager } from './integrations/integrationManager';
+import { IIntegrationManager } from './integrations/types';
 
 /**
  * Service responsible for activating and configuring Deepnote notebook support in VS Code.
@@ -15,14 +15,14 @@ import { IntegrationManager } from './integrations/integrationManager';
 export class DeepnoteActivationService implements IExtensionSyncActivationService {
     private explorerView: DeepnoteExplorerView;
 
-    private integrationManager: IntegrationManager;
+    private integrationManager: IIntegrationManager;
 
     private serializer: DeepnoteNotebookSerializer;
 
     constructor(
         @inject(IExtensionContext) private extensionContext: IExtensionContext,
         @inject(IDeepnoteNotebookManager) private readonly notebookManager: IDeepnoteNotebookManager,
-        @inject(IntegrationManager) integrationManager: IntegrationManager
+        @inject(IIntegrationManager) integrationManager: IIntegrationManager
     ) {
         this.integrationManager = integrationManager;
     }

src/notebooks/deepnote/deepnoteActivationService.unit.test.ts

@@ -3,13 +3,13 @@ import { assert } from 'chai';
 import { DeepnoteActivationService } from './deepnoteActivationService';
 import { DeepnoteNotebookManager } from './deepnoteNotebookManager';
 import { IExtensionContext } from '../../platform/common/types';
-import { IntegrationManager } from './integrations/integrationManager';
+import { IIntegrationManager } from './integrations/types';
 
 suite('DeepnoteActivationService', () => {
     let activationService: DeepnoteActivationService;
     let mockExtensionContext: IExtensionContext;
     let manager: DeepnoteNotebookManager;
-    let mockIntegrationManager: IntegrationManager;
+    let mockIntegrationManager: IIntegrationManager;
 
     setup(() => {
         mockExtensionContext = {

src/notebooks/deepnote/integrations/integrationManager.ts

@@ -4,15 +4,15 @@ import { commands, NotebookDocument, window, workspace } from 'vscode';
 import { IExtensionContext } from '../../../platform/common/types';
 import { Commands } from '../../../platform/common/constants';
 import { logger } from '../../../platform/logging';
-import { IIntegrationDetector, IIntegrationStorage, IIntegrationWebviewProvider } from './types';
+import { IIntegrationDetector, IIntegrationManager, IIntegrationStorage, IIntegrationWebviewProvider } from './types';
 import { IntegrationStatus, IntegrationWithStatus } from './integrationTypes';
 import { BlockWithIntegration, scanBlocksForIntegrations } from './integrationUtils';
 
 /**
  * Manages integration UI and commands for Deepnote notebooks
  */
 @injectable()
-export class IntegrationManager {
+export class IntegrationManager implements IIntegrationManager {
     private hasIntegrationsContext = 'deepnote.hasIntegrations';
 
     private hasUnconfiguredIntegrationsContext = 'deepnote.hasUnconfiguredIntegrations';

src/notebooks/deepnote/integrations/types.ts

@@ -30,3 +30,11 @@ export interface IIntegrationWebviewProvider {
      */
     show(integrations: Map<string, IntegrationWithStatus>): Promise<void>;
 }
+
+export const IIntegrationManager = Symbol('IIntegrationManager');
+export interface IIntegrationManager {
+    /**
+     * Activate the integration manager by registering commands and event listeners
+     */
+    activate(): void;
+}

src/notebooks/serviceRegistry.node.ts

@@ -47,7 +47,12 @@ import { IntegrationStorage } from './deepnote/integrations/integrationStorage';
 import { IntegrationDetector } from './deepnote/integrations/integrationDetector';
 import { IntegrationManager } from './deepnote/integrations/integrationManager';
 import { IntegrationWebviewProvider } from './deepnote/integrations/integrationWebview';
-import { IIntegrationDetector, IIntegrationStorage, IIntegrationWebviewProvider } from './deepnote/integrations/types';
+import {
+    IIntegrationDetector,
+    IIntegrationManager,
+    IIntegrationStorage,
+    IIntegrationWebviewProvider
+} from './deepnote/integrations/types';
 import {
     IDeepnoteToolkitInstaller,
     IDeepnoteServerStarter,
@@ -136,7 +141,7 @@ export function registerTypes(serviceManager: IServiceManager, isDevMode: boolea
     serviceManager.addSingleton<IIntegrationStorage>(IIntegrationStorage, IntegrationStorage);
     serviceManager.addSingleton<IIntegrationDetector>(IIntegrationDetector, IntegrationDetector);
     serviceManager.addSingleton<IIntegrationWebviewProvider>(IIntegrationWebviewProvider, IntegrationWebviewProvider);
-    serviceManager.addSingleton<IntegrationManager>(IntegrationManager, IntegrationManager);
+    serviceManager.addSingleton<IIntegrationManager>(IIntegrationManager, IntegrationManager);
 
     // Deepnote kernel services
     serviceManager.addSingleton<IDeepnoteToolkitInstaller>(IDeepnoteToolkitInstaller, DeepnoteToolkitInstaller);

src/notebooks/serviceRegistry.web.ts

@@ -42,7 +42,12 @@ import { IntegrationStorage } from './deepnote/integrations/integrationStorage';
 import { IntegrationDetector } from './deepnote/integrations/integrationDetector';
 import { IntegrationManager } from './deepnote/integrations/integrationManager';
 import { IntegrationWebviewProvider } from './deepnote/integrations/integrationWebview';
-import { IIntegrationDetector, IIntegrationStorage, IIntegrationWebviewProvider } from './deepnote/integrations/types';
+import {
+    IIntegrationDetector,
+    IIntegrationManager,
+    IIntegrationStorage,
+    IIntegrationWebviewProvider
+} from './deepnote/integrations/types';
 
 export function registerTypes(serviceManager: IServiceManager, isDevMode: boolean) {
     registerControllerTypes(serviceManager, isDevMode);
@@ -100,7 +105,7 @@ export function registerTypes(serviceManager: IServiceManager, isDevMode: boolea
     serviceManager.addSingleton<IIntegrationStorage>(IIntegrationStorage, IntegrationStorage);
     serviceManager.addSingleton<IIntegrationDetector>(IIntegrationDetector, IntegrationDetector);
     serviceManager.addSingleton<IIntegrationWebviewProvider>(IIntegrationWebviewProvider, IntegrationWebviewProvider);
-    serviceManager.addSingleton<IntegrationManager>(IntegrationManager, IntegrationManager);
+    serviceManager.addSingleton<IIntegrationManager>(IIntegrationManager, IntegrationManager);
 
     serviceManager.addSingleton<IExportBase>(IExportBase, ExportBase);
     serviceManager.addSingleton<IFileConverter>(IFileConverter, FileConverter);