feat: make cli close revert config optional (#1)

Author: peterpeterparker

src/page-objects/_browser.page.ts

@@ -1,28 +1,22 @@
 import type {Browser, BrowserContext, Page} from '@playwright/test';
-import {TestPage} from './_page';
 
 export interface BrowserPageParams {
   page: Page;
   context: BrowserContext;
   browser: Browser;
 }
 
-export abstract class BrowserPage extends TestPage {
+export abstract class BrowserPage {
   protected readonly page: Page;
   protected readonly context: BrowserContext;
   protected readonly browser: Browser;
 
   protected constructor({page, context, browser}: BrowserPageParams) {
-    super();
-
     this.page = page;
     this.context = context;
     this.browser = browser;
   }
 
-  /**
-   * @override
-   */
   async close(): Promise<void> {
     await this.page.close();
   }

src/page-objects/_page.ts

@@ -4,7 +4,6 @@ import {execute, spawn} from '@junobuild/cli-tools';
 import {statSync} from 'node:fs';
 import {readdir, readFile, writeFile} from 'node:fs/promises';
 import {join} from 'node:path';
-import {TestPage} from './_page';
 
 const DEV = (process.env.NODE_ENV ?? 'production') === 'development';
 
@@ -22,12 +21,10 @@ export interface CliPageParams {
   satelliteId: PrincipalText;
 }
 
-export class CliPage extends TestPage {
+export class CliPage {
   #satelliteId: PrincipalText;
 
   private constructor({satelliteId}: CliPageParams) {
-    super();
-
     this.#satelliteId = satelliteId;
   }
 
@@ -214,11 +211,11 @@ export class CliPage extends TestPage {
     return {accessKey: accessKey.trim()};
   }
 
-  /**
-   * @override
-   */
-  async close(): Promise<void> {
-    await this.revertConfig();
+  async close({revertConfig}: {revertConfig: boolean} = {revertConfig: true}): Promise<void> {
+    if (revertConfig) {
+      await this.revertConfig();
+    }
+
     await this.logout();
   }
 }