Add a "cancel" button on patch files to cancel testing [#2194493]

This probably belongs in https://drupal.org/project/project_issue_file_test but I think we're being asked to file issues here, and I don't see it here or there...

So... Here's the idea:

When you upload a patch to an issue whose project has testing enabled, it is automatically queued for testing. Then if soon thereafter, someone (or you again) uploads another patch because they forgot a comma or something, it would be really cool if there was an easy way to cancel the in-progress test.

I realize that someone with admin privs on qa.drupal.org is available, they can cancel the test from there, but I think it would be nice if along with the "View" and "Retest" links you can see on the issue page on an uploaded patch file, we also had a "Cancel" link that would cancel the test in progress.

Comments

Comment #1

sun

German

Karlsruhe

CreditAttribution: sun commented 12 February 2014 at 14:23

If possible, I'd recommend to limit the availability of this button to the author of the comment to which the patch was attached.

Comment #2

jhodgdon

she/her

English

CreditAttribution: jhodgdon commented 12 February 2014 at 14:29

I disagree with limiting it to the comment author. Just today I was in an issue and someone had uploaded a patch to remove "a" when he should have removed "the" from a sentence that said "... a the ...". I uploaded a new patch and wanted to cancel the test of the previous patch, which I didn't author.

So I don't really see a reason why anyone shouldn't be able to cancel a test. If we need the test to run, someone can still come along and hit retest.