Skip to content

Tests: Document CUSTOM_USER_TABLE SQLite incompatibility as expected behavior #328

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
swissspidy merged 2 commits into from
Apr 3, 2026
+4 −4
Merged

Tests: Document CUSTOM_USER_TABLE SQLite incompatibility as expected behavior #328

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
54defea
Initial plan
Copilot Apr 3, 2026
eedc8c6
Tests: Document CUSTOM_USER_TABLE SQLite incompatibility investigatio…
Copilot Apr 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions features/core-install.feature
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
Feature: Install WordPress core

# TODO: Requires investigation for SQLite support.
# See https://github.com/wp-cli/core-command/issues/244
# Not compatible with SQLite: each WordPress install uses its own separate SQLite database
# file, so cross-install table sharing via CUSTOM_USER_TABLE is not possible with SQLite.
Comment on lines +3 to +4
Copy link

Copilot AI Apr 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The new explanation is helpful, but removing the reference to #244 loses traceability for why these scenarios are permanently @require-mysql. Consider keeping the link (e.g., as an additional comment line) so future maintainers can find the original discussion. Please mirror the update in the duplicated comment block before the second CUSTOM_USER_TABLE scenario as well.

Copilot uses AI. Check for mistakes.
@require-mysql
Scenario: Two WordPress installs sharing the same user table won't update existing user
Given an empty directory
Expand Down Expand Up @@ -86,8 +86,8 @@ Feature: Install WordPress core
wp_users
"""

# TODO: Requires investigation for SQLite support.
# See https://github.com/wp-cli/core-command/issues/244
# Not compatible with SQLite: each WordPress install uses its own separate SQLite database
# file, so cross-install table sharing via CUSTOM_USER_TABLE is not possible with SQLite.
@require-mysql
Scenario: Two WordPress installs sharing the same user table will create new user
Given an empty directory
Expand Down