Adding @todos. (acquia#1684)

* Fixing issue preventing Behat tests from running on ci.

* Importing features after site installation.

* Adding @todos.

* PHPCS.

scripts/travis/run_tests

@@ -7,6 +7,6 @@ yaml-cli update:value blt/project.yml project.local.hostname '127.0.0.1:8888'
 
 blt validate:all
 blt setup -D drush.alias='${drush.aliases.ci}' --no-interaction
-blt tests:all --define tests.run-server=true --yes -v
+blt tests:all drush.alias='${drush.aliases.ci}' --define tests.run-server=true --yes -v
 
 set +v

src/Robo/Commands/Blt/UpdateCommand.php

@@ -265,6 +265,7 @@ protected function executeSchemaUpdates($starting_version) {
     if ($updates) {
       $this->say("<comment>The following BLT updates are outstanding:</comment>");
       $updater->printUpdates($updates);
+      // @todo Do not prompt if this is being run from Plugin.php.
       $confirm = $this->confirm('Would you like to perform the listed updates?');
       if ($confirm) {
         try {

src/Robo/Commands/Setup/BuildCommand.php

@@ -45,7 +45,7 @@ public function setup() {
   public function drupalInstall() {
     $commands = ['internal:drupal:install'];
     $strategy = $this->getConfigValue('cm.strategy');
-    if ($strategy == 'config-split') {
+    if (in_array($strategy, ['config-split', 'features'])) {
       $commands[] = 'setup:config-import';
     }
 

src/Robo/Exceptions/BltException.php

@@ -17,6 +17,9 @@ public function __construct(
     $code = 0,
     \Throwable $previous = NULL
   ) {
+
+    // @todo Check verbosity level. If not verbose, append "Re-run the command
+    // with -v to see more verbose output."
     parent::__construct($message, $code, $previous);
 
     $this->transmitAnalytics();