From 4e8a5f4da857705406dba123bb5818f2951b7953 Mon Sep 17 00:00:00 2001 From: Tim Abbott Date: Tue, 13 Apr 2021 10:47:35 -0700 Subject: [PATCH] test_signup: Fix test making DNS query to public Internet. This appears to be the result of our having copied parts of the previous test without copying its DNS query decorator. --- zerver/tests/test_signup.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/zerver/tests/test_signup.py b/zerver/tests/test_signup.py index 4c37066ce7..ae6b69542a 100644 --- a/zerver/tests/test_signup.py +++ b/zerver/tests/test_signup.py @@ -4607,7 +4607,11 @@ class UserSignUpTest(InviteUserBase): self.assertEqual(result.status_code, 302) self.assert_logged_in_user_id(user_profile.id) - def test_registration_of_active_mirror_dummy_user(self) -> None: + @patch( + "DNS.dnslookup", + return_value=[["sipbtest:*:20922:101:Fred Sipb,,,:/mit/sipbtest:/bin/athena/tcsh"]], + ) + def test_registration_of_active_mirror_dummy_user(self, ignored: Any) -> None: """ Trying to activate an already-active mirror dummy user should raise an AssertionError.