fix: remove --connection argument to avoid conflict with testinfra

As testinfra is already using `--connection` argument, we will only
recognize `--ansible-connection` variant from now on, so we do not
endup with conflicts.

Author: ssbarnea

src/pytest_ansible/plugin.py

@@ -144,7 +144,6 @@ def pytest_addoption(parser: pytest.Parser) -> None:
         help="further limit selected hosts to an additional pattern",
     )
     group.addoption(
-        "--connection",
         "--ansible-connection",
         action="store",
         dest="ansible_connection",

tests/test_params.py

@@ -45,7 +45,7 @@ def test_plugin_help(pytester):  # type: ignore[no-untyped-def]  # noqa: ANN001,
             # Check for the specific args
             "  --inventory*, --ansible-inventory=ANSIBLE_INVENTORY",
             "  --host-pattern*, --ansible-host-pattern=ANSIBLE_HOST_PATTERN",
-            "  --connection*, --ansible-connection=ANSIBLE_CONNECTION",
+            "  --ansible-connection=ANSIBLE_CONNECTION",
             "  --user*, --ansible-user=ANSIBLE_USER",
             "  --check, --ansible-check",
             "  --module-path*, --ansible-module-path=ANSIBLE_MODULE_PATH",
@@ -129,7 +129,6 @@ def test_func({fixture_name}):
         "--ansible-inventory",
         "--inventory",
         "--ansible-connection",
-        "--connection",
         "--ansible-user",
         "--user",
         "--ansible-become-method",