Replacing Fixtures with Constants

Author: dakshbhayana11811137

tests/unit/email/test_init.py

@@ -74,7 +74,7 @@ def test_compute_recipient(user, address, expected):
         (None, None, "127.0.0.1", True),
     ],
 )
-def test_redact_ip(unauthenticated_userid, user, expected):
+def test_redact_ip(unauthenticated_userid, user, remote_addr, expected):
     user_email = pretend.stub(user_id="the_users_id")
 
     request = pretend.stub(
@@ -85,7 +85,7 @@ def test_redact_ip(unauthenticated_userid, user, expected):
                 filter=lambda a: pretend.stub(one=lambda: user_email)
             )
         ),
-        remote_addr=REMOTE_ADDR,
+        remote_addr=remote_addr,
     )
     assert email._redact_ip(request, user_email) == expected
 

tests/unit/forklift/test_legacy.py

@@ -4398,6 +4398,7 @@ def test_upload_new_project_fails_ratelimited(
         db_request,
         project_service,
         failing_limiter,
+        remote_addr,
     ):
         user = UserFactory.create()
         EmailFactory.create(user=user)
@@ -4420,7 +4421,7 @@ def test_upload_new_project_fails_ratelimited(
                 ),
             }
         )
-        db_request.remote_addr = REMOTE_ADDR
+        db_request.remote_addr = remote_addr
 
         project_service.ratelimiters[failing_limiter] = pretend.stub(
             test=lambda *a, **kw: False,