Changeset 0dedfbe in OpenWorkouts-current for ow/tests/views/test_workout.py

Timestamp:

Apr 22, 2019, 10:47:55 PM (5 years ago)

Author:

Borja Lopez <borja@…>

Branches:

current

Children:

a6fa857

Parents:

42baca4

Message:

(#39) Duplicated workouts, fixed broken tests, added more tests coverage

File:

• ow/tests/views/test_workout.py (modified) (4 diffs)

ow/tests/views/test_workout.py

@@ -182 +182 @@
         assert len(response['form'].form.errors) == 0
         assert isinstance(response['form'].form.schema, UploadedWorkoutSchema)
+        assert response['duplicate'] is None
 
     def test_add_workout_post_invalid(self, dummy_request):
@@ -196 +197 @@
         # Only one required field in this case, the tracking file
         assert len(response['form'].form.errors) == 1
+        assert response['duplicate'] is None
 
     def test_add_workout_post_invalid_bytes(self, dummy_request):
@@ -214 +216 @@
         # Only one required field in this case, the tracking file
         assert len(response['form'].form.errors) == 1
+        assert response['duplicate'] is None
 
     @pytest.mark.parametrize('filename', gpx_filenames)
@@ -234 +237 @@
         assert response.location.endswith('/2/')
         assert len(request.root['john'].workouts()) == 2
+        self.close_uploaded_file(uploaded_file)
+
+    def test_add_workout_post_duplicate(self, dummy_request):
+        """
+        POST request, first add a workout uploading one of the sample
+        tracking files, then try to upload it again.
+        """
+        # first, upload the workout
+        filename = self.gpx_filenames[0]
+        request = dummy_request
+        uploaded_file = self.open_uploaded_file(filename)
+        filestorage = self.create_filestorage(uploaded_file)
+        user = request.root['john']
+        request.method = 'POST'
+        request.POST = MultiDict({
+            'tracking_file': filestorage,
+            'submit': True,
+            })
+        assert len(request.root['john'].workouts()) == 1
+        response = workout_views.add_workout(user, request)
+        assert isinstance(response, HTTPFound)
+        assert response.location.endswith('/2/')
+        assert len(request.root['john'].workouts()) == 2
+        self.close_uploaded_file(uploaded_file)
+        # now, try to upload it again
+        uploaded_file = self.open_uploaded_file(filename)
+        filestorage = self.create_filestorage(uploaded_file)
+        user = request.root['john']
+        request.method = 'POST'
+        request.POST = MultiDict({
+            'tracking_file': filestorage,
+            'submit': True,
+            })
+        response = workout_views.add_workout(user, request)
+        assert response['duplicate'] == request.root['john']['2']
+        assert len(request.root['john'].workouts()) == 2
+        self.close_uploaded_file(uploaded_file)
+        # finally, override the duplicate prevention code and save it
+        uploaded_file = self.open_uploaded_file(filename)
+        filestorage = self.create_filestorage(uploaded_file)
+        user = request.root['john']
+        request.method = 'POST'
+        request.POST = MultiDict({
+            'tracking_file': filestorage,
+            'allow_duplicates': 'on',
+            'submit': True,
+            })
+        response = workout_views.add_workout(user, request)
+        assert isinstance(response, HTTPFound)
+        assert response.location.endswith('/3/')
+        assert len(request.root['john'].workouts()) == 3
         self.close_uploaded_file(uploaded_file)