feat: add configuration tests (#411)

chore: add configuration tests

lib/modules/config.js

@@ -5,29 +5,31 @@ const { addContributorWithDetails } = require("all-contributors-cli");
 const { AllContributorBotError } = require("./errors");
 
 class Config {
-  constructor(repository) {
+  constructor(repository, options) {
     this.repository = repository;
+
+    this.DEFAULT_OPTIONS = {
+      files: ["README.md"],
+      imageSize: 100,
+      commit: false,
+      commitConvention: "angular",
+      contributors: [],
+      contributorsPerLine: 7,
+      skipCi: true,
+      repoType: "github",
+      repoHost: "https://github.com"
+    };
+
+    this.options = Object.assign(this.DEFAULT_OPTIONS, options);
   }
 
   ensureValid() {
     const { repo, owner } = this.repository;
 
+    this.get();
+
     this.options.projectName = repo;
     this.options.projectOwner = owner;
-    this.options.repoType = "github";
-    this.options.repoHost = "https://github.com";
-
-    if (typeof this.options.skipCi !== "boolean") {
-      this.options.skipCi = true;
-    }
-
-    if (!this.options.contributors) {
-      this.options.contributors = [];
-    }
-
-    if (!Array.isArray(this.options.contributors)) {
-      this.options.contributors = [];
-    }
   }
 
   async fetch() {
@@ -56,29 +58,25 @@ class Config {
   }
 
   init() {
-    this.options = {
-      files: ["README.md"],
-      imageSize: 100,
-      commit: false,
-      commitConvention: "angular",
-      contributors: [],
-      contributorsPerLine: 7,
-    };
-
+    this.get();
     this.ensureValid();
   }
 
   get() {
     const options = this.options;
+
     if (!Array.isArray(options.files)) {
       options.files = ["README.md"];
     }
+
     if (!Number.isInteger(options.contributorsPerLine)) {
       options.contributorsPerLine = 7;
     }
-    if (!("commitConvention" in options)) {
-      options.commitConvention = "angular";
+
+    if (!Array.isArray(options.contributors)) {
+      options.contributors = [];
     }
+
     return options;
   }
 
@@ -98,6 +96,7 @@ class Config {
     const options = this.options;
 
     const contributors = options.contributors;
+    // TODO: Improve `contributors` mapping from CLI
     for (let i = 0; i < contributors.length; i++) {
       if (contributors[i].login === username) {
         return contributors[i].contributions;
@@ -117,10 +116,6 @@ class Config {
   async addContributor({ login, contributions, name, avatar_url, profile }) {
     const options = this.options;
 
-    const profileWithProtocol = profile.startsWith("http")
-      ? profile
-      : `http://${profile}`;
-
     const oldContributions = this._findOldContributions(login);
     const newContributions = [
       ...new Set([...oldContributions, ...contributions]),
@@ -132,7 +127,7 @@ class Config {
       contributions: newContributions,
       name,
       avatar_url,
-      profile: profileWithProtocol,
+      profile: profile,
     });
     const newOptions = {
       ...options,