Fix RPM epoch filter to not crash on non-numeric values

This patch also fixes the documentation to properly show the value
should be an int.

JIRA: PDC-997

pdc/apps/common/filters.py

@@ -16,6 +16,7 @@
 import django_filters
 import django.forms.widgets as widgets
 from .models import Label, SigKey
+from .hacks import convert_str_to_int
 
 SelectMultiple = widgets.SelectMultiple
 
@@ -49,6 +50,22 @@ def filter(self, qs, value):
         return qs
 
 
+class MultiIntFilter(MultiValueFilter):
+    """
+    MultiValueFilter that reports error when input is not a number.
+    """
+    @value_is_not_empty
+    def filter(self, qs, value):
+        # This can't actually call to parent method, as double invocation of
+        # @value_is_not_empty would cause the filter with empty value to be
+        # ignored.
+        value = [convert_str_to_int(val, name=self.name) for val in value]
+        qs = qs.filter(**{self.name + '__in': value})
+        if self.distinct:
+            qs = qs.distinct()
+        return qs
+
+
 class ComposeFilterSetOptions(FilterSetOptions):
     def __init__(self, options=None):
         # HACK NOTE:

pdc/apps/common/renderers.py

@@ -176,6 +176,7 @@ def replace_url(match):
     'NullableCharFilter': 'string | null',
     'BooleanFilter': 'bool',
     'ActiveReleasesFilter': 'bool',
+    'MultiIntFilter': 'int',
 }
 LOOKUP_TYPES = {
     'icontains': 'case insensitive, substring match',

pdc/apps/package/filters.py

@@ -8,14 +8,14 @@
 
 import django_filters
 
-from pdc.apps.common.filters import MultiValueFilter, NullableCharFilter
+from pdc.apps.common.filters import MultiValueFilter, MultiIntFilter, NullableCharFilter
 from . import models
 
 
 class RPMFilter(django_filters.FilterSet):
     name        = MultiValueFilter()
     version     = MultiValueFilter()
-    epoch       = MultiValueFilter()
+    epoch       = MultiIntFilter()
     release     = MultiValueFilter()
     arch        = MultiValueFilter()
     srpm_name   = MultiValueFilter()

pdc/apps/package/tests.py

@@ -154,6 +154,11 @@ def test_query_with_wrong_params(self):
         response = self.client.get(url + 'wrong_param/', format='json')
         self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND)
 
+    def test_query_with_bad_epoch(self):
+        url = reverse('rpms-list')
+        response = self.client.get(url, {'epoch': 'foo'}, format='json')
+        self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
+
     def test_query_with_only_key(self):
         url = reverse('rpms-list')
         response = self.client.get(url + '?name', format='json')