quick fix, drupal keeps result in "admin/config/development/testing/results/%"

message to novice tester/reviewer:
1. enabled simpletest.
2. disabled "clear result after..." (admin/config/development/testing/settings)
3. run a test
4. check the result page and reload page few time.

CommentFileSizeAuthor
simpletest_result_url.patch1.57 KBdroplet
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cachee’s picture

cachee CreditAttribution: cachee commented

I performed the following to test this patch:

1. Loaded Drupal 8.x
2. Viewed the patch
3. Went to the Extend page and enabled "Testing"
4. Went to the configuration page and disabled "clear results after..." and noted the message
message before patch
5. Click on the "List" tab and selected "Test" box to enable a test of all the available modules
6. Noted the url for the results page did not match the message
7. Applied the patch
8. Returned to the Testing configuration page and noted the change in the message
message after patch
9. The patch worked as expected

droplet’s picture

droplet CreditAttribution: droplet commented
Issue tags: -Quick fix

@cachee,

Great!, very detail review.

cirage’s picture

cirage CreditAttribution: cirage commented

Patch works as advertised

Dragan Eror’s picture

Dragan Eror CreditAttribution: Dragan Eror commented
Status: Needs review » Reviewed & tested by the community

Works perfect!

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Thanks! Committed/pushed to 8.x.

Automatically closed -- issue fixed for 2 weeks with no activity.

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein commented
Issue summary: View changes

Looks like this affects Drupal 7 but was never backported. (It's been independently proposed for Drupal 7 more recently at #2401743: Wrong link to test results in settings form so the backport can now happen there.)