Merge pull request #47 from johncmacy/patch-2

Test if `signal` has attribute `SIGQUIT`

Author: j4mie

README.md

@@ -243,6 +243,10 @@ jobs in the "NEW" or "READY" states will be returned.
 
 It may be necessary to supply a DATABASE_PORT environment variable.
 
+## Windows support
+
+Windows is supported on a best-effort basis only, and is not covered by automated or manual testing.
+
 ## Code of conduct
 
 For guidelines regarding the code of conduct when contributing to this repository please review [https://www.dabapps.com/open-source/code-of-conduct/](https://www.dabapps.com/open-source/code-of-conduct/)

django_dbq/management/commands/worker.py

@@ -84,7 +84,11 @@ def __init__(self, name, rate_limit_in_seconds):
 
     def init_signals(self):
         signal.signal(signal.SIGINT, self.shutdown)
-        signal.signal(signal.SIGQUIT, self.shutdown)
+
+        # for Windows, which doesn't support the SIGQUIT signal
+        if hasattr(signal, "SIGQUIT"):
+            signal.signal(signal.SIGQUIT, self.shutdown)
+
         signal.signal(signal.SIGTERM, self.shutdown)
 
     def shutdown(self, signum, frame):

django_dbq/tests.py

@@ -103,7 +103,9 @@ def test_queue_depth_multiple_queues(self):
         )
 
         stdout = StringIO()
-        call_command("queue_depth", queue_name=("default", "testqueue",), stdout=stdout)
+        call_command(
+            "queue_depth", queue_name=("default", "testqueue",), stdout=stdout,
+        )
         output = stdout.getvalue()
         self.assertEqual(output.strip(), "event=queue_depths default=2 testqueue=2")