Remove redundant buttons
This commit is contained in:
1 parent
afa0feeb87
commit
e2d0d31edb
11 files changed
+81
-9
No files matched your search
@@ -155,10 +155,10 @@ pub(crate) fn is_hashed_management_secret_key(secret_key: &str) -> bool {
|
||||
}
|
||||
|
||||
pub(crate) fn validate_routing_strategy(strategy: &str) -> Result<(), String> {
|
||||
if matches!(strategy, "round-robin" | "weighted-round-robin" | "fill-first") {
|
||||
if matches!(strategy, "round-robin" | "fill-first") {
|
||||
return Ok(());
|
||||
}
|
||||
Err("Routing strategy supports only round-robin, weighted-round-robin or fill-first".to_string())
|
||||
Err("Routing strategy supports only round-robin or fill-first".to_string())
|
||||
}
|
||||
|
||||
pub(crate) fn normalize_optional_config_string(
|
||||
|
||||
@@ -1852,7 +1852,7 @@ fn core_config_validates_keys_and_routing_strategy() {
|
||||
assert!(validate_core_api_key("").is_err());
|
||||
assert!(validate_core_api_key("contains space").is_err());
|
||||
assert!(validate_routing_strategy("round-robin").is_ok());
|
||||
assert!(validate_routing_strategy("weighted-round-robin").is_ok());
|
||||
assert!(validate_routing_strategy("weighted-round-robin").is_err());
|
||||
assert!(validate_routing_strategy("fill-first").is_ok());
|
||||
assert!(validate_routing_strategy("random").is_err());
|
||||
}
|
||||
|
||||
@@ -152,7 +152,7 @@ async fn v8_accepts_gui_settings_and_reloads_client_keys() {
|
||||
remark: String::new(),
|
||||
}],
|
||||
proxy_url: "direct".into(),
|
||||
routing_strategy: "weighted-round-robin".into(),
|
||||
routing_strategy: "round-robin".into(),
|
||||
routing_session_affinity: true,
|
||||
routing_session_affinity_ttl: "2h30m".into(),
|
||||
request_retry: 2,
|
||||
@@ -184,7 +184,7 @@ async fn v8_accepts_gui_settings_and_reloads_client_keys() {
|
||||
fs::write(&core.config, &content).unwrap();
|
||||
core.start().await;
|
||||
let view = core.validate_save(&content).await;
|
||||
assert_eq!(view["routing"]["strategy"], "weighted-round-robin");
|
||||
assert_eq!(view["routing"]["strategy"], "round-robin");
|
||||
assert_eq!(view["routing"]["retry"]["request-retry"], 2);
|
||||
assert_eq!(view["routing"]["retry"]["max-retry-credentials"], 3);
|
||||
assert_eq!(view["routing"]["retry"]["max-retry-interval"], 5);
|
||||
|
||||
Reference in new issue
Block a user