summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLukas Fleischer <lfleischer@archlinux.org>2016-10-11 07:44:21 +0200
committerLukas Fleischer <lfleischer@archlinux.org>2016-10-11 22:14:41 +0200
commitfc6dc44295bed8c5d9c8014c2eab2d6bf33d2541 (patch)
tree3fe0ca49397f71530d642dacd246119bbfbbabb7
parent51101d21b972b6afcef299795430cb5aea81e373 (diff)
downloadaur-fc6dc44295bed8c5d9c8014c2eab2d6bf33d2541.tar.gz
aur-fc6dc44295bed8c5d9c8014c2eab2d6bf33d2541.tar.xz
git-serve: Close orphan requests upon disown
When disowning a package base via the SSH interface, auto-accept all pending orphan requests for the affected package. Also, add a test case that checks whether (only) orphan requests belonging to disowned packages are closed correctly. Signed-off-by: Lukas Fleischer <lfleischer@archlinux.org>
-rwxr-xr-xaurweb/git/serve.py44
-rwxr-xr-xtest/t1200-git-serve.sh23
2 files changed, 66 insertions, 1 deletions
diff --git a/aurweb/git/serve.py b/aurweb/git/serve.py
index ebfef946..476aea86 100755
--- a/aurweb/git/serve.py
+++ b/aurweb/git/serve.py
@@ -183,6 +183,44 @@ def pkgbase_set_comaintainers(pkgbase, userlist, user, privileged):
conn.close()
+def pkgreq_by_pkgbase(pkgbase_id, reqtype):
+ conn = aurweb.db.Connection()
+
+ cur = conn.execute("SELECT PackageRequests.ID FROM PackageRequests " +
+ "INNER JOIN RequestTypes ON " +
+ "RequestTypes.ID = PackageRequests.ReqTypeID " +
+ "WHERE PackageRequests.Status = 0 " +
+ "AND PackageRequests.PackageBaseID = ?" +
+ "AND RequestTypes.Name = ?", [pkgbase_id, reqtype])
+
+ return [row[0] for row in cur.fetchall()]
+
+
+def pkgreq_close(reqid, reason, comments, autoclose=False):
+ statusmap = {'accepted': 2, 'rejected': 3}
+ if reason not in statusmap:
+ die('{:s}: invalid reason: {:s}'.format(action, reason))
+ status = statusmap[reason]
+
+ conn = aurweb.db.Connection()
+
+ if autoclose:
+ userid = 0
+ else:
+ cur = conn.execute("SELECT ID FROM Users WHERE Username = ?", [user])
+ userid = cur.fetchone()[0]
+ if userid == 0:
+ die('{:s}: unknown user: {:s}'.format(action, user))
+
+ conn.execute("UPDATE PackageRequests SET Status = ?, ClosureComment = ? " +
+ "WHERE ID = ?", [status, comments, reqid])
+ conn.commit()
+ conn.close()
+
+ subprocess.Popen((notify_cmd, 'request-close', str(userid), str(reqid),
+ reason)).wait()
+
+
def pkgbase_disown(pkgbase, user, privileged):
pkgbase_id = pkgbase_from_name(pkgbase)
if not pkgbase_id:
@@ -193,7 +231,11 @@ def pkgbase_disown(pkgbase, user, privileged):
die('{:s}: permission denied: {:s}'.format(action, user))
# TODO: Support disowning package bases via package request.
- # TODO: Scan through pending orphan requests and close them.
+
+ # Scan through pending orphan requests and close them.
+ comment = 'The user {:s} disowned the package.'.format(user)
+ for reqid in pkgreq_by_pkgbase(pkgbase_id, 'orphan'):
+ pkgreq_close(reqid, 'accepted', comment, True)
comaintainers = []
new_maintainer_userid = None
diff --git a/test/t1200-git-serve.sh b/test/t1200-git-serve.sh
index 2f1926e2..5054ce36 100755
--- a/test/t1200-git-serve.sh
+++ b/test/t1200-git-serve.sh
@@ -317,4 +317,27 @@ test_expect_success "Force-disown a package base and check (co-)maintainer list.
test_cmp expected actual
'
+test_expect_success "Check whether package requests are closed when disowning." '
+ SSH_ORIGINAL_COMMAND="adopt foobar" AUR_USER=user AUR_PRIVILEGED=0 \
+ "$GIT_SERVE" 2>&1 &&
+ cat <<-EOD | sqlite3 aur.db &&
+ INSERT INTO PackageRequests (ID, ReqTypeID, PackageBaseID, PackageBaseName, UsersID) VALUES (1, 2, 3, "foobar", 4);
+ INSERT INTO PackageRequests (ID, ReqTypeID, PackageBaseID, PackageBaseName, UsersID) VALUES (2, 3, 3, "foobar", 5);
+ INSERT INTO PackageRequests (ID, ReqTypeID, PackageBaseID, PackageBaseName, UsersID) VALUES (3, 2, 2, "foobar2", 6);
+ EOD
+ >sendmail.out &&
+ SSH_ORIGINAL_COMMAND="disown foobar" AUR_USER=user AUR_PRIVILEGED=0 \
+ "$GIT_SERVE" 2>&1 &&
+ cat <<-EOD >expected &&
+ Subject: [PRQ#1] Request Accepted
+ EOD
+ grep "^Subject.*PRQ" sendmail.out >sendmail.parts &&
+ test_cmp sendmail.parts expected &&
+ cat <<-EOD >expected &&
+ 1|2|3|foobar||4||The user user disowned the package.|0|2
+ EOD
+ echo "SELECT * FROM PackageRequests WHERE Status = 2;" | sqlite3 aur.db >actual &&
+ test_cmp actual expected
+'
+
test_done