513376 - get_site_to_test in cesiumcron doesn't really get sites with more than one user

Status: RESOLVED FIXED

(Webtools Graveyard :: Cesium, defect)

Comment 1

[Jeff Balogh (:jbalogh)]

Attachment: http://github.com/jbalogh/cesium/tree/513376-sites-to-test

This awesome query requires Django 1.1.  I've updated the pip requirements doc, make sure you pip install -r again.

Comment 2

[Ryan Doherty (:rdoherty)]

Comment on attachment 397749 [details]
http://github.com/jbalogh/cesium/tree/513376-sites-to-test

r- for tracking purposes.

manage.py cesiumcron spits out errors. 

 File "/Users/rdoherty/Documents/Mozilla/projects/cesium/autoyslow/spawnff.py", line 16, in run_test
    if len(page_list) == 0:
TypeError: object of type 'generator' has no len()

Comment 3

[Jeff Balogh (:jbalogh)]

Comment on attachment 397749 [details]
http://github.com/jbalogh/cesium/tree/513376-sites-to-test

updated spawnff to handle generators

Comment 4

[Ryan Doherty (:rdoherty)]

Comment on attachment 397749 [details]
http://github.com/jbalogh/cesium/tree/513376-sites-to-test

Looks good!

Comment 5

[Jeff Balogh (:jbalogh)]

Thanks, committed in r50719.