Add support for volumes for deployments

Author: yamalight

src/commands/config.js

@@ -20,7 +20,24 @@ const pairValidation = input => {
     return key && val;
   });
   if (res.some(r => !r)) {
-    return `Value should be specified in 'key=val,key2=val2' format!`;
+    return `Values should be specified in 'key=val,key2=val2' format!`;
+  }
+  return true;
+};
+
+const volumeValidation = input => {
+  if (!input) {
+    return true;
+  }
+
+  const pairs = input.split(',');
+  const res = pairs.map(pair => {
+    const s = pair.split(':');
+    const [key, val] = s;
+    return key && val;
+  });
+  if (res.some(r => !r)) {
+    return `Values should be specified in 'src:dest,src2:dest2' format!`;
   }
   return true;
 };
@@ -112,6 +129,14 @@ exports.handler = async () => {
     filter,
     validate: pairValidation,
   });
+  prompts.push({
+    type: 'input',
+    name: 'volumes',
+    message: 'Volumes [comma-separated, optional]:',
+    default: defaultConfig.volumes ? defaultConfig.volumes.join(', ') : '',
+    filter,
+    validate: volumeValidation,
+  });
   prompts.push({
     type: 'confirm',
     name: 'enableRatelimit',
@@ -167,6 +192,7 @@ exports.handler = async () => {
     type: 'confirm',
     name: 'basicAuth',
     message: 'Add a basic auth user? [optional]:',
+    default: Boolean(defaultConfig.basicAuth),
   });
 
   // prompts for recursive questions
@@ -209,6 +235,7 @@ exports.handler = async () => {
     project,
     env,
     labels,
+    volumes,
     enableRatelimit,
     ratelimitPeriod,
     ratelimitAverage,
@@ -247,6 +274,9 @@ exports.handler = async () => {
       .map(pair => ({key: pair[0].trim(), value: pair[1].trim()}))
       .reduce((prev, obj) => Object.assign(prev, {[obj.key]: obj.value}), {});
   }
+  if (volumes && volumes.length) {
+    config.volumes = volumes.split(',').map(v => v.trim());
+  }
   if (enableRatelimit) {
     config.rateLimit = {
       period: ratelimitPeriod,

test/config.test.js

@@ -26,23 +26,27 @@ const configData = {
   ratelimitPeriod: 10,
   ratelimitAverage: 20,
   ratelimitBurst: 30,
+  volumes: 'test:/volume',
   basicAuth: true,
 };
-const users = [{
-  username: 'user1',
-  password: 'pass',
-  askAgain: true
-}, {
-  username: 'user2',
-  password: 'pass',
-  askAgain: false
-}];
+const users = [
+  {
+    username: 'user1',
+    password: 'pass',
+    askAgain: true,
+  },
+  {
+    username: 'user2',
+    password: 'pass',
+    askAgain: false,
+  },
+];
 const configPath = path.join(process.cwd(), 'exoframe.json');
 
 const verifyBasicAuth = (input, actual) => {
   actual.split(',').forEach((element, index) => {
-    const hash = element.split(':')[1]
-    expect(hash).toEqual(md5(input[index].password, hash))
+    const hash = element.split(':')[1];
+    expect(hash).toEqual(md5(input[index].password, hash));
   });
 };
 
@@ -58,10 +62,14 @@ beforeAll(() => {
 // test config generation
 test('Should generate config file', done => {
   // stup inquirer answers
-  sinon.stub(inquirer, 'prompt')
-  .onFirstCall().callsFake(() => Promise.resolve(configData))
-  .onSecondCall().callsFake(() => Promise.resolve(users[0]))
-  .onThirdCall().callsFake(() => Promise.resolve(users[1]));
+  sinon
+    .stub(inquirer, 'prompt')
+    .onFirstCall()
+    .callsFake(() => Promise.resolve(configData))
+    .onSecondCall()
+    .callsFake(() => Promise.resolve(users[0]))
+    .onThirdCall()
+    .callsFake(() => Promise.resolve(users[1]));
   // spy on console
   const consoleSpy = sinon.spy(console, 'log');
   // execute login