Add record tests. Version 0.19.1

Author: SRJ9

README.md

@@ -30,6 +30,10 @@ Versions
 - 0.19 (Ayrton Senna's Toleman's car number '84 - Debut)
 - 0.27-VIL (Gilles Villeneuve 27)
 
+0.19.1
+======
+- Add records by season
+
 0.19
 ====
 - Spanish translation
@@ -75,7 +79,9 @@ $ # login /admin: admin:pass
 
 Todo
 ====
-- [ ] Add records by season, driver, team, competition
+- [x] Add records by season (driver, team)
+- [ ] Add records by driver/team career
+- [ ] Add records by competition
 - [ ] Add drivers profile with records, last_wins, teams...
 - [x] Add easy clone to Season
 - [x] Translate

README.rst

@@ -31,6 +31,10 @@ Versions
 -  0.19 (Ayrton Senna's Toleman's car number '84 - Debut)
 -  0.27-VIL (Gilles Villeneuve 27)
 
+0.19.1
+======
+-  Records by season
+
 0.19
 ====
 -  Spanish translation
@@ -86,7 +90,9 @@ Demo (virtualenv recommended)
 Todo
 ====
 
--  [ ] Add records by season, driver, team, competition
+-  [x] Add records by season (driver, team)
+-  [ ] Add records by driver/team career
+-  [ ] Add records by competition
 -  [ ] Add drivers profile with records, last\_wins, teams...
 -  [x] Add easy clone to Season
 -  [x] Translate

driver27/tests/test_views.py

@@ -108,6 +108,53 @@ def test_race_view(self):
         # Check that the response is 200 OK.
         self.assertEqual(response.status_code, 404)
 
+    def test_driver_records_view(self):
+        # Issue a GET request.
+        response = self.client.get(reverse('season-driver-record-index',
+                                           kwargs={'competition_slug': 'f1', 'year': 2016}))
+        # Check that the response is 200 OK.
+        self.assertEqual(response.status_code, 200)
+
+        # Issue a GET request.
+        response = self.client.get(reverse('season-driver-record',
+                                           kwargs={'competition_slug': 'f1', 'year': 2016, 'record': 'POLE'}))
+        # Check that the response is 200 OK.
+        self.assertEqual(response.status_code, 200)
+
+        # Issue a GET request.
+        response = self.client.get(reverse('season-driver-record',
+                                           kwargs={'competition_slug': 'f1', 'year': 2016, 'record': 'FFF'}))
+        # Check that the response is 200 OK.
+        self.assertEqual(response.status_code, 404)
+
+    def test_team_records_view(self):
+        # Issue a GET request.
+        response = self.client.get(reverse('season-team-record-index',
+                                           kwargs={'competition_slug': 'f1', 'year': 2016}))
+        # Check that the response is 200 OK.
+        self.assertEqual(response.status_code, 200)
+
+        # Issue a GET request.
+        response = self.client.get(reverse('season-team-record',
+                                           kwargs={'competition_slug': 'f1', 'year': 2016, 'record': 'POLE'}))
+        # Check that the response is 200 OK.
+        self.assertEqual(response.status_code, 200)
+
+        # Issue a GET request.
+        response = self.client.get(reverse('season-team-record-by-race',
+                                           kwargs={'competition_slug': 'f1', 'year': 2016, 'record': 'POLE'}))
+        # Check that the response is 200 OK.
+        self.assertEqual(response.status_code, 200)
+
+        # Issue a GET request.
+        response = self.client.get(reverse('season-team-record',
+                                           kwargs={'competition_slug': 'f1', 'year': 2016, 'record': 'FFF'}))
+        # Check that the response is 200 OK.
+        self.assertEqual(response.status_code, 404)
+
+
+
+
     # def test_admin(self):
     #     response = self.client.get(reverse('admin:driver27_driver_changelist'))
     #     # Check that the response is 200 OK.
@@ -238,62 +285,6 @@ def test_race_with_no_results(self):
         self.assertIsNone(ma.print_winner(race))
         self.assertIsNone(ma.print_fastest(race))
 
-    # def test_result_admin(self):
-    #     ma = RaceAdmin(Race, self.site)
-    #     race = Race.objects.get(pk=21)
-    #     request = self.factory.get(reverse("admin:driver27_race_results", args=[race.pk]))
-    #     self.assertTrue(ma.results(request, race.pk))
-    #     results_post = {
-    #         'entry[]': [1, 2, 3],
-    #         'seat-1-qualifying': 1,
-    #         'seat-1-finish': None,
-    #         'seat-1-wildcard': False,
-    #         'seat-1-fastest-lap': True,
-    #         'seat-1-retired': True,
-    #         'seat-2-qualifying': 2,
-    #         'seat-2-finish': 5,
-    #         'seat-2-wildcard': False,
-    #         'seat-2-fastest-lap': False,
-    #         'seat-2-retired': False,
-    #         'seat-3-qualifying': 20,
-    #         'seat-3-finish': 1,
-    #         'seat-3-wildcard': False,
-    #         'seat-3-fastest-lap': False,
-    #         'seat-3-retired': False,
-    #     }
-    #     request = self.factory.post(reverse("admin:driver27_race_results", args=[race.pk]), data=results_post)
-    #     self.assertTrue(ma.results(request, race.pk))
-    #
-    #     # update and remove results
-    #     results_post = {
-    #         'entry[]': [1, 2, 4],  # 1: equal, 2: update finish, 3: remove, 4: add
-    #         'seat-1-qualifying': 1, 'seat-1-finish': None, 'seat-1-wildcard': False,
-    #         'seat-1-fastest-lap': True, 'seat-1-retired': True,
-    #         'seat-2-qualifying': 2, 'seat-2-finish': 7, 'seat-2-wildcard': False,
-    #         'seat-2-fastest-lap': False, 'seat-2-retired': False,
-    #         'seat-4-qualifying': 3, 'seat-4-finish': 10, 'seat-4-wildcard': False,
-    #         'seat-4-fastest-lap': False, 'seat-4-retired': False,
-    #         # entry 3 removed
-    #     }
-    #     request = self.factory.post(reverse("admin:driver27_race_results", args=[race.pk]), data=results_post)
-    #     self.assertTrue(ma.results(request, race.pk))
-    #
-    #     self.assertEqual(len(request.POST.getlist('entry[]')), 3)
-    #
-    #     result_seat1 = Result.objects.get(seat__pk=1, race=race)
-    #     result_seat2 = Result.objects.get(seat__pk=2, race=race)
-    #     result_seat3 = Result.objects.filter(seat__pk=3, race=race)
-    #     result_seat4 = Result.objects.get(seat__pk=4, race=race)
-    #
-    #     self.assertTrue(result_seat1.fastest_lap)
-    #     self.assertEquals(result_seat2.finish, 7)
-    #     self.assertEquals(result_seat3.count(), 0)
-    #     self.assertEquals(result_seat4.qualifying, 3)
-    #
-    #     # no seats
-    #     request = self.factory.post(reverse("admin:driver27_race_results", args=[race.pk]), data={})
-    #     self.assertTrue(ma.results(request, race.pk))
-
     def test_contender_admin(self):
         ma = ContenderAdmin(Contender, self.site)
         self._check_get_changelist(ma)

driver27/views.py

@@ -147,7 +147,7 @@ def get_record_config(record):
     for record_config in DR27_RECORDS_FILTER:
         if record_config['code'] == record:
             return record_config
-    return None
+    raise Http404(_('Record does not exist'))
 
 
 def get_record_common_context(request, competition_slug, year, record=None):