cleanup

posit-dev · Sep 30, 2024 · 7120a6c · 7120a6c
1 parent 92d4195
commit 7120a6c
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/test/smoke/src/areas/positron/example.test.ts b/test/smoke/src/areas/positron/example.test.ts
@@ -8,7 +8,8 @@ import { Application, PositronPythonFixtures } from '../../../../automation';
 import { setupEnvAndHooks } from '../../positronUtils';
 
 describe('Major Test Area', () => {
-	// Shared before/after look which returns logger (in case you need to log in test)
+	// Needed at parent `describe` block to setup shared before/after hooks
+	// It does return the logger (in case you want/need to log in test)
 	setupEnvAndHooks();
 
 	describe('Minor Test area', () => {

diff --git a/test/smoke/src/areas/positron/welcome/welcome.test.ts b/test/smoke/src/areas/positron/welcome/welcome.test.ts
@@ -22,7 +22,7 @@ if (!web) {
 			const app = this.app as Application;
 
 			await app.workbench.quickaccess.runCommand('Help: Welcome');
-			app.workbench.editors.waitForActiveEditor('Welcome');
+			await app.workbench.editors.waitForActiveEditor('Welcome');
 		});
 
 		afterEach(async function () {

diff --git a/test/smoke/src/positronUtils.ts b/test/smoke/src/positronUtils.ts
@@ -22,10 +22,9 @@ const EXTENSIONS_PATH = path.join(TEST_DATA_PATH, 'extensions-dir');
 const OPTS = parseOptions();
 
 /**
- * Setup the environment and hooks for the test
+ * Setup the environment and hooks for the test suite.
  *
- * @param suiteName name of the test
- * @returns
+ * @returns The logger instance for the test suite.
  */
 export function setupEnvAndHooks(): Logger {
 	// Dynamically determine the test file name