Merge lp:~elachuni/ubuntu-webcatalog/noreviews-last into lp:ubuntu-webcatalog

[Anthony Lenton]

Sent apps with no reviews to the end of all app listings.

This branch sends apps with no reviews to the end of all app listings (department overviews and search results) by assigning wilson_score to all apps before starting to import ratings data.

=== modified file 'src/webcatalog/management/commands/import_ratings_stats.py'
--- src/webcatalog/management/commands/import_ratings_stats.py	2012-06-06 16:38:11 +0000
+++ src/webcatalog/management/commands/import_ratings_stats.py	2012-06-19 13:25:02 +0000
@@ -50,6 +50,9 @@
 
         stats = self.download_review_stats(distroseries)
 
+        Application.objects.filter(distroseries=distroseries).update(
+            wilson_score=-1)
+
         self.update_apps_with_stats(distroseries, stats)
 
         self.update_last_import_timestamp(distroseries)

=== modified file 'src/webcatalog/tests/test_commands.py'
--- src/webcatalog/tests/test_commands.py	2012-06-14 16:18:12 +0000
+++ src/webcatalog/tests/test_commands.py	2012-06-19 13:25:02 +0000
@@ -776,6 +776,20 @@
         result = command.update_apps_with_stats(app.distroseries, stats)
         self.assertIsNone(result)
 
+    def test_apps_with_no_reviews_end_up_with_minus_one(self):
+        # Apps with no reviews should end up with wilson_score = -1
+        natty = self.factory.make_distroseries(code_name='natty')
+        app = self.factory.make_application(wilson_score=3.5,
+                                            distroseries=natty)
+        another_app = self.factory.make_application(wilson_score=3.5)
+
+        call_command('import_ratings_stats', 'natty')
+
+        retrieved = Application.objects.get(pk=app.id)
+        self.assertEqual(-1, retrieved.wilson_score)
+        retrieved = Application.objects.get(pk=another_app.id)
+        self.assertEqual(3.5, retrieved.wilson_score)
+        
 
 class ImportExhibitsTestCase(TestCaseWithFactory):