Changeset d517001 in OpenWorkouts-current for ow/tests

Timestamp:

Feb 15, 2019, 6:09:04 PM (5 years ago)

Author:

Borja Lopez <borja@…>

Branches:

current, feature/docs, master

Children:

d5429c5

Parents:

d459ee2

Message:

(#58) Set a title automatically when adding manually a workout without
providing one.

The title is generated based on the only required data we have (starting
date and time) + sport (if provided).

Location:

ow/tests

Files:

• test_utilities.py (modified) (2 diffs)
• views/test_user.py (modified) (1 diff)
• views/test_workout.py (modified) (2 diffs)

ow/tests/test_utilities.py

@@ -25 +25 @@
     save_map_screenshot,
     timedelta_to_hms,
-    get_week_days
+    get_week_days,
+    part_of_day
 )
 
@@ -235 +236 @@
             assert m[0] == m[1]
 
+    def test_part_of_day(self):
+        parts = [
+            ((5, 11), 'Morning'),
+            ((12, 17), 'Afternoon'),
+            ((18, 22), 'Evening'),
+            ((23, 4), 'Night'),
+        ]
+        for part in parts:
+            hours_range = range(*part[0])
+            expected = part[1]
+            for hour in hours_range:
+                dt = datetime(2019, 1, 15, hour, 0)
+                assert part_of_day(dt) == expected
+
 
 class TestGPXParseMinidom(object):

ow/tests/views/test_user.py

@@ -447 +447 @@
         assert original_image.size == returned_image.size
 
-
     def test_edit_profile_get(self, dummy_request, john):
         """

ow/tests/views/test_workout.py

@@ -126 +126 @@
         assert len(response['form'].form.errors) == 6
 
-    def test_add_workout_manually_post_valid(self, valid_post_request):
+    add_workout_params = [
+        # no title, no sport, we generate a title based on when the
+        # workout started
+        ({'title': None, 'sport': None}, 'Morning workout'),
+        # no title, sport given, we use the sport too in the automatically
+        # generated title
+        ({'title': None, 'sport': 'cycling'}, 'Morning cycling workout'),
+        # title given, no sport, we use the provided title
+        ({'title': 'Example workout', 'sport': None}, 'Example workout'),
+        # title given, sport too, we use the provided title
+        ({'title': 'Example workout', 'sport': 'cycling'}, 'Example workout'),
+    ]
+
+    @pytest.mark.parametrize(('params', 'expected'), add_workout_params)
+    def test_add_workout_manually_post_valid(self, params, expected,
+                                             valid_post_request):
         """
         POST request to add a workout manually, providing the needed data
         """
         request = valid_post_request
+        if params['title'] is not None:
+            request.POST['title'] = params['title']
+        if params['sport'] is not None:
+            request.POST['sport'] = params['sport']
         user = request.root['john']
         assert len(user.workouts()) == 1
@@ -137 +156 @@
         assert response.location.endswith('/2/')
         assert len(user.workouts()) == 2
+        assert user['2'].title == expected
 
     def test_add_workout_get(self, dummy_request):