rename iter_req_id properly

Author: agnesnatasya

tests/test_asyncio/test_sentinel_managed_connection.py

@@ -157,7 +157,7 @@ async def test_connects_to_same_conn_object_if_same_id_and_conn_released_replica
     )
 
 
-async def test_connects_to_diff_address_if_noiter_req_id_replica(
+async def test_connects_to_diff_address_if_no_iter_req_id_replica(
     connection_pool_replica_mock: SentinelConnectionPool,
 ) -> None:
     """
@@ -182,7 +182,7 @@ async def test_connects_to_diff_address_if_noiter_req_id_replica(
     )
 
 
-async def test_connects_to_same_address_if_sameiter_req_id_master(
+async def test_connects_to_same_address_if_same_iter_req_id_master(
     connection_pool_master_mock: SentinelConnectionPool,
 ) -> None:
     """
@@ -199,7 +199,7 @@ async def test_connects_to_same_address_if_sameiter_req_id_master(
     )
 
 
-async def test_connects_to_same_conn_object_if_sameiter_req_id_and_released_master(
+async def test_connects_to_same_conn_object_if_same_iter_req_id_and_released_master(
     connection_pool_master_mock: SentinelConnectionPool,
 ) -> None:
     """
@@ -218,7 +218,7 @@ async def test_connects_to_same_conn_object_if_sameiter_req_id_and_released_mast
     )
 
 
-async def test_connects_to_same_address_if_noiter_req_id_master(
+async def test_connects_to_same_address_if_no_iter_req_id_master(
     connection_pool_master_mock: SentinelConnectionPool,
 ) -> None:
     """

tests/test_sentinel_managed_connection.py

@@ -88,15 +88,15 @@ def test_connects_to_same_address_if_same_id_replica(
     connection_pool_replica_mock: SentinelConnectionPool,
 ) -> None:
     """
-    Assert that the connection address is the same if the ``_iter_req_id`` is the same
+    Assert that the connection address is the same if the ``iter_req_id`` is the same
     when we are in replica mode using a
     :py:class:`~redis.sentinel.SentinelConnectionPool`.
     """
     connection_for_req_1 = connection_pool_replica_mock.get_connection(
-        "ANY", _iter_req_id=1
+        "ANY", iter_req_id=1
     )
     assert same_address(
-        connection_pool_replica_mock.get_connection("ANY", _iter_req_id=1),
+        connection_pool_replica_mock.get_connection("ANY", iter_req_id=1),
         connection_for_req_1,
     )
 
@@ -105,18 +105,18 @@ def test_connects_to_same_conn_object_if_same_id_and_conn_released_replica(
     connection_pool_replica_mock: SentinelConnectionPool,
 ) -> None:
     """
-    Assert that the connection object is the same if the ``_iter_req_id`` is the same
+    Assert that the connection object is the same if the ``iter_req_id`` is the same
     when we are in replica mode using a
     :py:class:`~redis.sentinel.SentinelConnectionPool`
     and if we release the connection back to the connection pool before
     trying to connect again.
     """
     connection_for_req_1 = connection_pool_replica_mock.get_connection(
-        "ANY", _iter_req_id=1
+        "ANY", iter_req_id=1
     )
     connection_pool_replica_mock.release(connection_for_req_1)
     assert (
-        connection_pool_replica_mock.get_connection("ANY", _iter_req_id=1)
+        connection_pool_replica_mock.get_connection("ANY", iter_req_id=1)
         == connection_for_req_1
     )
 
@@ -125,12 +125,12 @@ def test_connects_to_diff_address_if_no_iter_req_id_replica(
     connection_pool_replica_mock: SentinelConnectionPool,
 ) -> None:
     """
-    Assert that the connection object is different if no _iter_req_id is supplied.
+    Assert that the connection object is different if no iter_req_id is supplied.
     In reality, they can be the same, but in this case, we're not
     releasing the connection to the pool so they should always be different.
     """
     connection_for_req_1 = connection_pool_replica_mock.get_connection(
-        "ANY", _iter_req_id=1
+        "ANY", iter_req_id=1
     )
     connection_for_random_req = connection_pool_replica_mock.get_connection("ANYY")
     assert not same_address(connection_for_random_req, connection_for_req_1)
@@ -156,7 +156,7 @@ def test_connects_to_same_address_if_same_iter_req_id_master(
         "ANY", _iter_req_id=1
     )
     assert same_address(
-        connection_pool_master_mock.get_connection("ANY", _iter_req_id=1),
+        connection_pool_master_mock.get_connection("ANY", iter_req_id=1),
         connection_for_req_1,
     )
 
@@ -172,10 +172,10 @@ def test_connects_to_same_conn_object_if_same_iter_req_id_and_released_master(
     trying to connect again.
     """
     connection_for_req_1 = connection_pool_master_mock.get_connection(
-        "ANY", _iter_req_id=1
+        "ANY", iter_req_id=1
     )
     assert same_address(
-        connection_pool_master_mock.get_connection("ANY", _iter_req_id=1),
+        connection_pool_master_mock.get_connection("ANY", iter_req_id=1),
         connection_for_req_1,
     )
 
@@ -190,7 +190,7 @@ def test_connects_to_same_address_if_no_iter_req_id_master(
     :py:class:`~redis.sentinel.SentinelConnectionPool`.
     """
     connection_for_req_1 = connection_pool_master_mock.get_connection(
-        "ANY", _iter_req_id=1
+        "ANY", iter_req_id=1
     )
     connection_for_random_req = connection_pool_master_mock.get_connection("ANYY")
     assert same_address(connection_for_random_req, connection_for_req_1)