FI-765 fix: writes HTML report after exit signals to `dockerEntrypoin…

…t.sh`

autotests/bin/runDocker.sh

@@ -22,6 +22,8 @@ fi
 onExit() {
     CONTAINER_ID=$(docker ps --filter "label=$CONTAINER_LABEL" --format "{{.ID}}")
 
+    sleep 4
+
     if [[ -z $CONTAINER_ID ]]
     then
         echo "Docker container from image $E2ED_DOCKER_IMAGE:$VERSION already stopped"

bin/dockerEntrypoint.sh

@@ -14,7 +14,8 @@ onExit() {
     if [[ $PID ]] && ps -p $PID > /dev/null
     then
         echo "$PID is running"
-        kill $PID
+        kill -USR1 $PID
+        sleep 4
     fi
 
     restoreE2edPackage;

src/types/global.d.ts

@@ -21,7 +21,7 @@ declare module 'bin-v8-flags-filter' {
   }>;
 
   /**
-   * Filters out nodejs cli options and runs node module on cliPath.
+   * Filters out `nodejs` cli options and runs node module on `cliPath`.
    */
   const v8FlagsFilter: (cliPath: string, options: Options) => void;
 

src/utils/end/setProcessEndHandlers.ts

@@ -5,7 +5,7 @@ import {generalLog} from '../generalLog';
 import {endE2ed} from './endE2ed';
 
 /**
- * nodejs e2ed process end hanlder.
+ * `nodejs` e2ed process end hanlder.
  * @internal
  */
 const endHandler = (signal: NodeJS.Signals): void => {
@@ -15,10 +15,12 @@ const endHandler = (signal: NodeJS.Signals): void => {
 };
 
 /**
- * Set nodejs e2ed process end hanlders (SIGINT, SIGTERM).
+ * Set `nodejs` e2ed process end hanlders (`SIGHUP`, `SIGINT`, `SIGTERM`).
  * @internal
  */
 export const setProcessEndHandlers = (): void => {
+  process.on('SIGHUP', endHandler);
   process.on('SIGINT', endHandler);
   process.on('SIGTERM', endHandler);
+  process.on('SIGUSR1', endHandler);
 };

src/utils/error/E2edError.ts

@@ -85,7 +85,7 @@ export class E2edError extends Error {
   }
 
   /**
-   * Custom presentation of error for nodejs `inspect`.
+   * Custom presentation of error for `nodejs` `inspect`.
    */
   [inspect.custom](): string {
     return this.toString();

src/utils/events/registerEndE2edRunEvent.ts

@@ -10,11 +10,19 @@ import {runAfterPackFunctions} from './runAfterPackFunctions';
 
 import type {ReportData} from '../../types/internal';
 
+let isEndAlreadyCalled = false;
+
 /**
  * Registers end e2ed run event (for report) after closing of all tests.
  * @internal
  */
 export const registerEndE2edRunEvent = async (): Promise<void> => {
+  if (isEndAlreadyCalled) {
+    return;
+  }
+
+  isEndAlreadyCalled = true;
+
   generalLog('Starting to close e2ed...');
 
   let reportData: ReportData | undefined;

src/utils/report/writeHtmlReport.ts

@@ -12,7 +12,7 @@ import {renderReportToHtml} from './render';
 import type {FilePathFromRoot, ReportData, UtcTimeInMs} from '../../types/internal';
 
 /**
- * Writes HTML report (report.html file) with test runs results.
+ * Writes HTML report (`report.html` file) with test runs results.
  * @internal
  */
 export const writeHtmlReport = async (reportData: ReportData): Promise<void> => {

src/utils/retry/runRetry.ts

@@ -40,7 +40,8 @@ export const runRetry = (runRetryOptions: RunRetryOptions): Promise<void> =>
     setTestsSubprocess(newTestsSubprocess);
 
     newTestsSubprocess.on('error', reject);
-    newTestsSubprocess.on('exit', (exitCode) => {
+
+    newTestsSubprocess.on('exit', (exitCode): void => {
       const error = new E2edError(
         `Tests subprocess with label "${runLabel}" exit with non-zero exit code ${String(
           exitCode,

src/utils/valueToString/wrapStringForLogs.ts

@@ -17,7 +17,7 @@ function toMultipleString(this: StringForLogs): string {
 
 /**
  * If the text consists of several lines, replaces the text with an object
- * with a more beautiful presentation through nodejs `inspect`.
+ * with a more beautiful presentation through `nodejs` `inspect`.
  * @internal
  */
 export const wrapStringForLogs = (text: string, options?: Options): StringForLogs | string => {