700628 - selecting top crashers from the report menu does not load the latest version

Status: VERIFIED FIXED

(Socorro :: Webapp, task)

Description

[Trif Andrei-Alin[:AlinT]]

Attachment: Screenshot of the issue

Steps to reproduce:
1.Go to http://crash-stats.mozilla.com
2.select from the report menu top crashers

Expected results:
Top crashers for FF version 10.0a1 page loads.

Actual results:
Top crashers for FF version 3.5.13 page loads

Comment 1

[Robert Kaiser]

Well, it's basically unspecified what to do in the case where you didn't actually select a version. Right now, it looks like it's taking the bottom one from the selection list of "current" versions. I guess it probably should use the top one instead.

Comment 2

[Trif Andrei-Alin[:AlinT]]

I agree with you, it should take the top of the "current" versions list.
Now it's happening also on dev and stage.
Do you have any idea when this issue will be fixed?

Comment 3

[Stephen Donner [:stephend] Not actively reading bugmail]

Alin, did you find this from our Selenium tests?  If so, let's please add [fromAutomation] to the status whiteboard; nice find!

Comment 4

[Matt Brandt [:mbrandt]]

:stephend, yes this was caught by our automation. Alin has xfailed this test until this bug is resolved.

Comment 5

[Schalk Neethling [:espressive]]

Sent pull request: https://github.com/mozilla/socorro/pull/213

Test Case
---------

Head over to the front page of Socorro: i.e
https://espressive-dev.allizom.org/products/Firefox

Under the version drop down, do not select a version and then select top crashers from the reports drop down. The page should redirect to:
https://espressive-dev.allizom.org/topcrasher/byversion/Firefox/9.0b3

The version string at the end should match the top most version in the version drop down.

To test that other cases still work correctly. Go back to the front page. This time select a version and then select top crashers. This time the version string at the end of the URL should match the version number you selected.

Comment 7

[[github robot]]

Commit pushed to https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/6f20fceceae4038377377bd10f74d53e955f6d0c
Merge pull request #213 from ossreleasefeed/bug700628-select-top-if-noselection

Select top most version if non selected for top crashers :: Fixes bug 700628

Comment 8

[Trif Andrei-Alin[:AlinT]]

QA verified on dev and stage.