Merge pull request #825 from fractal-analytics-platform/body-request-normalizer

Added body request normalizer

Author: zonia3000

CHANGELOG.md

@@ -1,5 +1,9 @@
 *Note: Numbers like (\#123) point to closed Pull Requests on the fractal-web repository.*
 
+# Unreleased
+
+* Added body request normalizer (\#825);
+
 # 1.19.3
 
 * Added "Open in Feature Explorer" link and `PUBLIC_FRACTAL_FEATURE_EXPLORER_URL` environment variable (\#820);

components/__tests__/utils.test.js

@@ -1,5 +1,5 @@
 import { describe, it, expect } from 'vitest';
-import { stripNullAndEmptyObjectsAndArrays } from '../src/lib/common/utils';
+import { stripNullAndEmptyObjectsAndArrays, _normalize } from '../src/lib/common/utils';
 
 describe('utils', () => {
 	it('should strip data objects correctly', () => {
@@ -39,4 +39,44 @@ describe('utils', () => {
 			}
 		});
 	});
+
+	it('should normalize a complex nested object and track paths', () => {
+		const payload = {
+			foo: {
+				bar: [{ baz: '  Value 1\u200B ' }, { baz: '  Value 2  ' }, null, { baz: undefined }],
+				qux: '  Some text\u200B '
+			},
+			quux: {
+				corge: '  Another value  ',
+				grault: [{ garply: '  Nested value  ' }, { garply: null }],
+				'key_with_space ': true
+			},
+			'key_with_space ': true
+		};
+
+		const normalizedPaths = _normalize(payload);
+
+		expect(payload).toEqual({
+			foo: {
+				bar: [{ baz: 'Value 1' }, { baz: 'Value 2' }, null, { baz: undefined }],
+				qux: 'Some text'
+			},
+			quux: {
+				corge: 'Another value',
+				grault: [{ garply: 'Nested value' }, { garply: null }],
+				key_with_space: true
+			},
+			key_with_space: true
+		});
+
+		expect(normalizedPaths).toEqual([
+			'foo.bar[0].baz',
+			'foo.bar[1].baz',
+			'foo.qux',
+			'quux.corge',
+			'quux.grault[0].garply',
+			'quux',
+			'root'
+		]);
+	});
 });