Skip to content

Commit c589829

Browse files
committed
Add root user & password constants
1 parent 3c07a97 commit c589829

File tree

2 files changed

+17
-13
lines changed

2 files changed

+17
-13
lines changed

.github/workflows/testing.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -159,6 +159,8 @@ jobs:
159159
run: |
160160
export MYSQL_HOST=127.0.0.1
161161
export MYSQL_TCP_PORT=${{ job.services.mysql.ports['3306'] }}
162+
echo "WP_CLI_TEST_DBROOTUSER=root" >> $GITHUB_ENV
163+
echo "WP_CLI_TEST_DBROOTPASS=root" >> $GITHUB_ENV
162164
echo "WP_CLI_TEST_DBUSER=wp_cli_test" >> $GITHUB_ENV
163165
echo "WP_CLI_TEST_DBPASS=password1" >> $GITHUB_ENV
164166
echo "WP_CLI_TEST_DBHOST=$MYSQL_HOST:$MYSQL_TCP_PORT" >> $GITHUB_ENV

bin/install-package-tests

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,10 @@
22

33
# Database credentials can be provided via environment variables:
44
# - WP_CLI_TEST_DBHOST is the host to use and can include a port, i.e "127.0.0.1:33060" (defaults to "localhost")
5-
# - WP_CLI_TEST_DBUSER is the user that has permission to administer databases and users (defaults to "root").
6-
# - WP_CLI_TEST_DBPASS is the password to use for the above user (defaults to an empty password).
5+
# - WP_CLI_TEST_DBROOTUSER is the user that has permission to administer databases and users (defaults to "root").
6+
# - WP_CLI_TEST_DBROOTPASS is the password to use for the above user (defaults to an empty password).
7+
# - WP_CLI_TEST_DBUSER is the user that the tests run under (defaults to "wp_cli_test").
8+
# - WP_CLI_TEST_DBPASS is the password to use for the above user (defaults to "password1").
79

810
HOST=localhost
911
PORT=""
@@ -20,28 +22,28 @@ if [ -n "$WP_CLI_TEST_DBHOST" ]; then
2022
fi
2123

2224
USER=root
23-
if [ -n "$WP_CLI_TEST_DBUSER" ]; then
24-
USER="$WP_CLI_TEST_DBUSER"
25+
if [ -n "$WP_CLI_TEST_DBROOTUSER" ]; then
26+
USER="$WP_CLI_TEST_DBROOTUSER"
2527
fi
2628

2729
PASSWORD_STRING=""
28-
if [ -n "$WP_CLI_TEST_DBPASS" ]; then
29-
PASSWORD_STRING="-p$WP_CLI_TEST_DBPASS"
30+
if [ -n "$WP_CLI_TEST_DBROOTPASS" ]; then
31+
PASSWORD_STRING="-p$WP_CLI_TEST_DBROOTPASS"
3032
fi
3133

3234
# Prepare the database for running the tests with a MySQL version 8.0 or higher.
3335
install_db_8_0_plus() {
34-
mysql -e "CREATE DATABASE IF NOT EXISTS wp_cli_test;" $HOST_STRING -u"$USER" "$PASSWORD_STRING"
35-
mysql -e "CREATE USER IF NOT EXISTS wp_cli_test@$HOST IDENTIFIED BY 'password1'" $HOST_STRING -u"$USER" "$PASSWORD_STRING"
36-
mysql -e "GRANT ALL PRIVILEGES ON wp_cli_test.* TO wp_cli_test@%" $HOST_STRING -u"$USER" "$PASSWORD_STRING"
37-
mysql -e "GRANT ALL PRIVILEGES ON wp_cli_test_scaffold.* TO wp_cli_test@%" $HOST_STRING -u"$USER" "$PASSWORD_STRING"
36+
mysql -e "CREATE DATABASE IF NOT EXISTS \`wp_cli_test\`;" $HOST_STRING -u"$USER" "$PASSWORD_STRING"
37+
mysql -e "CREATE USER IF NOT EXISTS \`wp_cli_test\`@$HOST IDENTIFIED BY '$TEST_PASSWORD'" $HOST_STRING -u"$USER" "$PASSWORD_STRING"
38+
mysql -e "GRANT ALL PRIVILEGES ON \`wp_cli_test\`.* TO '$TEST_USER'@'%'" $HOST_STRING -u"$USER" "$PASSWORD_STRING"
39+
mysql -e "GRANT ALL PRIVILEGES ON \`wp_cli_test_scaffold\`.* TO '$TEST_USER'@'%'" $HOST_STRING -u"$USER" "$PASSWORD_STRING"
3840
}
3941

4042
# Prepare the database for running the tests with a MySQL version lower than 8.0.
4143
install_db_lower_than_8_0() {
42-
mysql -e "CREATE DATABASE IF NOT EXISTS wp_cli_test;" $HOST_STRING -u"$USER" "$PASSWORD_STRING"
43-
mysql -e "GRANT ALL ON wp_cli_test.* TO wp_cli_test@% IDENTIFIED BY 'password1'" $HOST_STRING -u"$USER" "$PASSWORD_STRING"
44-
mysql -e "GRANT ALL ON wp_cli_test_scaffold.* TO wp_cli_test@% IDENTIFIED BY 'password1'" $HOST_STRING -u"$USER" "$PASSWORD_STRING"
44+
mysql -e "CREATE DATABASE IF NOT EXISTS \`wp_cli_test\`;" $HOST_STRING -u"$USER" "$PASSWORD_STRING"
45+
mysql -e "GRANT ALL ON \`wp_cli_test\`.* TO '$TEST_USER'@'%' IDENTIFIED BY '$TEST_PASSWORD'" $HOST_STRING -u"$USER" "$PASSWORD_STRING"
46+
mysql -e "GRANT ALL ON \`wp_cli_test_scaffold\`.* TO '$TEST_USER'@'%' IDENTIFIED BY '$TEST_PASSWORD'" $HOST_STRING -u"$USER" "$PASSWORD_STRING"
4547
}
4648

4749
set -ex

0 commit comments

Comments
 (0)