Future date for date_start and nothing for date_end would should not …

…result in assertionerror. Bug 639863.

apps/search/forms.py

@@ -101,6 +101,10 @@ def __init__(self, *args, **kwargs):
     def clean(self):
         cleaned = self.cleaned_data
 
+        # default date_end to today
+        if not self.cleaned_data.get('date_end'):
+            self.cleaned_data['date_end'] = date.today() + timedelta(days=1)
+
         # Flip start and end if necessary.
         if (cleaned.get('date_start') and cleaned.get('date_end') and
             cleaned['date_start'] > cleaned['date_end']):

apps/search/tests/test_views.py

@@ -344,3 +344,14 @@ def test_error(self, get_results):
         get_results.side_effect = SearchError()
         r = self.get_request()
         eq_(r.status_code, 500)
+
+    def test_date_flip_future_start(self):
+        """
+        With date_start in the future, and date_end empty, ensure we do something
+        useful (default to end=today and flip the dates).
+        """
+        dates = dict(
+            date_start=datetime.date.today() + datetime.timedelta(days=10),
+            date_end='')
+        r = self.get_request(**dates)
+        eq_(r.status_code, 200)