fix: try/catch dockerfile replace (#1022)

lib/workers/branch/dockerfile.js

@@ -9,11 +9,16 @@ function setNewValue(
   newVersion,
   logger
 ) {
-  logger.debug(`setNewValue: ${depName} = ${newVersion}`);
+  try {
-  const regexReplace = new RegExp(`(^|\n)FROM ${currentVersion}\n`);
+    logger.debug(`setNewValue: ${depName} = ${newVersion}`);
-  const newFileContent = currentFileContent.replace(
+    const regexReplace = new RegExp(`(^|\n)FROM ${currentVersion}\n`);
-    regexReplace,
+    const newFileContent = currentFileContent.replace(
-    `$1FROM ${newVersion}\n`
+      regexReplace,
-  );
+      `$1FROM ${newVersion}\n`
-  return newFileContent;
+    );
+    return newFileContent;
+  } catch (err) {
+    logger.info({ err }, 'Error setting new Dockerfile value');
+    return null;
+  }
 }

test/workers/branch/dockerfile.spec.js

@@ -18,5 +18,19 @@ describe('workers/branch/dockerfile', () => {
       );
       expect(res).toMatchSnapshot();
     });
+    it('returns null on error', () => {
+      const currentFileContent = null;
+      const depName = 'node';
+      const currentVersion = 'node:8';
+      const newVersion = 'node:8@sha256:abcdefghijklmnop';
+      const res = dockerfile.setNewValue(
+        currentFileContent,
+        depName,
+        currentVersion,
+        newVersion,
+        logger
+      );
+      expect(res).toBe(null);
+    });
   });
 });