fix(utils): compress should not run if not needed (#4661)

Author: brunozoric

packages/api-headless-cms/__tests__/contentAPI/richTextField.test.ts

@@ -246,7 +246,7 @@ describe("richTextField", () => {
 
         const category = await createCategory();
 
-        const { createProduct, updateProduct } = useProductManageHandler({
+        const { createProduct, updateProduct, getProduct } = useProductManageHandler({
             ...manageOpts
         });
 
@@ -322,6 +322,24 @@ describe("richTextField", () => {
                 }
             }
         });
+
+        const product = createProductResponse.data.createProduct.data;
+
+        const [getAfterCreateResult] = await getProduct({
+            revision: product.id
+        });
+
+        expect(getAfterCreateResult).toMatchObject({
+            data: {
+                getProduct: {
+                    data: {
+                        id: product.id,
+                        ...productData
+                    },
+                    error: null
+                }
+            }
+        });
         /**
          * We now update the rich text field with some value.
          */
@@ -347,5 +365,22 @@ describe("richTextField", () => {
                 }
             }
         });
+
+        const [getAfterUpdateResult] = await getProduct({
+            revision: product.id
+        });
+
+        expect(getAfterUpdateResult).toMatchObject({
+            data: {
+                getProduct: {
+                    data: {
+                        id: product.id,
+                        ...productData,
+                        richText: richTextMock
+                    },
+                    error: null
+                }
+            }
+        });
     });
 });

packages/utils/src/compression/plugins/GzipCompression.ts

@@ -23,6 +23,9 @@ export class GzipCompression extends CompressionPlugin {
     }
 
     public override async compress(data: any): Promise<ICompressedValue> {
+        if (data === null || data === undefined) {
+            return data;
+        }
         // This stringifies both regular strings and JSON objects.
         const value = await gzip(JSON.stringify(data));
 
@@ -43,6 +46,11 @@ export class GzipCompression extends CompressionPlugin {
     }
 
     public override async decompress(data: ICompressedValue): Promise<any> {
+        if (!data) {
+            return data;
+        } else if (!data.value) {
+            return null;
+        }
         try {
             const buf = await ungzip(convertToBuffer(data.value));
             const value = buf.toString(FROM_STORAGE_ENCODING);

packages/utils/src/compression/plugins/JsonpackCompression.ts

@@ -17,6 +17,9 @@ export class JsonpackCompression extends CompressionPlugin {
     }
 
     public override async compress(data: any): Promise<ICompressedValue> {
+        if (data === null || data === undefined) {
+            return data;
+        }
         const value = await compress(data);
 
         return {
@@ -37,6 +40,11 @@ export class JsonpackCompression extends CompressionPlugin {
     }
 
     public override async decompress(data: ICompressedValue): Promise<any> {
+        if (!data) {
+            return data;
+        } else if (!data.value) {
+            return null;
+        }
         try {
             return await decompress(data.value);
         } catch (ex) {