Duplicate results can crash the site

VERIFIED FIXED

Status

VERIFIED FIXED
7 years ago
4 years ago

People

(Reporter: carljm, Unassigned)

Tracking

Details

(Reporter)

Description

7 years ago
If somehow a duplicate result gets created for the same tester, environment, and included-test-case, that currently results in a 500 error. We should be robust against this.
A Pivotal Tracker story has been created for this Bug: http://www.pivotaltracker.com/story/show/27121339
Carl Meyer added a comment in Pivotal Tracker:   
   
I think these dupe results are being created by the dupe-included-caseversions cleanup code. That code should be fixed to not create dupe results (probably by just discarding any that aren't started yet if possible).
Cameron Dawson changed story state to started in Pivotal Tracker
Cameron Dawson added a comment in Pivotal Tracker:   
   
hey, I just was thinking that we'd love to get this onto staging tomorrow asap (because it blocks rebecca running the test run).  So would you merge this fix into 0.9?  Is that too hard to do now?
Carl Meyer added a comment in Pivotal Tracker:   
   
Here's the fix commit: https://github.com/mozilla/caseconductor/commit/613aab9d492ee28f9feb43b7b3c880afdc1ac849
Carl Meyer changed story state to delivered in Pivotal Tracker
Cameron Dawson added a comment in Pivotal Tracker:   
   
makes sense.  thanks!!
Cameron Dawson changed story state to accepted in Pivotal Tracker

Updated

7 years ago
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Bumping to verified as [qa-] due to age of bug, shipped.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.