Thread safe tests and --ignored note

tests/test_with_credential.rs

@@ -2,15 +2,15 @@ use rspotify::client::Spotify;
 use rspotify::oauth2::SpotifyClientCredentials;
 use rspotify::senum::{AlbumType, Country};
 
-use std::sync::Mutex;
+use std::sync::{Arc, Mutex};
 
 use lazy_static::lazy_static;
 
 lazy_static! {
     // Set client_id and client_secret in .env file or
     // export CLIENT_ID="your client_id"
     // export CLIENT_SECRET="secret"
-    static ref CLIENT_CREDENTIAL: Mutex<SpotifyClientCredentials> = Mutex::new(SpotifyClientCredentials::default().build());
+    static ref CLIENT_CREDENTIAL: Arc<Mutex<SpotifyClientCredentials>> = Arc::new(Mutex::new(SpotifyClientCredentials::default().build()));
 }
 
 #[tokio::test]

tests/test_with_oauth.rs

@@ -8,8 +8,8 @@ use rspotify::senum::{Country, RepeatState, SearchType, TimeRange};
 use rspotify::util::get_token;
 
 // Because of all these tests need to poll up the browser, it is impossible to
-// run in a CI envirnment like travis, so I just ignore them all. You could
-// delete the #[ignore] tag, and run it on your local machine.
+// run in a CI environment like travis, so I just ignore them all. The
+// --ignored flag can be used with `cargo test` to manually run them.
 
 #[tokio::test]
 #[ignore]