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

@@ -827,6 +827,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 +873,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 +888,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