Closed (fixed)
Project:
[Archive] Drupal.org D7 upgrade QA
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
2 Nov 2013 at 16:16 UTC
Updated:
27 Dec 2013 at 03:20 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #1
hass commentedComment #2
tr commentedThe test results color has always been a problem, ever since automated testing was first added to individual projects several years ago. Many times the test shows "Sent" with a yellow color even though the test complete and ran green long ago. Perhaps a caching issue, but that part isn't related to the upgrade.
I haven't personally seen the order problem, but all my projects have at most one branch per major version. These are always shown listed in order: 6.x-1.x, 7.x-1.x, and 8.x-1.x for example.
Comment #3
jthorson commentedCorresponding code for generating the table (and thus the code needing a patch to properly order the results) can be found at http://drupalcode.org/project/project_issue_file_test.git/blob/refs/head...
Comment #4
tvn commentedComment #5
m1r1k commentedSubissue for Project issue file test
Comment #6
jthorson commentedThis needs a couple of patches to fully resolve:
1) For project_release, the updated project_release_query_release() code is sorting by 'major' first, not 'core release'. So we end up with '6.x-1.x', '8.x-1.x', '7.x-1.x' ordering. The patch at https://drupal.org/comment/8254223#comment-8254223 fixes that.
2) Then, the attached PIFT patch leverages the project_release_query_release() ordering.
Comment #7
jthorson commentedhttps://drupal.org/node/2150865 got re-closed, so here's the project_release patch that this relies on.
Comment #8
dwwRe: #7 -- let's address that at #2154085: Enhance project_release_query_releases() to also sort by API compatibility terms not here.
Thanks,
-Derek
Comment #9
jthorson commentedCombined patch from this issue (portion which adds releases to the tab) and m1r1k's patch from #2152457: Sort Branch Test Results on "Automated Testing" tab.
Comment #10
jthorson commentedFixed the function naming, reversed the sorting order, and committed PIFT in http://drupalcode.org/project/project_issue_file_test.git/commit/5829aa6... and http://drupalcode.org/project/project_issue_file_test.git/commit/36b2005...
Will merge to bzr and deploy.