Closed (won't fix)
Project:
DrupalCI: Environments
Component:
PHP Containers
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
1 Jun 2016 at 18:06 UTC
Updated:
5 Jul 2020 at 00:03 UTC
Jump to comment: Most recent
Comments
Comment #2
David_Rothstein commentedBased on my results trying this with patch-level testing on a contrib module (see https://www.drupal.org/node/2762535#comment-11379427), as well as running tests on SQLite locally, it seems like the Drupal core bug at #1713332: The SQLite database driver fails to drop simpletest tables results in a test failure for practically every test that is run in both core and contrib. So turning this on for Drupal 7 probably wouldn't be that useful yet. Postponing until that issue is fixed.
Comment #3
MixologicComment #4
MixologicComment #5
elachlan commentedMoved to DrupalCI Environments.
Comment #6
MixologicSo, any patch as well as any branch can choose to test against sqlite. When d7 core is ready to pass all sqlite tests, we can easily add it as a potential default. There exists a 5.5/5.6/7.0.14 environment for sqlite and you can set this as a non-standard environment.
Comment #7
David_Rothstein commentedYeah, things have changed since this issue was originally filed. It used to be impossible to have regular branch testing (e.g. on commit) on SQLite. Now it's possible, but requires choosing something from the "non-standard" section of the Environment dropdown.
That's still appropriate for now, but as soon as #1713332: The SQLite database driver fails to drop simpletest tables is fixed this issue can be un-postponed. (Doesn't necessarily require other SQLite test failures in Drupal core, if there are any, to be resolved - but as explained above that issue essentially blocks contrib testing too.)
Comment #8
David_Rothstein commentedClarifying title - it's up to individual project maintainers whether they want to actually enable this or not :)
Comment #9
MixologicOkay, we deployed a change to the way jobs were being sent so sqlite tests are possible on d7 now: #2842855: DCI_Jobtype should be set by the api version, not configured in the environment settings
Comment #10
MixologicDrupalCI wasnt properly building a d7 site with sqlite, and now it is, so work/testing on d7 sqlite is no longer blocked on the infrastructure.
Unpostpone when you want sqlite moved from "non-standard" to "supported"
Comment #11
MixologicThe 'non-standard' and supported designations no longer exist, thus, nothing to do here.