Skip drupal:update if Drupal is not installed

[richardbporter]

Fixes #3277

Changes proposed

This change checks to see if a site can bootstrap via Inspector::isDrupalInstalled before invoking the drupal:update command. If the site cannot bootstrap, it logs a warning but continues to attempt database updates on other multisites.

Steps to replicate the issue

The steps originally outlined in #3277 suffice but locally you can replicate this by:

• Creating two multisites locally
  • a.com
  • b.com
• Installing Drupal in b.com and not in a.com
• Run blt auda

Previous (bad) behavior, before applying PR

The previous behavior would cause the blt auda command to fail on a.com. This actually has potentially negative consequences like leaving b.com or any number of other multisites in a wonky state during AC deployments.

Expected behavior, after applying PR and re-running test steps

The blt auda command logs a warning, skips running database updates on a.com and proceeds to b.com.

[danepowell]

Looks great! Thanks a lot for the contribution!

I tested on a local multisite with default installed, but not sites "foo" or "bar". As expected, updates were applied to default but skipped for foo and bar.

vagrant@local:/var/www/blt-canary$ blt auda
[warning] Drupal is not installed for bar. Skipping updates.
Deploying updates to default...
> drupal:update
[Acquia\Blt\Robo\Tasks\DrushTask] Running /var/www/blt-canary/vendor/bin/drush @self updb --no-interaction --ansi in /var/www/blt-canary/docroot
 [success] No pending updates.
[Acquia\Blt\Robo\Tasks\DrushTask] Done in 4.37s
> drupal:config:import
[Acquia\Blt\Robo\Tasks\DrushTask] Running /var/www/blt-canary/vendor/bin/drush @self config:set system.site uuid 641cfab7-0980-487d-b5b1-3d9c45dd10b3 --no-interaction --ansi &&
 /var/www/blt-canary/vendor/bin/drush @self pm-enable config_split --no-interaction --ansi &&
 /var/www/blt-canary/vendor/bin/drush @self config-import sync --no-interaction --ansi &&
 /var/www/blt-canary/vendor/bin/drush @self config-import sync --no-interaction --ansi &&
 /var/www/blt-canary/vendor/bin/drush @self cache-rebuild --no-interaction --ansi in /var/www/blt-canary/docroot
 [notice] Already enabled: config_split
 [notice] There are no changes to import.
 [notice] There are no changes to import.
 [success] Cache rebuild complete.
[Acquia\Blt\Robo\Tasks\DrushTask] Done in 6.411s
> drupal:toggle:modules
Finished deploying updates to default.
[warning] Drupal is not installed for foo. Skipping updates.

[richardbporter]

Thanks! I briefly tried to write a test but I had trouble getting BLT tests to run locally. I also didn't see an existing Inspector or DeployCommand test.

[stephenthedev]

Looks good!