From a99db00fb2bc9926d911fa3859141984a1994442 Mon Sep 17 00:00:00 2001
From: Thomas Dupont <thomas.dupont@ugent.be>
Date: Wed, 1 Jun 2022 10:43:37 +0200
Subject: [PATCH] fix: remove workaround for cli parameters being turned into
 string

---
 src/init/cluster/ClusterManager.ts            |  6 +-----
 test/unit/init/cluster/ClusterManager.test.ts | 18 +++++++++---------
 2 files changed, 10 insertions(+), 14 deletions(-)

diff --git a/src/init/cluster/ClusterManager.ts b/src/init/cluster/ClusterManager.ts
index 608da28869..72e6988af3 100644
--- a/src/init/cluster/ClusterManager.ts
+++ b/src/init/cluster/ClusterManager.ts
@@ -49,12 +49,8 @@ export class ClusterManager {
   private readonly workers: number;
   private readonly clusterMode: ClusterMode;
 
-  public constructor(workers: number | string) {
+  public constructor(workers: number) {
     const cores = cpus().length;
-    // Workaround for https://github.com/CommunitySolidServer/CommunitySolidServer/issues/1182
-    if (typeof workers === 'string') {
-      workers = Number.parseInt(workers, 10);
-    }
 
     if (workers <= -cores) {
       throw new InternalServerError('Invalid workers value (should be in the interval ]-num_cores, +∞).');
diff --git a/test/unit/init/cluster/ClusterManager.test.ts b/test/unit/init/cluster/ClusterManager.test.ts
index 4541bcd8f8..a9538a0f65 100644
--- a/test/unit/init/cluster/ClusterManager.test.ts
+++ b/test/unit/init/cluster/ClusterManager.test.ts
@@ -30,7 +30,7 @@ describe('A ClusterManager', (): void => {
   });
 
   it('can handle workers input as string.', (): void => {
-    const cm = new ClusterManager('4');
+    const cm = new ClusterManager(4);
     expect(cm.isSingleThreaded()).toBeFalsy();
   });
 
@@ -46,14 +46,14 @@ describe('A ClusterManager', (): void => {
   });
 
   it('errors on invalid workers amount.', (): void => {
-    expect((): ClusterManager => new ClusterManager('10')).toBeDefined();
-    expect((): ClusterManager => new ClusterManager('2')).toBeDefined();
-    expect((): ClusterManager => new ClusterManager('1')).toBeDefined();
-    expect((): ClusterManager => new ClusterManager('0')).toBeDefined();
-    expect((): ClusterManager => new ClusterManager('-1')).toBeDefined();
-    expect((): ClusterManager => new ClusterManager('-5')).toBeDefined();
-    expect((): ClusterManager => new ClusterManager('-6')).toThrow('Invalid workers value');
-    expect((): ClusterManager => new ClusterManager('-10')).toThrow('Invalid workers value');
+    expect((): ClusterManager => new ClusterManager(10)).toBeDefined();
+    expect((): ClusterManager => new ClusterManager(2)).toBeDefined();
+    expect((): ClusterManager => new ClusterManager(1)).toBeDefined();
+    expect((): ClusterManager => new ClusterManager(0)).toBeDefined();
+    expect((): ClusterManager => new ClusterManager(-1)).toBeDefined();
+    expect((): ClusterManager => new ClusterManager(-5)).toBeDefined();
+    expect((): ClusterManager => new ClusterManager(-6)).toThrow('Invalid workers value');
+    expect((): ClusterManager => new ClusterManager(-10)).toThrow('Invalid workers value');
   });
 
   it('has an isPrimary() that works.', (): void => {