Adding tox to manage tests with multiple environments

README.md

@@ -1,4 +1,4 @@
-Hello - this project is looking for a new maintainer. Please open an issue if you'd like to help maintain. 
+Hello - this project is looking for a new maintainer. Please open an issue if you'd like to help maintain.
 
 Most pressing issue is to support modern python/django See [#6](https://github.com/burke-software/django-admin-export/issues/6)
 
@@ -35,6 +35,13 @@ Running tests
 2. ``pip install -e . -r test_requirements.txt``
 3. ``py.test tests``
 
+Or to run tests with all supported environments:
+
+1. Acquire a checkout of the repository
+2. Install tox with ``pip install tox``
+3. ``tox``
+
+
 Security
 --------
 

admin_export/templates/admin_export/export.html

@@ -3,7 +3,8 @@
 
 {% block extrahead %}
     {{ block.super }}
-    <script src="{% static "admin/js/jquery.min.js" %}"></script>
+    <script src="{% static "admin/js/vendor/jquery/jquery.min.js" %}"></script>
+    <script src="{% static "admin/js/jquery.init.js" %}"></script>
     <script type="text/javascript">
         (function ($) {
             window.show_fields = function (event, model_ct, field, path, path_verbose) {
@@ -22,7 +23,7 @@
                     $('.check_field').prop('checked', checked);
                 });
             });
-        }(jQuery));
+        }(django.jQuery));
 
     </script>
 {% endblock %}

admin_export/views.py

@@ -2,7 +2,8 @@
 from django.contrib import admin
 from django.contrib.contenttypes.models import ContentType
 from django.http.response import HttpResponse
-from django.template import loader, Context
+from django.template import Context, Engine
+from django.utils.text import force_text
 from django.views.generic import TemplateView
 import csv
 from report_utils.mixins import GetFieldsMixin, DataExportMixin
@@ -33,14 +34,14 @@
 class ExtDataExportMixin(DataExportMixin):
 
     def list_to_html_response(self, data, title='', header=None):
-        html = loader.get_template_from_string(HTML_TEMPLATE).render(Context(locals()))
+        html = Engine().from_string(HTML_TEMPLATE).render(Context(locals()))
         return HttpResponse(html)
 
     def list_to_csv_response(self, data, title='', header=None):
         resp = HttpResponse(content_type="text/csv; charset=UTF-8")
         cw = csv.writer(resp)
         for row in chain([header] if header else [], data):
-            cw.writerow([unicode(s).encode(resp._charset) for s in row])
+            cw.writerow([force_text(s).encode(resp.charset) for s in row])
         return resp
 
 
@@ -99,7 +100,7 @@ def post(self, request, **kwargs):
             return self.list_to_xlsx_response(data_list, header=fields)
 
     def get(self, request, *args, **kwargs):
-        if request.REQUEST.get("related"):  # Dispatch to the other view
+        if request.GET.get("related", request.POST.get("related")):  # Dispatch to the other view
             return AdminExportRelated.as_view()(request=self.request)
         return super(AdminExport, self).get(request, *args, **kwargs)
 

setup.py

@@ -20,5 +20,5 @@
         'Intended Audience :: System Administrators',
         "License :: OSI Approved :: BSD License",
     ],
-    install_requires=['django-report-utils'],
+    install_requires=['django-report-utils', 'openpyxl'],
 )

tox.ini

@@ -0,0 +1,13 @@
+[tox]
+envlist =
+    {py27}-django1{8,9,10}
+
+[testenv]
+setenv =
+    PYTHONPATH = {toxinidir}:{toxinidir}/admin_export
+commands = py.test tests
+deps =
+    django18: Django>=1.8,<1.9
+    django19: Django>=1.9,<1.10
+    django110: Django>=1.10,<1.11
+    -r{toxinidir}/test_requirements.txt