fix: make failing e2e tests to create screenshot

Signed-off-by: mdolhalo <[email protected]>

tests/e2e/index.ts

@@ -47,6 +47,7 @@ export * from './constants/API_TEST_CONSTANTS';
 export * from './constants/BASE_TEST_CONSTANTS';
 export * from './constants/CHROME_DRIVER_CONSTANTS';
 export * from './constants/FACTORY_TEST_CONSTANTS';
+export * from './constants/MOCHA_CONSTANTS';
 export * from './constants/MONACO_CONSTANTS';
 export * from './constants/OAUTH_CONSTANTS';
 export * from './constants/PLUGIN_TEST_CONSTANTS';

tests/e2e/package.json

@@ -38,6 +38,7 @@
 		"eslint-plugin-header": "^3.1.1",
 		"eslint-plugin-jsdoc": "^46.5.0",
 		"eslint-plugin-prettier": "^5.0.0",
+		"inversify-inject-decorators": "^3.1.0",
 		"husky": "^8.0.3",
 		"mocha": "^9.1.3",
 		"monaco-page-objects": "3.9.1",

tests/e2e/specs/MochaHooks.ts

@@ -76,6 +76,8 @@ exports.mochaHooks = {
 	],
 	afterAll: [
 		async function stopTheDriver(): Promise<void> {
+			// ensure that fired events done
+			await driverHelper.wait(5000);
 			if (!BASE_TEST_CONSTANTS.TS_DEBUG_MODE && CHROME_DRIVER_CONSTANTS.TS_USE_WEB_DRIVER_FOR_TEST) {
 				await driverHelper.getDriver().quit();
 				Logger.info('Chrome driver session stopped.');

tests/e2e/utils/CheReporter.ts

@@ -23,21 +23,22 @@ import { CHROME_DRIVER_CONSTANTS } from '../constants/CHROME_DRIVER_CONSTANTS';
 import { OAUTH_CONSTANTS } from '../constants/OAUTH_CONSTANTS';
 import { REPORTER_CONSTANTS } from '../constants/REPORTER_CONSTANTS';
 import { PLUGIN_TEST_CONSTANTS } from '../constants/PLUGIN_TEST_CONSTANTS';
-import { inject, injectable } from 'inversify';
+import { injectable } from 'inversify';
+import getDecorators from 'inversify-inject-decorators';
+import { e2eContainer } from '../configs/inversify.config';
+
+const { lazyInject } = getDecorators(e2eContainer);
 
 @injectable()
 class CheReporter extends mocha.reporters.Spec {
 	private static methodIndex: number = 0;
 	private static deleteScreencast: boolean = true;
+	@lazyInject(CLASSES.DriverHelper)
+	private readonly driverHelper!: DriverHelper;
+	@lazyInject(CLASSES.ScreenCatcher)
+	private readonly screenCatcher!: ScreenCatcher;
 
-	constructor(
-		runner: mocha.Runner,
-		options: mocha.MochaOptions,
-		@inject(CLASSES.DriverHelper)
-		private readonly driverHelper: DriverHelper,
-		@inject(CLASSES.ScreenCatcher)
-		private readonly screenCatcher: ScreenCatcher
-	) {
+	constructor(runner: mocha.Runner, options: mocha.MochaOptions) {
 		super(runner, options);
 
 		runner.on('start', (): void => {
@@ -111,13 +112,7 @@ class CheReporter extends mocha.reporters.Spec {
 			}
 		});
 
-		runner.on('end', async (): Promise<void> => {
-			// ensure that fired events done
-			await this.driverHelper.wait(5000);
-
-			// close driver
-			await this.driverHelper.getDriver().quit();
-
+		runner.on('end', (): void => {
 			// delete screencast folder if conditions matched
 			if (CheReporter.deleteScreencast && REPORTER_CONSTANTS.DELETE_SCREENCAST_IF_TEST_PASS) {
 				rm.sync(REPORTER_CONSTANTS.TS_SELENIUM_REPORT_FOLDER);
@@ -158,14 +153,12 @@ class CheReporter extends mocha.reporters.Spec {
 			// take screenshot and write to file
 			const screenshot: string = await this.driverHelper.getDriver().takeScreenshot();
 			const screenshotStream: WriteStream = fs.createWriteStream(screenshotFileName);
-			screenshotStream.write(Buffer.from(screenshot, 'base64'));
-			screenshotStream.end();
+			screenshotStream.write(Buffer.from(screenshot, 'base64'), (): void => screenshotStream.end());
 
 			// take page source and write to file
 			const pageSource: string = await this.driverHelper.getDriver().getPageSource();
 			const pageSourceStream: WriteStream = fs.createWriteStream(pageSourceFileName);
-			pageSourceStream.write(Buffer.from(pageSource));
-			pageSourceStream.end();
+			pageSourceStream.write(Buffer.from(pageSource), (): void => pageSourceStream.end());
 
 			// take browser console logs and write to file
 			const browserLogsEntries: logging.Entry[] = await this.driverHelper.getDriver().manage().logs().get('browser');
@@ -176,8 +169,9 @@ class CheReporter extends mocha.reporters.Spec {
 			});
 
 			const browserLogsStream: WriteStream = fs.createWriteStream(browserLogsFileName);
-			browserLogsStream.write(Buffer.from(browserLogs));
-			browserLogsStream.end();
+			browserLogsStream.write(Buffer.from(browserLogs), (): void => {
+				browserLogsStream.end();
+			});
 		});
 	}
 }