Add tests for get-config

- Add test suite for `get-config`
- Fix `get-config` to handle merging nested `gpg` property correctly

lib/get-config.js

@@ -3,6 +3,6 @@ const defaultConfig = {
 };
 
 module.exports = async context => {
-  const repoConfig = await context.config('config.yml', { gpg: defaultConfig });
-  return repoConfig.gpg;
+  const repoConfig = await context.config('config.yml', { gpg: {} });
+  return Object.assign({}, defaultConfig, repoConfig.gpg);
 };

test/get-config.js

@@ -0,0 +1,43 @@
+const assert = require('assertive');
+
+const getConfig = require('../lib/get-config');
+
+const ContextMock = require('./mocks/context');
+
+describe('get-config', () => {
+  it('should get the config', async () => {
+    // Arrange
+    const expected = { ignoreWebFlow: true };
+    const contextMock = new ContextMock(null, { gpg: expected });
+
+    // Act
+    const actual = await getConfig(contextMock);
+
+    // Assert
+    assert.deepEqual(expected, actual);
+  });
+
+  it('should return the default config if no GPG options are defined', async () => {
+    // Arrange
+    const expected = { ignoreWebFlow: false };
+    const contextMock = new ContextMock(null, {});
+
+    // Act
+    const actual = await getConfig(contextMock);
+
+    // Assert
+    assert.deepEqual(expected, actual);
+  });
+
+  it('should return the default config if other GPG options are defined', async () => {
+    // Arrange
+    const expected = { unused: 5, ignoreWebFlow: false };
+    const contextMock = new ContextMock(null, { gpg: { unused: 5 } });
+
+    // Act
+    const actual = await getConfig(contextMock);
+
+    // Assert
+    assert.deepEqual(expected, actual);
+  });
+});

test/mocks/context.js

@@ -1,6 +1,7 @@
 module.exports = class ContextMock {
-  constructor(payload) {
+  constructor(payload, config) {
     this.payload = payload;
+    this.configObj = config;
   }
 
   repo(object) {
@@ -9,4 +10,8 @@ module.exports = class ContextMock {
       repo: 'repo'
     }, object);
   }
+
+  async config() {
+    return Promise.resolve(this.configObj);
+  }
 };