set prompt value in when and return false (#665)

SBoudrias · May 14, 2018 · ff89798 · ff89798
1 parent 54cc022
commit ff89798
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 2 deletions.
diff --git a/lib/ui/prompt.js b/lib/ui/prompt.js
@@ -52,10 +52,10 @@ class PromptUI extends Base {
    * Once all prompt are over
    */
 
-  onCompletion(answers) {
+  onCompletion() {
     this.close();
 
-    return answers;
+    return this.answers;
   }
 
   processQuestion(question) {

diff --git a/test/specs/inquirer.js b/test/specs/inquirer.js
@@ -592,6 +592,26 @@ describe('inquirer.prompt', function() {
         expect(answers.q2).to.equal('foo-bar');
       });
     });
+
+    it('should get the value which set in `when` on returns false', function() {
+      var prompts = [
+        {
+          name: 'q',
+          message: 'message',
+          when: function(answers) {
+            answers.q = 'foo';
+            return false;
+          }
+        }
+      ];
+
+      var promise = this.prompt(prompts);
+      autosubmit(promise.ui);
+
+      return promise.then(answers => {
+        expect(answers.q).to.equal('foo');
+      });
+    });
   });
 
   describe('#registerPrompt()', function() {