WIP: sqlite testing #3385620
4 open threads
Closes #3385620
Merge request reports
Activity
66 66 allow_failure: false 67 67 variables: 68 68 _TARGET_PHP: "8.2" 69 _TARGET_DB: "mysql-8" 69 _TARGET_DB: "sqlite-3" -
leftover?I guess the default for all linting jobs as the image should be lighter than mysql.Edited by Fran Garcia-Linares
187 187 _TARGET_PHP: "8.2" 188 188 _TARGET_DB: "pgsql-16" 189 189 190 # 'PHP 8.2 SQLite 3.34.0': 191 # <<: [ *default-stage, *run-on-commit ] 192 # variables: 193 # _TARGET_PHP: "8.2" 194 # _TARGET_DB: "php-$_TARGET_PHP-apache" 190 'PHP 8.2 SQLite 3.34.0': 191 <<: [ *default-stage, *run-on-commit ] 192 variables: 193 _TARGET_PHP: "8.2" 194 _TARGET_DB: "sqlite-3" 137 137 script: 138 138 # Determine DB driver. 139 139 - | 140 [[ $_TARGET_DB == php* ]] && export SIMPLETEST_DB=sqlite://localhost/subdirectory/sites/default/files/db.sqlite?module=sqlite 140 [[ $_TARGET_DB == sqlite* ]] && export SIMPLETEST_DB=sqlite://localhost/$CI_PROJECT_DIR/sites/default/files/db.sqlite?module=sqlite -
I would either leave the
[[ $_TARGET_DB == php* ]]so you can use the real image name or even remove the[[ $_TARGET_DB == php* ]] &&part. That way we default to SQLite and then change only ifmysql,pssqlormariadbare given.Also, why change
subdirectoryfor$CI_PROJECT_DIR. The localhost endpoint is set to belocalhost/subdirectoryvia apache. changed this line in version 9 of the diff
added 43 commits
-
a1410e0f...f638625b - 42 commits from branch
project:11.x - e33203c8 - WIP: sqlite testing
-
a1410e0f...f638625b - 42 commits from branch
91 91 CI_PARALLEL_NODE_TOTAL: $CI_NODE_TOTAL 92 92 93 93 .with-database: &with-database 94 name: $_CONFIG_DOCKERHUB_ROOT/$_TARGET_DB:production 94 name: $_CONFIG_DOCKERHUB_ROOT/$_TARGET_DB:dev changed this line in version 11 of the diff
added 1 commit
- 193ec450 - Try variable again as some tests are failing.
added 2 commits
added 133 commits
-
61809e09...6ae1c4f8 - 132 commits from branch
project:11.x - 66929f71 - WIP: 8.1:dev image testing
-
61809e09...6ae1c4f8 - 132 commits from branch
Please register or sign in to reply