Merge pull request #152 from jmolivas/module-generator-test-fix

Show test question only if controller will be generated

Author: jmolivas

Tests/Command/GeneratorModuleCommandTest.php

@@ -39,7 +39,7 @@ public function getInteractiveData()
       // case one basic options
       [
         [],
-        ['foo', 'foo', $dir, 'My Awesome Module', '8.x', 'Other', false, true, true],
+        ['foo', 'foo', $dir, 'My Awesome Module', '8.x', 'Other', false, false, true],
         "foo\nfoo\n$dir\n"
       ],
     ];

src/Command/GeneratorModuleCommand.php

@@ -142,9 +142,14 @@ function ($machine_name) {
     }
     $input->setOption('controller', $controller);
 
-    $tests = $input->getOption('tests');
-    if (!$tests && $dialog->askConfirmation($output, $dialog->getQuestion('Do you want to generate Test', 'yes', '?'), TRUE)) {
-      $tests = TRUE;
+    if ($controller){
+      $tests = $input->getOption('tests');
+      if (!$tests && $dialog->askConfirmation($output, $dialog->getQuestion('Do you want to generate Test', 'yes', '?'), true)) {
+        $tests = true;
+      }
+    }
+    else {
+      $tests = false;
     }
     $input->setOption('tests', $tests);