1 files changed, 7 insertions, 9 deletions

diff --git a/tests/integration/test_accesslist.py b/tests/integration/test_accesslist.py
index b4e134e..41ff654 100644
--- a/tests/integration/test_accesslist.py
+++ b/tests/integration/test_accesslist.py
@@ -37,12 +37,10 @@ class Blacklist(TrackerTestCase):
         self.assertNotIn(b"failure reason", answer)
         self.assertEqual(answer[b"complete"], 1)
 
-    def test_the_block_holds_over_udp_but_the_answer_is_malformed(self):
-        """The UDP path has no spelling for "not authorized": the announce
-        writes no body, so the tracker sends the eight byte header alone
-        where BEP 15 calls for at least twenty. Pinned rather than blessed —
-        an error packet or silence would both be defensible, a short announce
-        is neither."""
+    def test_the_block_is_an_error_packet_over_udp(self):
+        """BEP 15 has an error action, which is what a refusal should use.
+        It used to answer the announce header with no body behind it, a
+        packet too short for any client to read."""
         sock, connection_id = self.tracker.udp_connect()
         sock.close()
         request = struct.pack(
@@ -50,10 +48,10 @@ class Blacklist(TrackerTestCase):
         )
         answer = self.tracker.udp_raw(request)
         self.assertIsNotNone(answer)
-        self.assertEqual(len(answer), 8, "header only, no interval or counts")
-        action, transaction = struct.unpack(">II", answer)
-        self.assertEqual(action, 1, "it still claims to be an announce")
+        action, transaction = struct.unpack(">II", answer[:8])
+        self.assertEqual(action, 3, "action 3 is the error packet")
         self.assertEqual(transaction, 7)
+        self.assertIn(b"not authorized", answer[8:])
 
     def test_a_blocked_hash_is_not_stored_when_it_arrives_over_udp(self):
         sock, connection_id = self.tracker.udp_connect()