Handle single Contents element in ListObjectsV2 XML parsing

Issue: CLDSRVCLT-12

Author: maeldonn

src/commands/s3Extended/utils.ts

@@ -75,7 +75,7 @@ export function parseListObjectsUserMetadataMiddleware(captured: { xml: string }
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
     return (next: any) => async (args: any) => {
         const result = await next(args);
-        const parsed = new XMLParser().parse(captured.xml);
+        const parsed = new XMLParser({ isArray: name => name === 'Contents' }).parse(captured.xml);
         const xmlContents = parsed?.ListBucketResult?.Contents;
         if (result.output.Contents && xmlContents) {
             for (let i = 0; i < result.output.Contents.length; i++) {

tests/commands/s3Extended/utils.test.ts

@@ -0,0 +1,56 @@
+import { parseListObjectsUserMetadataMiddleware } from '../../../src/commands/s3Extended/utils';
+
+describe('parseListObjectsUserMetadataMiddleware', () => {
+    function buildXml(contents: Record<string, string>[]): string {
+        const inner = contents.map(obj => `<Contents>${
+            Object.entries(obj).map(([k, v]) => `<${k}>${v}</${k}>`).join('')
+        }</Contents>`).join('');
+        return `<ListBucketResult>${inner}</ListBucketResult>`;
+    }
+
+    function createNext(output: unknown) {
+        return async () => ({ output });
+    }
+
+    it('should extract user metadata when XML contains a single Contents element', async () => {
+        const captured = {
+            xml: buildXml([
+                { 'Key': 'obj1', 'x-amz-meta-foo': 'bar' },
+            ]),
+        };
+
+        const middleware = parseListObjectsUserMetadataMiddleware(captured);
+        const result = await middleware(createNext({
+            Contents: [{ Key: 'obj1' }],
+        }))({ request: {} });
+        expect(result.output.Contents[0]['x-amz-meta-foo']).toBe('bar');
+    });
+
+    it('should extract user metadata when XML contains multiple Contents elements', async () => {
+        const captured = {
+            xml: buildXml([
+                { 'Key': 'obj1', 'x-amz-meta-foo': 'bar' },
+                { 'Key': 'obj2', 'x-amz-meta-foo': 'baz' },
+            ]),
+        };
+
+        const middleware = parseListObjectsUserMetadataMiddleware(captured);
+        const result = await middleware(createNext({
+            Contents: [{ Key: 'obj1' }, { Key: 'obj2' }],
+        }))({ request: {} });
+        expect(result.output.Contents[0]['x-amz-meta-foo']).toBe('bar');
+        expect(result.output.Contents[1]['x-amz-meta-foo']).toBe('baz');
+    });
+
+    it('should handle empty Contents gracefully', async () => {
+        const captured = {
+            xml: buildXml([]),
+        };
+
+        const middleware = parseListObjectsUserMetadataMiddleware(captured);
+        const result = await middleware(createNext({
+            Contents: undefined,
+        }))({ request: {} });
+        expect(result.output.Contents).toBeUndefined();
+    });
+});