[PATCH 1/2] t2500: use unique identifiers

2018-08-12 Thread Lukas Fleischer
Use disjoint sets of IDs for users, package bases, package comments and package requests to ensure the notification script expects the parameters in the same order we pass them. Signed-off-by: Lukas Fleischer --- test/t2500-notify.sh | 75 +++- 1 file

[PATCH] Initialize locale directory for tests

2018-08-12 Thread Lukas Fleischer
Since commit a7865ef (Make the locale directory configurable, 2018-07-22), we need to specify the locale directory in the configuration file. Signed-off-by: Lukas Fleischer --- test/setup.sh | 1 + 1 file changed, 1 insertion(+) diff --git a/test/setup.sh b/test/setup.sh index 535e819..5c761f2

[PATCH 2/2] t2500: add test for disown notifications

2018-08-12 Thread Lukas Fleischer
Signed-off-by: Lukas Fleischer --- test/t2500-notify.sh | 19 +++ 1 file changed, 19 insertions(+) diff --git a/test/t2500-notify.sh b/test/t2500-notify.sh index 3f5a09c..c9c7208 100755 --- a/test/t2500-notify.sh +++ b/test/t2500-notify.sh @@ -175,6 +175,25 @@

Re: [aur-dev][PATCH] Fix notifications emails going to the right people

2018-08-12 Thread Eli Schwartz
On 8/11/18 4:25 AM, Lukas Fleischer wrote: > On Fri, 10 Aug 2018 at 23:26:28, Eli Schwartz wrote: >> The notify script expects to see the userid followed by additional >> arguments like the pkgbase id, however, these were getting sent swapped >> around (presumably due to the similarity with the db