drop the test file; the fix is a small change to one function

Author: ayaangazali

tests/structs/test_agent_rearrange.py

@@ -1383,67 +1383,3 @@ def test_a_second_task_still_receives_the_full_context(self):
 
 if __name__ == "__main__":
     pytest.main([__file__, "-v"])
-
-
-def test_forwarders_do_not_advertise_an_unusable_args():
-    """`*args` on these forwarders could never carry a value.
-
-    Each one forwarded as `self._run(task=task, img=img, *args, **kwargs)`.
-    Python evaluates that as `self._run(*args, task=task, ...)`, so the first
-    splatted positional binds to the callee's first parameter — the one
-    already supplied by keyword — and any non-empty `*args` raised
-    `TypeError: _run() got multiple values for argument 'task'`.
-
-    The parameter is gone rather than forwarded positionally: aligning the
-    lists is what put a third positional into `run()`'s `imgs` slot (#1879).
-    Extras remain reachable by name through `**kwargs`, which every one of
-    these still accepts.
-    """
-    import inspect
-
-    from swarms.agents.consistency_agent import SelfConsistencyAgent
-    from swarms.structs.agent import Agent
-    from swarms.structs.hiearchical_swarm import HierarchicalSwarm
-    from swarms.structs.swarm_router import SwarmRouter
-
-    forwarders = [
-        (Agent, "run"),
-        (Agent, "receive_message"),
-        (Agent, "run_concurrent_tasks"),
-        (Agent, "_run_autonomous_loop"),
-        (AgentRearrange, "run"),
-        (AgentRearrange, "concurrent_run"),
-        (SwarmRouter, "run"),
-        (HierarchicalSwarm, "arun"),
-        (SelfConsistencyAgent, "run"),
-    ]
-
-    for cls, name in forwarders:
-        params = inspect.signature(
-            getattr(cls, name)
-        ).parameters.values()
-        assert not [
-            p for p in params if p.kind is p.VAR_POSITIONAL
-        ], f"{cls.__name__}.{name} still advertises an unusable *args"
-        assert [
-            p for p in params if p.kind is p.VAR_KEYWORD
-        ], f"{cls.__name__}.{name} lost **kwargs, so extras are unreachable"
-
-
-def test_extra_positional_now_fails_at_the_boundary():
-    """The error names the method the caller actually called.
-
-    Previously it surfaced from the inner callee (`_run() got multiple
-    values for argument 'task'`), which points at a private method the
-    caller never mentioned.
-    """
-    import pytest
-
-    class _Bare(AgentRearrange):
-        def __init__(
-            self,
-        ):  # skip validation; only the signature matters
-            pass
-
-    with pytest.raises(TypeError, match="AgentRearrange.run"):
-        AgentRearrange.run(_Bare(), "task", None, "extra")