In preparation of #1205458: Move modules/themes/libraries/files/private directories out of /sites/example.com, we'll need tests to ensure that a site's directories actually get backed up and redeployed, which we aren't doing currently. The attached patch adds two helper functions:

  • drush_provision_tests_dl_module_to_site(), and
  • drush_provision_tests_en_module_on_site()

Basically, it inserts them into the migration tests, downloading the 'token' module to the test site's modules directory just before migration, then attempts to enable the module after migration.

CommentFileSizeAuthor
add_site_dir_tests.patch3.04 KBergonlogic
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
CreditAttribution: ergonlogic commented
Status: Active » Needs review

There's a patch attached that needs review :)

anarcat’s picture

anarcat CreditAttribution: anarcat commented
Status: Needs review » Fixed

i merged this in since this is just good extra tests.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit f2355d5 on dev-drupal-8, 6.x-2.x, dev-ssl-ip-allocation-refactor, dev-1205458-move_sites_out_of_platforms, 7.x-3.x, dev-subdir-multiserver, 6.x-2.x-backports, dev-helmo-3.x by anarcat: 
    Issue #1826074 by ergonlogic: Added tests to ensure site directories are...