fix: correct precendence at client connect

client/gefyra/api/connect.py

@@ -97,7 +97,7 @@ def connect(  # noqa: C901
             cargo_endpoint_host=gclient_conf.gefyra_server.split(":")[0],
             cargo_endpoint_port=gclient_conf.gefyra_server.split(":")[1],
             cargo_container_name=f"gefyra-cargo-{connection_name}",
-            wireguard_mtu=gclient_conf.wireguard_mtu or (str(mtu) if mtu else None),
+            wireguard_mtu=(str(mtu) if mtu else None) or gclient_conf.wireguard_mtu,
         )
 
         gclient = handle_get_gefyraclient(config, gclient_conf.client_id)

client/tests/e2e/base.py

@@ -1135,3 +1135,37 @@ def test_util_for_pod_not_found(self):
                 config=default_configuration, name="foo", namespace="default"
             )
         self.assertIn("Pod foo not found.", str(rte.exception))
+
+    def test_v_client_connect_params_precedence(self):
+        runner = CliRunner()
+        self.gefyra_up()
+        self.assert_gefyra_client_state(
+            client_id=CONNECTION_NAME, state=GefyraClientState.ACTIVE
+        )
+
+        c_file = write_client_file(
+            client_id=CONNECTION_NAME,
+            host="127.0.0.1",
+        )
+
+        file_loc = os.path.join(
+            get_gefyra_config_location(),
+            f"{CONNECTION_NAME}_client.json",
+        )
+        fh = open(file_loc, "w+")
+        fh.write(c_file)
+        fh.seek(0)
+        fh.close()
+        sleep(10)
+        res = runner.invoke(
+            cli,
+            ["connections", "connect", "-n", CONNECTION_NAME, "--mtu", "1200"],
+            catch_exceptions=False,
+        )
+        self.assertEqual(res.exit_code, 0)
+
+        clients = list_client()
+
+        applied_config = clients[0].get_client_config(gefyra_server="127.0.0.1:31820")
+
+        self.assertEqual(applied_config.WIREGUARD_MTU, "1200")