Fixed ToJSON accidental mutation (#191)

packages/azure-kusto-data/source/clientRequestProperties.ts

@@ -78,14 +78,14 @@ export class ClientRequestProperties {
         } = {};
 
         if (Object.keys(this._options).length !== 0) {
-            json.Options = this._options;
+            json.Options = { ...this._options };
             if (json.Options.servertimeout) {
                 json.Options.servertimeout = this._msToTimespan(json.Options.servertimeout as number);
             }
         }
 
         if (Object.keys(this._parameters).length !== 0) {
-            json.Parameters = this._parameters;
+            json.Parameters = { ...this._parameters };
         }
 
         return Object.keys(json).length !== 0 ? json : null;

packages/azure-kusto-data/test/clientTest.ts

@@ -53,6 +53,29 @@ describe("KustoClient", () => {
         });
     });
 
+    describe("timeout", () => {
+        it("ClientRequestProperties.ToJSON doesn't affect results", () => {
+            const url = "https://cluster.kusto.windows.net";
+            const client = new KustoClient(url);
+            const clientRequestProps = new ClientRequestProperties();
+            const timeout = moment.duration(3.53, "minutes");
+            const clientServerDelta = moment.duration(0.5, "minutes");
+            const totalTimeout = timeout.clone().add(clientServerDelta);
+
+            clientRequestProps.setTimeout(timeout.asMilliseconds());
+            assert.strictEqual(client._getClientTimeout(ExecutionType.Query, clientRequestProps), totalTimeout.asMilliseconds());
+
+            const json = clientRequestProps.toJSON();
+            assert.strictEqual(json?.Options?.servertimeout, "00:03:31.8");
+
+            client._getClientTimeout(ExecutionType.Query, clientRequestProps);
+            assert.strictEqual(client._getClientTimeout(ExecutionType.Query, clientRequestProps), totalTimeout.asMilliseconds());
+
+            const json2 = clientRequestProps.toJSON();
+            assert.strictEqual(json2?.Options?.servertimeout, "00:03:31.8");
+        });
+    });
+
     describe("#_parseResponse()", () => {
         it("valid v1", () => {
             const url = "https://cluster.kusto.windows.net";