Skip to content

Commit 99584b4

Browse files
committed
Update Debug bar timeout to use variable, remove some comments
1 parent 2243b9e commit 99584b4

1 file changed

Lines changed: 2 additions & 4 deletions

File tree

src/test/utils/testUtils.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -447,7 +447,6 @@ export async function clearCommandPalette() {
447447

448448
/**
449449
* Wait for the Liberty Dashboard to load and become ready.
450-
* This replaces arbitrary delays with condition-based waiting.
451450
*/
452451
export async function waitForDashboardToLoad(section: any): Promise<void> {
453452
const wait = getWaitHelper();
@@ -487,7 +486,6 @@ export async function waitForDashboardToLoad(section: any): Promise<void> {
487486

488487
/**
489488
* Wait for server to start by checking terminal output.
490-
* Replaces fixed 30-second delays with condition-based waiting.
491489
*/
492490
export async function waitForServerStart(serverStartString: string): Promise<boolean> {
493491
logger.info('Waiting for server to start');
@@ -509,7 +507,6 @@ export async function waitForServerStop(serverStopString: string): Promise<boole
509507

510508
/**
511509
* Wait for test report file to exist on filesystem.
512-
* Replaces polling loop with condition-based waiting.
513510
* @param reportPath Primary report path to check
514511
* @param alternatePath Optional alternate report path to check simultaneously
515512
* @returns true if report found at either location, false otherwise
@@ -555,7 +552,8 @@ export async function waitForDebuggerAttach(): Promise<boolean> {
555552

556553
try {
557554
// Wait for the debug toolbar to appear, which indicates debugger is attached
558-
await DebugToolbar.create(30000);
555+
const findDebugBarTimeout = 30000;
556+
await DebugToolbar.create(findDebugBarTimeout);
559557
logger.info('DebugToolbar appeared - debugger attached successfully');
560558
return true;
561559
} catch (error) {

0 commit comments

Comments
 (0)