fix(verify): gracefully handle options without validator (#364)

Author: fgreinacher

lib/verify.js

@@ -32,11 +32,14 @@ module.exports = async (pluginConfig, context) => {
   debug('apiUrl: %o', gitlabApiUrl);
   debug('repoId: %o', repoId);
 
+  const isValid = (option, value) => {
+    const validator = VALIDATORS[option];
+    return isNil(value) || isNil(validator) || VALIDATORS[option](value);
+  };
+
   const errors = Object.entries({...options}).reduce(
     (errors, [option, value]) =>
-      !isNil(value) && !VALIDATORS[option](value)
-        ? [...errors, getError(`EINVALID${option.toUpperCase()}`, {[option]: value})]
-        : errors,
+      isValid(option, value) ? errors : [...errors, getError(`EINVALID${option.toUpperCase()}`, {[option]: value})],
     []
   );
 

test/verify.test.js

@@ -783,3 +783,22 @@ test.serial('Throw SemanticReleaseError if "assignee" option is a whitespace Str
   t.is(error.code, 'EINVALIDASSIGNEE');
   t.true(gitlab.isDone());
 });
+
+test.serial('Does not throw an error for option without validator', async t => {
+  const owner = 'test_user';
+  const repo = 'test_repo';
+  const env = {GL_TOKEN: 'gitlab_token'};
+  const gitlab = authenticate(env)
+    .get(`/projects/${owner}%2F${repo}`)
+    .reply(200, {permissions: {project_access: {access_level: 30}}});
+
+  await t.notThrowsAsync(
+    verify(
+      {
+        someOption: 42,
+      },
+      {env, options: {repositoryUrl: `https://gitlab.com/${owner}/${repo}.git`}, logger: t.context.logger}
+    )
+  );
+  t.true(gitlab.isDone());
+});