WIP Add Shadowsocks access method test

Author: MarkusPettersson98

talpid-types/src/net/proxy.rs

@@ -76,6 +76,12 @@ pub struct Shadowsocks {
     pub cipher: String,
 }
 
+impl From<Shadowsocks> for CustomProxy {
+    fn from(value: Shadowsocks) -> Self {
+        CustomProxy::Shadowsocks(value)
+    }
+}
+
 #[derive(Clone, Debug, Serialize, Deserialize, PartialEq, Eq, Hash)]
 pub struct Socks5Local {
     pub remote_endpoint: Endpoint,

test/test-manager/src/tests/access_methods.rs

@@ -21,19 +21,59 @@ use test_rpc::ServiceClient;
 pub async fn test_custom_access_methods(
     _: TestContext,
     _rpc: ServiceClient,
-    _mullvad_client: MullvadProxyClient,
+    mullvad_client: MullvadProxyClient,
 ) -> Result<(), Error> {
     log::info!("Testing Shadowsocks access method");
-    test_shadowsocks().await?;
+    test_shadowsocks(mullvad_client.clone()).await?;
     log::info!("Testing SOCKS5 (Remote) access method");
     test_socks_remote().await?;
     Ok(())
 }
 
-async fn test_shadowsocks() -> Result<(), Error> {
-    panic!("Testing Shadowsocks access method has not been fully implemented yet!")
+async fn test_shadowsocks(mut mullvad_client: MullvadProxyClient) -> Result<(), Error> {
+    use crate::tests::helpers;
+    use mullvad_types::relay_list::RelayEndpointData;
+    use std::net::SocketAddr;
+    use talpid_types::net::proxy::Shadowsocks;
+    // Set up all the parameters needed to create a custom Shadowsocks access method.
+    //
+    // Since Mullvad host's Shadowsocks relays on their bridge servers, we can
+    // simply select a bridge server to derive all the needed parameters.
+    // mullvad_client
+    let bridges = helpers::filter_relays(&mut mullvad_client, |relay| {
+        matches!(relay.endpoint_data, RelayEndpointData::Bridge)
+    })
+    .await?;
+
+    let shadowsocks_relay = bridges
+        .first()
+        .expect("`test_shadowsocks` needs at least one shadowsocks relay to execute. Found non in relay list.");
+
+    let access_method = Shadowsocks {
+        endpoint: SocketAddr::from((
+            shadowsocks_relay.ipv4_addr_in,
+            // TODO(markus): Will this always be valid?
+            443,
+        )),
+        // TODO(markus): Will this always be valid?
+        password: "mullvad".to_string(),
+        // TODO(markus): Will this always be valid?
+        cipher: "aes-256-cfb".to_string(),
+    };
+
+    let successful = mullvad_client
+        .test_custom_api_access_method(access_method.clone().into())
+        .await?;
+
+    assert!(
+        successful,
+        "Failed while testing access method - {access_method:?}"
+    );
+
+    Ok(())
 }
 
+#[allow(clippy::unused_async)]
 async fn test_socks_remote() -> Result<(), Error> {
     unimplemented!("Testing SOCKS5 (Remote) access method is not implemented")
 }