Skip to content

Commit b904cb1

Browse files
JoshwinThomasIBMJoshwinThomasIBM
authored andcommitted
Added comments
1 parent d114608 commit b904cb1

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

src/liberty/devCommands.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -561,7 +561,8 @@ function checkReportAndDisplay(report: any, reportType: string, reportTypeLabel:
561561
{}, // Webview options
562562
);
563563
panel.webview.html = getReport(report); // display HTML content
564-
outputChannel.appendLine(reportType+" path is "+report);
564+
outputChannel.clear();// clear the content from Liberty tools Extension Output channel
565+
outputChannel.appendLine(reportType+" path is "+report);// writing the path to Liberty tools Extension Output channel
565566
/*
566567
For Maven projects we need to check for the test report in the 'reports' and 'site' dirs.
567568
We only need to show the message if it is not available in both locations.

src/test/MavenTestDevModeActions.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -289,8 +289,8 @@ it('View latest Unit test report when all the reports exists', async () => {
289289
outputView.clearText();
290290
await utils.launchDashboardAction(item, constants.UTR_DASHABOARD_ACTION, constants.UTR_DASHABOARD_MAC_ACTION);
291291
await outputView.selectChannel("Liberty tools Extension Output");
292-
const text = await outputView.getText();
293-
assert(text.includes('/target/reports/surefire.html'));
292+
const text = await outputView.getText();// Read the text from Liberty tools Extension Output channel
293+
assert(text.includes('/target/reports/surefire.html'));//check the path fetched from channel belong to the latest test report
294294
await utils.closeEditor();
295295
}).timeout(10000);
296296

0 commit comments

Comments
 (0)