chore: cleanup bootc.ts

The original template had a bootc.ts that we didn't use. None of the other extensions have this pattern, so we've decided to just clean it up for now. Tests merged in and extra files deleted. Fixes #55. Signed-off-by: Tim deBoer <[email protected]>
podman-desktop · Jan 19, 2024 · d5aae24 · d5aae24
1 parent 40cd556
commit d5aae24
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 123 deletions.
diff --git a/src/bootc.spec.ts b/src/bootc.spec.ts
diff --git a/src/bootc.ts b/src/bootc.ts
diff --git a/src/extension.spec.ts b/src/extension.spec.ts
@@ -18,12 +18,13 @@
 
 /* eslint-disable @typescript-eslint/no-explicit-any */
 
-import { beforeEach, expect, test, vi } from 'vitest';
+import { afterEach, beforeEach, expect, test, vi } from 'vitest';
 import type * as podmanDesktopApi from '@podman-desktop/api';
 import { activate, deactivate } from './extension';
 
-const bootcActivateMock = vi.fn();
-const bootcDeactivateMock = vi.fn();
+/// mock console.log
+const originalConsoleLog = console.log;
+const consoleLogMock = vi.fn();
 
 vi.mock('@podman-desktop/api', async () => {
   return {
@@ -36,20 +37,16 @@ vi.mock('@podman-desktop/api', async () => {
   };
 });
 
-vi.mock('./bootc', async () => {
-  return {
-    BootC: class {
-      public activate = bootcActivateMock;
-      public deactivate = bootcDeactivateMock;
-    },
-  };
-});
-
 beforeEach(() => {
   vi.clearAllMocks();
+  console.log = consoleLogMock;
 });
 
-test('check we call activate method on bootc ', async () => {
+afterEach(() => {
+  console.log = originalConsoleLog;
+});
+
+test('check activate', async () => {
   const fakeContext = {
     subscriptions: {
       push: vi.fn(),
@@ -58,19 +55,11 @@ test('check we call activate method on bootc ', async () => {
 
   await activate(fakeContext);
 
-  // expect the activate method to be called on the bootc class
-  expect(bootcActivateMock).toBeCalledTimes(1);
-
-  // no call on deactivate
-  expect(bootcDeactivateMock).not.toBeCalled();
+  expect(consoleLogMock).toBeCalledWith('starting bootc extension');
 });
 
-test('check we call deactivate method on bootc ', async () => {
+test('check deactivate', async () => {
   await deactivate();
 
-  // expect the activate method to be called on the bootc class
-  expect(bootcDeactivateMock).toBeCalledTimes(1);
-
-  // no call on activate
-  expect(bootcActivateMock).not.toBeCalled();
+  expect(consoleLogMock).toBeCalledWith('stopping bootc extension');
 });
diff --git a/src/extension.ts b/src/extension.ts
@@ -18,15 +18,11 @@
 
 import type { ExtensionContext } from '@podman-desktop/api';
 import * as extensionApi from '@podman-desktop/api';
-import { BootC } from './bootc';
 import { launchVFKit } from './launch-vfkit';
 import { buildDiskImage } from './build-disk-image';
 
-let bootc: BootC | undefined;
-
 export async function activate(extensionContext: ExtensionContext): Promise<void> {
-  bootc = new BootC(extensionContext);
-  await bootc?.activate();
+  console.log('starting bootc extension');
 
   extensionContext.subscriptions.push(
     extensionApi.commands.registerCommand('bootc.vfkit', async container => {
@@ -40,5 +36,5 @@ export async function activate(extensionContext: ExtensionContext): Promise<void
 }
 
 export async function deactivate(): Promise<void> {
-  await bootc?.deactivate();
+  console.log('stopping bootc extension');
 }