From: Daniel Axtens <dja@axtens.net>
Date: Tue, 22 Jun 2021 15:03:19 +0000 (+1000)
Subject: tests: fix parallel tests
X-Git-Tag: v3.1.0~91
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=d2174b9bd7cb6187e0dc6c16edf874cdce079067;p=thirdparty%2Fpatchwork.git

tests: fix parallel tests

Parallel tests require:

 - the % wildcard to be in a token enclosed by backticks, not single
   quotes

 - that the user still be able to use 'test_patchwork' (so we don't want
   the \_ before the %)

Presumably this was skipped because if you get permissions working
manually but you miss part of the required permissions in the automated
script, you need to delete the old db data in order to observe the
issue.

Amusingly postgres worked the whole time.

Fixes: 6025f0e2533f ("Add parallel testing")
Signed-off-by: Daniel Axtens <dja@axtens.net>
---

diff --git a/tools/docker/entrypoint.sh b/tools/docker/entrypoint.sh
index 8f7ea4f7..5450a535 100755
--- a/tools/docker/entrypoint.sh
+++ b/tools/docker/entrypoint.sh
@@ -26,7 +26,7 @@ reset_data_mysql() {
 DROP DATABASE IF EXISTS patchwork;
 CREATE DATABASE patchwork CHARACTER SET utf8;
 GRANT ALL ON patchwork.* TO 'patchwork' IDENTIFIED BY 'password';
-GRANT ALL ON 'test_patchwork\_%'.* to 'patchwork'@'%';
+GRANT ALL ON \`test\\_patchwork%\`.* to 'patchwork'@'%';
 FLUSH PRIVILEGES;
 EOF
 }