refactor: ensure logPrefix helper is always used
authorJérôme Benoit <jerome.benoit@sap.com>
Fri, 8 Sep 2023 20:56:44 +0000 (22:56 +0200)
committerJérôme Benoit <jerome.benoit@sap.com>
Fri, 8 Sep 2023 20:56:44 +0000 (22:56 +0200)
Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>
src/charging-station/Helpers.ts
src/utils/Configuration.ts
src/worker/WorkerUtils.ts

index 068bc6c970afeaa9e402cc4f69086ccafae59ded..90ecd0e8154814c6a2be8f70c6b508690f59c096 100644 (file)
@@ -714,7 +714,7 @@ const warnDeprecatedTemplateKey = (
       isNotEmptyString(logMsgToAppend) ? `. ${logMsgToAppend}` : ''
     }`;
     logger.warn(`${logPrefix} ${logMsg}`);
-    console.warn(chalk.yellow(`${logMsg}`));
+    console.warn(`${chalk.green(logPrefix)} ${chalk.yellow(logMsg)}`);
   }
 };
 
index 190a31a350f1a8c405bb47a96165c4643c09f9d8..bf691fa8c1d96ffcb2da9ed44fc6f62ca977e098 100644 (file)
@@ -6,7 +6,14 @@ import chalk from 'chalk';
 import merge from 'just-merge';
 
 import { Constants } from './Constants';
-import { hasOwnProp, isCFEnvironment, isNotEmptyString, isUndefined, once } from './Utils';
+import {
+  hasOwnProp,
+  isCFEnvironment,
+  isNotEmptyString,
+  isUndefined,
+  logPrefix,
+  once,
+} from './Utils';
 import {
   ApplicationProtocol,
   type ConfigurationData,
@@ -114,6 +121,10 @@ export class Configuration {
     );
   }
 
+  private static logPrefix = (): string => {
+    return logPrefix(' Simulator configuration |');
+  };
+
   private static isConfigurationSectionCached(sectionName: ConfigurationSection): boolean {
     return Configuration.configurationSectionCache.has(sectionName);
   }
@@ -287,10 +298,6 @@ export class Configuration {
     return workerConfiguration;
   }
 
-  private static logPrefix = (): string => {
-    return `${new Date().toLocaleString()} Simulator configuration |`;
-  };
-
   private static checkDeprecatedConfigurationKeys() {
     // connection timeout
     Configuration.warnDeprecatedConfigurationKey(
@@ -555,9 +562,9 @@ export class Configuration {
     file: string,
     fileType: FileType,
     error: NodeJS.ErrnoException,
-    logPrefix: string,
+    logPfx: string,
   ): void {
-    const prefix = isNotEmptyString(logPrefix) ? `${logPrefix} ` : '';
+    const prefix = isNotEmptyString(logPfx) ? `${logPfx} ` : '';
     let logMsg: string;
     switch (error.code) {
       case 'ENOENT':
index 452f8cfe8e4d549cb0563e20c78d8e686f4de018..e930a47f826a33674c08cd7f1f99ff31f15facbc 100644 (file)
@@ -15,5 +15,5 @@ export const defaultExitHandler = (code: number): void => {
 };
 
 export const defaultErrorHandler = (error: Error): void => {
-  console.error(chalk.red('Worker errored: ', error));
+  console.error(chalk.red('Worker errored: '), error);
 };