diff options
Diffstat (limited to 'tests/integration')
| -rw-r--r-- | tests/integration/test_accesslist.py | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/tests/integration/test_accesslist.py b/tests/integration/test_accesslist.py index b4e134e..e963bcc 100644 --- a/tests/integration/test_accesslist.py +++ b/tests/integration/test_accesslist.py | |||
| @@ -37,12 +37,10 @@ class Blacklist(TrackerTestCase): | |||
| 37 | self.assertNotIn(b"failure reason", answer) | 37 | self.assertNotIn(b"failure reason", answer) |
| 38 | self.assertEqual(answer[b"complete"], 1) | 38 | self.assertEqual(answer[b"complete"], 1) |
| 39 | 39 | ||
| 40 | def test_the_block_holds_over_udp_but_the_answer_is_malformed(self): | 40 | def test_the_block_is_an_error_packet_over_udp(self): |
| 41 | """The UDP path has no spelling for "not authorized": the announce | 41 | """BEP 15 has an error action, which is what a refusal should use. |
| 42 | writes no body, so the tracker sends the eight byte header alone | 42 | It used to answer the announce header with no body behind it, a |
| 43 | where BEP 15 calls for at least twenty. Pinned rather than blessed — | 43 | packet too short for any client to read.""" |
| 44 | an error packet or silence would both be defensible, a short announce | ||
| 45 | is neither.""" | ||
| 46 | sock, connection_id = self.tracker.udp_connect() | 44 | sock, connection_id = self.tracker.udp_connect() |
| 47 | sock.close() | 45 | sock.close() |
| 48 | request = struct.pack( | 46 | request = struct.pack( |
