Remove var lookup tests

Author: FooBarWidget

Cargo.toml

@@ -97,7 +97,6 @@ assert_cmd = "=2.0.4"
 cc = "1.0"
 chrono = "0.4"
 itertools = "0.10"
-lazy_static = "1.4"
 predicates = "=2.1.1"
 thirtyfour_sync = "0.27"
 once_cell = "1.9"

src/azure/credentials.rs

@@ -148,92 +148,6 @@ fn substr(text: &str, to_skip: usize) -> &str {
 #[cfg(test)]
 mod test {
     use super::*;
-    use lazy_static::lazy_static;
-    use std::ffi::{OsStr, OsString};
-    use std::sync::{Mutex, MutexGuard};
-
-    lazy_static! {
-        static ref ENV_TEST_MUTEX: Mutex<()> = Mutex::new(());
-    }
-
-    struct TempCredentialsEnv {
-        _guard: MutexGuard<'static, ()>,
-        prev_connection_string: Option<OsString>,
-        prev_container_name: Option<OsString>,
-    }
-
-    impl TempCredentialsEnv {
-        pub fn new(
-            connection_string: Option<String>,
-            container_name: Option<String>,
-        ) -> TempCredentialsEnv {
-            let result = TempCredentialsEnv {
-                _guard: ENV_TEST_MUTEX.lock().unwrap(),
-                prev_connection_string: var_os("SCCACHE_AZURE_CONNECTION_STRING"),
-                prev_container_name: var_os("SCCACHE_AZURE_BLOB_CONTAINER"),
-            };
-            set_var_os_option("SCCACHE_AZURE_CONNECTION_STRING", connection_string);
-            set_var_os_option("SCCACHE_AZURE_BLOB_CONTAINER", container_name);
-            result
-        }
-    }
-
-    impl Drop for TempCredentialsEnv {
-        fn drop(&mut self) {
-            set_var_os_option(
-                "SCCACHE_AZURE_CONNECTION_STRING",
-                self.prev_connection_string.as_ref(),
-            );
-            set_var_os_option(
-                "SCCACHE_AZURE_BLOB_CONTAINER",
-                self.prev_container_name.as_ref(),
-            );
-        }
-    }
-
-    fn set_var_os_option(key: impl AsRef<OsStr>, val: Option<impl AsRef<OsStr>>) {
-        if let Some(ref val) = val {
-            set_var(key, val);
-        } else {
-            remove_var(key);
-        }
-    }
-
-    #[test]
-    fn test_credentials_from_environment() {
-        let _temp_env = TempCredentialsEnv::new(Some("DefaultEndpointsProtocol=http;AccountName=devstoreaccount1;BlobEndpoint=http://127.0.0.1:10000/devstoreaccount1;".to_owned()), Some("container".to_owned()));
-
-        let creds = credentials_from_environment().unwrap();
-        assert_eq!(
-            "http://127.0.0.1:10000/devstoreaccount1/",
-            creds.azure_blob_endpoint()
-        );
-        assert_eq!("devstoreaccount1", creds.azure_account_name());
-        assert!(creds.azure_account_key().is_none());
-        assert_eq!("container", creds.blob_container_name());
-    }
-
-    #[test]
-    fn test_credentials_from_environment_no_conn_str() {
-        let _temp_env = TempCredentialsEnv::new(None, Some("container".to_owned()));
-
-        let err = credentials_from_environment().unwrap_err();
-        assert_eq!(
-            "No SCCACHE_AZURE_CONNECTION_STRING in environment",
-            err.to_string()
-        );
-    }
-
-    #[test]
-    fn test_credentials_from_environment_no_container_name() {
-        let _temp_env = TempCredentialsEnv::new(Some("DefaultEndpointsProtocol=http;AccountName=devstoreaccount1;BlobEndpoint=http://127.0.0.1:10000/devstoreaccount1;".to_owned()), None);
-
-        let err = credentials_from_environment().unwrap_err();
-        assert_eq!(
-            "No SCCACHE_AZURE_BLOB_CONTAINER in environment",
-            err.to_string()
-        );
-    }
 
     #[test]
     fn test_parse_connection_string() {