From e3056715015e3de292a61677ea2de83cd26fbb38 Mon Sep 17 00:00:00 2001
From: Will Vedder <willvedder@jxmqxvcv5v.myfiosgateway.com>
Date: Fri, 3 Jun 2022 09:18:07 -0400
Subject: [PATCH] Adding tests to getEnabledClients function

---
 test/tools/utils.test.ts | 64 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 64 insertions(+)
 create mode 100644 test/tools/utils.test.ts

diff --git a/test/tools/utils.test.ts b/test/tools/utils.test.ts
new file mode 100644
index 000000000..3648af362
--- /dev/null
+++ b/test/tools/utils.test.ts
@@ -0,0 +1,64 @@
+import { expect } from 'chai';
+import { getEnabledClients } from '../../src/tools/utils';
+
+describe('#getEnabledClients', () => {
+  const mockExclusions = {};
+  const mockExistingConnections = [
+    { name: 'Existing connection', enabled_clients: ['Client 1', 'Client 2'] },
+  ];
+  const mockClients = [
+    { name: 'Client 1', id: 'client-1-id' },
+    { name: 'Client 2', id: 'client-2-id' },
+    { name: 'Client 3', id: 'client-3-id' },
+  ];
+
+  it('should return no enabled clients if connection does not have any', () => {
+    const mockConnection = {
+      enabled_clients: [], // No enabled clients!
+      name: 'Target connection',
+    };
+
+    const enabledClients = getEnabledClients(
+      mockExclusions,
+      mockConnection,
+      mockExistingConnections,
+      mockClients
+    );
+
+    expect(enabledClients).to.deep.equal([]);
+  });
+
+  it('should return enabled clients when connection defines them', () => {
+    const expectedEnabledClients = ['client-id-1', 'client-id-2']; // Two enabled clients!
+
+    const mockConnection = {
+      enabled_clients: expectedEnabledClients,
+      name: 'Target connection',
+    };
+
+    const enabledClients = getEnabledClients(
+      mockExclusions,
+      mockConnection,
+      mockExistingConnections,
+      mockClients
+    );
+
+    expect(enabledClients).to.deep.equal(expectedEnabledClients);
+  });
+
+  it('should return undefined when enable clients is not defined', () => {
+    const mockConnection = {
+      enabled_clients: undefined, // This connection has no defined clients enabled. See: GH issue #523
+      name: 'Target connection',
+    };
+
+    const enabledClients = getEnabledClients(
+      mockExclusions,
+      mockConnection,
+      mockExistingConnections,
+      mockClients
+    );
+
+    expect(enabledClients).to.equal(undefined);
+  });
+});