I just did a manual branch test of role_change notify (it worked). Visiting admin/pifr it seemed to understand that it had been successful.

However, when following it up with a patch test (*not* checking the "interrupt test in progress"), I get this:

    Notice: Undefined index: interrupt in pifr_client_review_form_validate() (line 171 of /home/rfay/workspace/testbot/sites/all/modules/project_issue_file_review/client/pifr_client.review.inc).
    Unable to test due to an existing test in progress (t: 147784).

Comments

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented

BTW, the UI would be better if the "Interrupt tests in progress" were on the second page, so you could click that if you knew it was the right thing to do. As it is you have to back out.

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented

I just did the exact same thing on scratchtestbot, and had no trouble.

What could be the difference? (Both have current/same git checkout) Both have debug enabled.

I guess maybe this isn't so terribly important?

jthorson’s picture

jthorson commented

Interrupt had to be on the first page for something in the form processing ... but I can't for the life of me remember what it was.

I haven't seen this, but will keep an eye out for it.

jthorson’s picture

jthorson commented
Status: Active » Closed (cannot reproduce)