Added additional test cases for fixPropertiesJsonString, that handles nested objects and no properties field

Author: d1g1tal-lore-34

azure/test/activity_logs_monitoring.test.js

@@ -3,6 +3,7 @@ const client = require('../activity_logs_monitoring').forTests;
 const constants = client.constants;
 const sinon = require('sinon');
 const { InvocationContext } = require('@azure/functions');
+const { hostname } = require('os');
 
 function fakeContext() {
     return new InvocationContext({
@@ -827,6 +828,43 @@ describe('EventhubLogHandler Fix Properties Json String', function() {
 	it('parses properties string with single quotes into object', function() {
 		const record = { properties: "{'key':'value'}" };
         const result = this.forwarder.fixPropertiesJsonString(record);
+
+        const expectedProperties = { properties: { key: "value" } };
+
+        assert.deepEqual(
+            expectedProperties,
+            result
+        );
+        assert.equal(
+            'object',
+            typeof result.properties
+        )
+	});
+
+    it('parses object that doesnt have properties', function() {
+		const record = { hostname: "server_name", subObject: { key:"value"} };
+        const result = this.forwarder.fixPropertiesJsonString(record);
+
+        assert.deepEqual(
+            record,
+            result
+        );
+        assert.equal(
+            'object',
+            typeof result
+        )
+	});
+
+	it('parses properties string with nested objects', function() {
+		const record = { properties: "{'key':'value','subObj':{ 'subKey' : 'subValue' }}" };
+        const result = this.forwarder.fixPropertiesJsonString(record);
+
+        const expectedProperties = { properties: { key: "value", subObj: { subKey: "subValue"} } };
+
+        assert.deepEqual(
+            expectedProperties,
+            result
+        );
         assert.equal(
             'object',
             typeof result.properties
@@ -836,6 +874,11 @@ describe('EventhubLogHandler Fix Properties Json String', function() {
 	it("leaves properties string unchanged when it doesn't match the malformed pattern", function() {
 		const record = { properties: 'some plain string without colons' };
 		const result = this.forwarder.fixPropertiesJsonString(record);
+
+        assert.deepEqual(
+            record,
+            result
+        );
         assert.equal(
             'string',
             typeof result.properties
@@ -846,6 +889,11 @@ describe('EventhubLogHandler Fix Properties Json String', function() {
 		// includes the "':'" marker so the function attempts replacement, but JSON remains invalid
 		const badRecord = { properties: "Look i know i shouldn't but, i will do this ':' " };
 		const result = this.forwarder.fixPropertiesJsonString(badRecord);
+        
+        assert.deepEqual(
+            badRecord,
+            result
+        );
         assert.equal(
             'string',
             typeof result.properties