Fix eslint and prettier configuration

Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>

.eslintrc

@@ -12,7 +12,7 @@
     "plugin:import/warnings",
     "plugin:import/typescript",
     "plugin:jsdoc/recommended",
-    "prettier"
+    "plugin:prettier/recommended"
   ],
   "plugins": ["import", "jsdoc", "@typescript-eslint"],
   "settings": {
@@ -21,25 +21,6 @@
     }
   },
   "rules": {
-    "indent": "off",
-    "@typescript-eslint/indent": [
-      "error",
-      2,
-      {
-        "SwitchCase": 1,
-        "CallExpression": {
-          "arguments": 1
-        },
-        "FunctionDeclaration": {
-          "parameters": 2,
-          "body": 1
-        },
-        "FunctionExpression": {
-          "parameters": 2,
-          "body": 1
-        }
-      }
-    ],
     "@typescript-eslint/array-type": "off",
     "semi": "off",
     "@typescript-eslint/semi": ["error", "always"],
@@ -55,7 +36,7 @@
     "@typescript-eslint/no-floating-promises": "error",
     "@typescript-eslint/promise-function-async": "error",
     "@typescript-eslint/no-misused-promises": "error",
-    "curly": "error",
+    "curly": ["error", "all"],
     "brace-style": "error",
     "eqeqeq": ["error", "always"],
     "no-else-return": "error",
@@ -108,13 +89,8 @@
     "max-len": [
       "warn",
       {
-        "code": 180,
-        "ignoreComments": true,
-        "ignoreUrls": true,
-        "ignoreStrings": true,
-        "ignoreTemplateLiterals": true,
-        "ignoreRegExpLiterals": true,
-        "ignorePattern": "(^(import|export)|(public|private))\\s.+;"
+        "code": 80,
+        "ignoreUrls": true
       }
     ],
     "no-lonely-if": "error",
@@ -156,7 +132,14 @@
         "exceptAfterSingleLine": true
       }
     ],
-    "quotes": ["error", "single"]
+    "quotes": [
+      "error",
+      "single",
+      {
+        "avoidEscape": true,
+        "allowTemplateLiterals": false
+      }
+    ]
   },
   "overrides": [
     {

package.json

@@ -110,6 +110,7 @@
     "eslint-plugin-import": "^2.25.4",
     "eslint-plugin-jsdoc": "^37.9.6",
     "eslint-plugin-node": "^11.1.0",
+    "eslint-plugin-prettier": "^4.0.0",
     "expect": "^27.5.1",
     "mocha": "^9.2.1",
     "mochawesome": "^7.1.1",

src/charging-station/AutomaticTransactionGenerator.ts

@@ -20,6 +20,7 @@ export default class AutomaticTransactionGenerator {
     string,
     AutomaticTransactionGenerator
   >();
+
   public started: boolean;
   private readonly chargingStation: ChargingStation;
   private readonly connectorsStatus: Map<number, Status>;

src/charging-station/ocpp/OCPPIncomingRequestService.ts

@@ -9,6 +9,7 @@ export default abstract class OCPPIncomingRequestService {
     string,
     OCPPIncomingRequestService
   >();
+
   protected chargingStation: ChargingStation;
 
   protected constructor(chargingStation: ChargingStation) {

src/charging-station/ocpp/OCPPRequestService.ts

@@ -34,6 +34,7 @@ export default abstract class OCPPRequestService {
     string,
     OCPPRequestService
   >();
+
   protected readonly chargingStation: ChargingStation;
   private readonly ocppResponseService: OCPPResponseService;
 
@@ -338,37 +339,44 @@ export default abstract class OCPPRequestService {
     meterType?: string,
     params?: SendParams
   ): Promise<BootNotificationResponse>;
+
   public abstract sendStatusNotification(
     connectorId: number,
     status: ChargePointStatus,
     errorCode?: ChargePointErrorCode
   ): Promise<void>;
+
   public abstract sendAuthorize(connectorId: number, idTag?: string): Promise<AuthorizeResponse>;
   public abstract sendStartTransaction(
     connectorId: number,
     idTag?: string
   ): Promise<StartTransactionResponse>;
+
   public abstract sendStopTransaction(
     transactionId: number,
     meterStop: number,
     idTag?: string,
     reason?: StopTransactionReason
   ): Promise<StopTransactionResponse>;
+
   public abstract sendMeterValues(
     connectorId: number,
     transactionId: number,
     interval: number
   ): Promise<void>;
+
   public abstract sendTransactionBeginMeterValues(
     connectorId: number,
     transactionId: number,
     beginMeterValue: MeterValue
   ): Promise<void>;
+
   public abstract sendTransactionEndMeterValues(
     connectorId: number,
     transactionId: number,
     endMeterValue: MeterValue
   ): Promise<void>;
+
   public abstract sendDiagnosticsStatusNotification(
     diagnosticsStatus: DiagnosticsStatus
   ): Promise<void>;

src/charging-station/ocpp/OCPPResponseService.ts

@@ -7,6 +7,7 @@ export default abstract class OCPPResponseService {
     string,
     OCPPResponseService
   >();
+
   protected readonly chargingStation: ChargingStation;
 
   protected constructor(chargingStation: ChargingStation) {

src/performance/PerformanceStatistics.ts

@@ -18,6 +18,7 @@ export default class PerformanceStatistics {
     string,
     PerformanceStatistics
   >();
+
   private readonly objId: string;
   private readonly objName: string;
   private performanceObserver: PerformanceObserver;

src/utils/Configuration.ts

@@ -22,6 +22,7 @@ export default class Configuration {
     'assets',
     'config.json'
   );
+
   private static configurationFileWatcher: fs.FSWatcher;
   private static configuration: ConfigurationData | null = null;
   private static configurationChangeCallback: () => Promise<void>;

src/utils/Constants.ts

@@ -17,52 +17,68 @@ export default class Constants {
   static readonly OCPP_CONFIGURATION_RESPONSE_ACCEPTED = Object.freeze({
     status: ConfigurationStatus.ACCEPTED,
   });
+
   static readonly OCPP_CONFIGURATION_RESPONSE_REJECTED = Object.freeze({
     status: ConfigurationStatus.REJECTED,
   });
+
   static readonly OCPP_CONFIGURATION_RESPONSE_REBOOT_REQUIRED = Object.freeze({
     status: ConfigurationStatus.REBOOT_REQUIRED,
   });
+
   static readonly OCPP_CONFIGURATION_RESPONSE_NOT_SUPPORTED = Object.freeze({
     status: ConfigurationStatus.NOT_SUPPORTED,
   });
+
   static readonly OCPP_SET_CHARGING_PROFILE_RESPONSE_ACCEPTED = Object.freeze({
     status: ChargingProfileStatus.ACCEPTED,
   });
+
   static readonly OCPP_SET_CHARGING_PROFILE_RESPONSE_REJECTED = Object.freeze({
     status: ChargingProfileStatus.REJECTED,
   });
+
   static readonly OCPP_SET_CHARGING_PROFILE_RESPONSE_NOT_SUPPORTED = Object.freeze({
     status: ChargingProfileStatus.NOT_SUPPORTED,
   });
+
   static readonly OCPP_CLEAR_CHARGING_PROFILE_RESPONSE_ACCEPTED = Object.freeze({
     status: ClearChargingProfileStatus.ACCEPTED,
   });
+
   static readonly OCPP_CLEAR_CHARGING_PROFILE_RESPONSE_UNKNOWN = Object.freeze({
     status: ClearChargingProfileStatus.UNKNOWN,
   });
+
   static readonly OCPP_RESPONSE_UNLOCKED = Object.freeze({ status: UnlockStatus.UNLOCKED });
   static readonly OCPP_RESPONSE_UNLOCK_FAILED = Object.freeze({
     status: UnlockStatus.UNLOCK_FAILED,
   });
+
   static readonly OCPP_RESPONSE_UNLOCK_NOT_SUPPORTED = Object.freeze({
     status: UnlockStatus.NOT_SUPPORTED,
   });
+
   static readonly OCPP_AVAILABILITY_RESPONSE_ACCEPTED = Object.freeze({
     status: AvailabilityStatus.ACCEPTED,
   });
+
   static readonly OCPP_AVAILABILITY_RESPONSE_REJECTED = Object.freeze({
     status: AvailabilityStatus.REJECTED,
   });
+
   static readonly OCPP_AVAILABILITY_RESPONSE_SCHEDULED = Object.freeze({
     status: AvailabilityStatus.SCHEDULED,
   });
+
   static readonly OCPP_TRIGGER_MESSAGE_RESPONSE_ACCEPTED = Object.freeze({
     status: TriggerMessageStatus.ACCEPTED,
   });
+
   static readonly OCPP_TRIGGER_MESSAGE_RESPONSE_REJECTED = Object.freeze({
     status: TriggerMessageStatus.REJECTED,
   });
+
   static readonly OCPP_TRIGGER_MESSAGE_RESPONSE_NOT_IMPLEMENTED = Object.freeze({
     status: TriggerMessageStatus.NOT_IMPLEMENTED,
   });