fix: correctly handle empty sections in the input

redhat-plumbers-in-action · Mar 4, 2024 · f1dd4e9 · f1dd4e9
1 parent 7ebb022
commit f1dd4e9
Show file tree

Hide file tree

Showing 6 changed files with 21 additions and 15 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/dist/utils.js b/dist/utils.js
diff --git a/dist/utils.js.map b/dist/utils.js.map
diff --git a/src/utils.ts b/src/utils.ts
@@ -1,12 +1,14 @@
 export function composeComment(message: string): string {
   const lines = message.split('\n');
-  const composed = lines.map(line => {
-    try {
-      return JSON.parse(line);
-    } catch (e) {
-      return line;
-    }
-  });
+  const composed = lines
+    .map(line => {
+      try {
+        return JSON.parse(line);
+      } catch (e) {
+        return line;
+      }
+    })
+    .filter(section => section !== '');
 
   return composed.join('\n\n---\n\n');
 }
diff --git a/test/unit/util.test.ts b/test/unit/util.test.ts
@@ -31,8 +31,8 @@ describe('Unit tests of general utilities', () => {
       expect(
         composeComment(
           `${JSON.stringify('Hello, world!')}\n${JSON.stringify(
-            'Hello, universe!'
-          )}`
+            ''
+          )}\n${JSON.stringify('Hello, universe!')}`
         )
       ).toMatchInlineSnapshot(`
         "Hello, world!