Re-phrase mullvad api-access test

If the API call failed, point out which api-access-method-under-test
which did not work.

mullvad-cli/src/cmds/api_access.rs

@@ -166,11 +166,15 @@ impl ApiAccess {
     async fn test(item: SelectItem) -> Result<()> {
         let mut rpc = MullvadProxyClient::new().await?;
         let access_method = Self::get_access_method(&mut rpc, &item).await?;
+        let name = access_method.get_name();
         rpc.set_access_method(access_method).await?;
         // Make the daemon perform an network request which involves talking to the Mullvad API.
         match rpc.get_api_addressess().await {
-            Ok(_) => println!("API call succeeded!"),
-            Err(_) => println!("API call failed :-("),
+            Ok(_) => println!("Connected to the Mullvad API!"),
+            Err(_) => println!(
+                "Could *not* connect to the Mullvad API using access method \"{}\"",
+                name
+            ),
         }
 
         Ok(())

mullvad-types/src/access_method.rs

@@ -84,6 +84,24 @@ pub enum AccessMethod {
     Custom(CustomAccessMethod),
 }
 
+impl AccessMethod {
+    pub fn get_name(&self) -> String {
+        match self {
+            AccessMethod::BuiltIn(x) => match x {
+                BuiltInAccessMethod::Direct(_) => "Direct".to_string(),
+                BuiltInAccessMethod::Bridge(_) => "Mullvad Bridges".to_string(),
+            },
+            AccessMethod::Custom(c) => match &c.access_method {
+                ObfuscationProtocol::Shadowsocks(s) => s.name.clone(),
+                ObfuscationProtocol::Socks5(s) => match s {
+                    Socks5::Local(l) => l.name.clone(),
+                    Socks5::Remote(r) => r.name.clone(),
+                },
+            },
+        }
+    }
+}
+
 /// Built-In access method datastructure.
 #[derive(Clone, Debug, Serialize, Deserialize, PartialEq)]
 pub enum BuiltInAccessMethod {