Originally part of #2171683: Remove all Simpletest overrides and rely on native multi-site functionality instead:
Simpletest only has an option to keep test assertion results, but does not allow to retain the test site + its database tables yet.
Retaining the complete test artifacts allows developers to perform advanced debugging.
Attached patch is extracted from the parent issue and rolled against HEAD, but it will be easier to re-extract the changes from the patch in the parent issue once that has landed.
This patch does not really make sense without the test site directory changes from the parent issue.
Comment | File | Size | Author |
---|---|---|---|
#4 | drupal8.test-keep-artifacts.4.patch | 10.31 KB | sun |
drupal8.test-keep-artifacts.0.patch | 9.66 KB | sun | |
Comments
Comment #1
sunComment #2
jibranNR for testing.
Comment #4
sunComment #5
amateescu CreditAttribution: amateescu commentedWhy are we keeping 'clear_results' separate instead of moving it under 'clear'?
Comment #6
amateescu CreditAttribution: amateescu commentedAlso, are there any security considerations that we need to take into account if we leave full-blown site installations around in the file system?
Comment #7
sunre: #5: Because that is out of scope here → #2175151: Rename simpletest.settings:clear_results into clear.results
re: #6:
Regarding security, the full corresponding chapter in the issue summary of #2171683: Remove all Simpletest overrides and rely on native multi-site functionality instead applies and explains the situation. In short: The sites cannot be accessed without having the valid private key of the corresponding site.
Comment #8
amateescu CreditAttribution: amateescu commentedOk then, let's add that issue as related.
Comment #9
sunComment #12
mgiffordUnassigning so someone else can take it on.
Comment #21
quietone CreditAttribution: quietone as a volunteer commentedTriaging issues in simpletest.module to determine if they should be in the Simpletest Project or core.
This looks like it a simpletest module issue, changing the project.