Branch version numbers are ordered randomly. And not to mention that all the 7.x tests are green, but they are showing as yellow.

aa

Comments

hass’s picture

Issue summary: View changes
tr’s picture

The 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.

jthorson’s picture

Corresponding 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...

tvn’s picture

Issue tags: +Drupal.org 7.1
m1r1k’s picture

jthorson’s picture

This 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.

jthorson’s picture

StatusFileSize
new2.79 KB

https://drupal.org/node/2150865 got re-closed, so here's the project_release patch that this relies on.

dww’s picture

jthorson’s picture

StatusFileSize
new5.26 KB

Combined 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.

jthorson’s picture

Status: Needs review » Fixed
StatusFileSize
new5.26 KB

Fixed 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.

Status: Fixed » Closed (fixed)

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