tests: add test for configuration with deprecated env var name

josecelano · josecelano · commit 43942ce2b0fc · 2024-05-09T13:26:46.000+01:00
Until it's updated in this repor and the Index repo you can overwrite
the admin token using the new env var name and the old one (deprecated):

- New: `TORRUST_TRACKER__HTTP_API__ACCESS_TOKENS__ADMIN`
- Old (deprecated): `TORRUST_TRACKER_API_ADMIN_TOKEN`

THe new one uses exactly the strcuture and atribute  names in the toml
file, using `__` as the separator for levels.

We can remove the test when we remove the deprecated name.
diff --git a/packages/configuration/src/v1/mod.rs b/packages/configuration/src/v1/mod.rs
@@ -540,7 +540,7 @@ mod tests {
     }
 
     #[test]
-    fn configuration_should_allow_to_overwrite_the_default_tracker_api_token_for_admin() {
+    fn configuration_should_allow_to_overwrite_the_default_tracker_api_token_for_admin_with_env_var() {
         figment::Jail::expect_with(|jail| {
             jail.set_env("TORRUST_TRACKER__HTTP_API__ACCESS_TOKENS__ADMIN", "NewToken");
 
@@ -559,4 +559,23 @@ mod tests {
             Ok(())
         });
     }
+
+    #[test]
+    fn configuration_should_allow_to_overwrite_the_default_tracker_api_token_for_admin_with_the_deprecated_env_var_name() {
+        figment::Jail::expect_with(|_jail| {
+            let info = Info {
+                tracker_toml: default_config_toml(),
+                api_admin_token: Some("NewToken".to_owned()),
+            };
+
+            let configuration = Configuration::load(&info).expect("Could not load configuration from file");
+
+            assert_eq!(
+                configuration.http_api.access_tokens.get("admin"),
+                Some("NewToken".to_owned()).as_ref()
+            );
+
+            Ok(())
+        });
+    }
 }
