fix: Resolve unused import and variable warnings
This commit is contained in:
parent
5759fdcd5b
commit
82f816ddcd
2 changed files with 9 additions and 6 deletions
|
|
@ -217,8 +217,8 @@ mod tests {
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
fn test_client_creation() {
|
fn test_client_creation() {
|
||||||
let client = BotServerClient::new(Some("http://localhost:8080".to_string()));
|
let client = BotServerClient::new(Some("http://localhost:9000".to_string()));
|
||||||
assert_eq!(client.base_url(), "http://localhost:8080");
|
assert_eq!(client.base_url(), "http://localhost:9000");
|
||||||
}
|
}
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
|
|
|
||||||
|
|
@ -1,6 +1,8 @@
|
||||||
use crate::error::{BotError, BotResult};
|
use crate::error::{BotError, BotResult};
|
||||||
use std::collections::HashMap;
|
use std::collections::HashMap;
|
||||||
|
#[cfg(not(feature = "i18n"))]
|
||||||
use std::fs;
|
use std::fs;
|
||||||
|
#[cfg(not(feature = "i18n"))]
|
||||||
use std::path::Path;
|
use std::path::Path;
|
||||||
|
|
||||||
#[cfg(feature = "i18n")]
|
#[cfg(feature = "i18n")]
|
||||||
|
|
@ -89,6 +91,7 @@ struct LocaleBundle {
|
||||||
}
|
}
|
||||||
|
|
||||||
impl LocaleBundle {
|
impl LocaleBundle {
|
||||||
|
#[cfg(not(feature = "i18n"))]
|
||||||
fn load(locale_dir: &Path) -> BotResult<Self> {
|
fn load(locale_dir: &Path) -> BotResult<Self> {
|
||||||
let dir_name = locale_dir
|
let dir_name = locale_dir
|
||||||
.file_name()
|
.file_name()
|
||||||
|
|
@ -172,7 +175,7 @@ pub struct I18nBundle {
|
||||||
}
|
}
|
||||||
|
|
||||||
impl I18nBundle {
|
impl I18nBundle {
|
||||||
pub fn load(base_path: &str) -> BotResult<Self> {
|
pub fn load(_base_path: &str) -> BotResult<Self> {
|
||||||
// When i18n feature is enabled, locales are ALWAYS embedded via rust-embed
|
// When i18n feature is enabled, locales are ALWAYS embedded via rust-embed
|
||||||
// Filesystem loading is deprecated - use embedded assets only
|
// Filesystem loading is deprecated - use embedded assets only
|
||||||
#[cfg(feature = "i18n")]
|
#[cfg(feature = "i18n")]
|
||||||
|
|
@ -183,13 +186,13 @@ impl I18nBundle {
|
||||||
|
|
||||||
#[cfg(not(feature = "i18n"))]
|
#[cfg(not(feature = "i18n"))]
|
||||||
{
|
{
|
||||||
let _base_path = base_path; // Suppress unused warning when i18n is enabled
|
// let _base_path = base_path; // Suppress unused warning when i18n is enabled
|
||||||
|
|
||||||
let base = Path::new(base_path);
|
let base = Path::new(_base_path);
|
||||||
|
|
||||||
if !base.exists() {
|
if !base.exists() {
|
||||||
return Err(BotError::config(format!(
|
return Err(BotError::config(format!(
|
||||||
"locales directory not found: {base_path}"
|
"locales directory not found: {_base_path}"
|
||||||
)));
|
)));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue