style: format config tests and fix clippy import

This commit is contained in:
Rasyidan Akbar F. 2026-03-17 08:52:00 +07:00
commit d8ab300748
2 changed files with 33 additions and 21 deletions

View file

@ -30,7 +30,6 @@ pub(crate) fn parse_fundamentals_from_str(
#[cfg(test)] #[cfg(test)]
mod tests { mod tests {
use super::{parse_fundamentals_from_str, parse_quote_from_str}; use super::{parse_fundamentals_from_str, parse_quote_from_str};
use crate::api::types::Period;
#[test] #[test]
fn parses_quote_fixture_json() { fn parses_quote_fixture_json() {

View file

@ -426,38 +426,39 @@ mod tests {
fn validate_rejects_invalid_history_provider() { fn validate_rejects_invalid_history_provider() {
let result = super::validate_config_key_value("general.history_provider", "bogus"); let result = super::validate_config_key_value("general.history_provider", "bogus");
assert!(result.is_err()); assert!(result.is_err());
assert!(result assert!(
.unwrap_err() result
.to_string() .unwrap_err()
.contains("invalid history provider")); .to_string()
.contains("invalid history provider")
);
} }
#[test] #[test]
fn validate_accepts_valid_history_provider_auto() { fn validate_accepts_valid_history_provider_auto() {
assert!( assert!(super::validate_config_key_value("general.history_provider", "auto").is_ok());
super::validate_config_key_value("general.history_provider", "auto").is_ok()
);
} }
#[test] #[test]
fn validate_accepts_valid_history_provider_yahoo() { fn validate_accepts_valid_history_provider_yahoo() {
assert!( assert!(super::validate_config_key_value("general.history_provider", "yahoo").is_ok());
super::validate_config_key_value("general.history_provider", "yahoo").is_ok()
);
} }
#[test] #[test]
fn validate_accepts_valid_history_provider_msn() { fn validate_accepts_valid_history_provider_msn() {
assert!( assert!(super::validate_config_key_value("general.history_provider", "msn").is_ok());
super::validate_config_key_value("general.history_provider", "msn").is_ok()
);
} }
#[test] #[test]
fn validate_rejects_invalid_output() { fn validate_rejects_invalid_output() {
let result = super::validate_config_key_value("general.output", "bogus"); let result = super::validate_config_key_value("general.output", "bogus");
assert!(result.is_err()); assert!(result.is_err());
assert!(result.unwrap_err().to_string().contains("invalid output format")); assert!(
result
.unwrap_err()
.to_string()
.contains("invalid output format")
);
} }
#[test] #[test]
@ -480,17 +481,24 @@ mod tests {
fn validate_rejects_negative_ttl() { fn validate_rejects_negative_ttl() {
let result = super::validate_config_key_value("cache.quote_ttl", "-1"); let result = super::validate_config_key_value("cache.quote_ttl", "-1");
assert!(result.is_err()); assert!(result.is_err());
assert!(result.unwrap_err().to_string().contains("must be non-negative")); assert!(
result
.unwrap_err()
.to_string()
.contains("must be non-negative")
);
} }
#[test] #[test]
fn validate_rejects_non_numeric_ttl() { fn validate_rejects_non_numeric_ttl() {
let result = super::validate_config_key_value("cache.quote_ttl", "not-a-number"); let result = super::validate_config_key_value("cache.quote_ttl", "not-a-number");
assert!(result.is_err()); assert!(result.is_err());
assert!(result assert!(
.unwrap_err() result
.to_string() .unwrap_err()
.contains("must be a non-negative integer")); .to_string()
.contains("must be a non-negative integer")
);
} }
#[test] #[test]
@ -522,7 +530,12 @@ mod tests {
fn validate_rejects_invalid_color() { fn validate_rejects_invalid_color() {
let result = super::validate_config_key_value("general.color", "foo"); let result = super::validate_config_key_value("general.color", "foo");
assert!(result.is_err()); assert!(result.is_err());
assert!(result.unwrap_err().to_string().contains("invalid color value")); assert!(
result
.unwrap_err()
.to_string()
.contains("invalid color value")
);
let result = super::validate_config_key_value("general.color", "123"); let result = super::validate_config_key_value("general.color", "123");
assert!(result.is_err()); assert!(result.is_err());