expanding tests a bit

rowers/tests/test_aworkouts.py

@@ -117,10 +117,23 @@ class ListWorkoutTest(TestCase):
 
       form_data = {
          'workouts': ['1','2'],
-         'action': ['remove'],
+         'action': ['export'],
+         'destination': 'strava',
       }
 
       response = self.c.post(expected_url, form_data, follow=True)
+      url = expected_url
+      expected_url = reverse('workouts_view')
+      self.assertRedirects(response,
+                           expected_url=expected_url,
+                           status_code=302,
+                           target_status_code=200)
+      form_data = {
+         'workouts': ['1','2'],
+         'action': ['remove'],
+      }
+
+      response = self.c.post(url, form_data, follow=True)
       expected_url = reverse('workouts_view')
       self.assertRedirects(response,
                            expected_url=expected_url,

rowers/views/workoutviews.py

@@ -2048,9 +2048,9 @@ def workouts_bulk_actions(request):
             w = get_workout_by_opaqueid(request, encid)
             if w.user == r:
                 workouts.append(w)
-            else:
+            else: # pragma: no cover
                 messages.error(request,'Bulk actions are not accessible to coaches')
-    except KeyError:
+    except KeyError: # pragma: no cover
         pass
 
     if request.method == 'POST':
@@ -2058,7 +2058,7 @@ def workouts_bulk_actions(request):
         form = WorkoutMultipleCompareForm(request.POST)
         if form.is_valid() and actionform.is_valid():
             workouts = form.cleaned_data['workouts']
-            if len(workouts) == 0:
+            if len(workouts) == 0: # pragma: no cover
                 url = reverse('workouts_view')
                 return HttpResponseRedirect(url)
             action = actionform.cleaned_data['action']
@@ -2078,7 +2078,7 @@ def workouts_bulk_actions(request):
                                           'Workout {id} exported to {destination}'.format(
                                               id=encoder.encode_hex(w.id),
                                               destination=destination))
-                        except NoTokenError:
+                        except NoTokenError: # pragma: no cover
                             messages.error(request,
                                            'Export to {destination} of workout {id} failed'.format(
                                                id=encoder.encode_hex(w.id),