Merge pull request #228 from asherf/tox

Use literals

Author: asherf

django_prometheus/middleware.py

@@ -35,9 +35,7 @@ def get_instance(cls):
             cls._instance = cls()
         return cls._instance
 
-    def register_metric(
-        self, metric_cls, name, documentation, labelnames=tuple(), **kwargs
-    ):
+    def register_metric(self, metric_cls, name, documentation, labelnames=(), **kwargs):
         return metric_cls(name, documentation, labelnames=labelnames, **kwargs)
 
     def __init__(self, *args, **kwargs):

django_prometheus/tests/end2end/testapp/test_middleware_custom_labels.py

@@ -18,9 +18,7 @@
 
 
 class CustomMetrics(Metrics):
-    def register_metric(
-        self, metric_cls, name, documentation, labelnames=tuple(), **kwargs
-    ):
+    def register_metric(self, metric_cls, name, documentation, labelnames=(), **kwargs):
         if name in EXTENDED_METRICS:
             labelnames.extend(("view_type", "user_agent_type"))
         return super(CustomMetrics, self).register_metric(

django_prometheus/tests/end2end/testapp/test_migrations.py

@@ -26,12 +26,12 @@ def test_counters(self):
         executor = MagicMock()
         executor.migration_plan = MagicMock()
         executor.migration_plan.return_value = set()
-        executor.loader.applied_migrations = set(["a", "b", "c"])
+        executor.loader.applied_migrations = {"a", "b", "c"}
         ExportMigrationsForDatabase("fakedb1", executor)
         self.assertEqual(executor.migration_plan.call_count, 1)
         executor.migration_plan = MagicMock()
-        executor.migration_plan.return_value = set(["a"])
-        executor.loader.applied_migrations = set(["b", "c"])
+        executor.migration_plan.return_value = {"a"}
+        executor.loader.applied_migrations = {"b", "c"}
         ExportMigrationsForDatabase("fakedb2", executor)
 
         self.assertMetricEquals(3, M("applied_total"), connection="fakedb1")