Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

System locale #5081

Merged
merged 79 commits into from
Aug 8, 2024
Merged
Changes from 1 commit
Commits
Show all changes
79 commits
Select commit Hold shift + click to select a range
1c7b634
Linux support added
ashu26jha Jun 2, 2024
f6b96a9
improved interface of linux
ashu26jha Jun 14, 2024
df361b8
calendar for linux
ashu26jha Jun 14, 2024
a3d8154
added readme
ashu26jha Jun 14, 2024
bb95d79
apple locale and calendar
ashu26jha Jun 18, 2024
ee23fd2
windows locale retrieval
ashu26jha Jun 19, 2024
1b1b514
windows calendar added
ashu26jha Jun 19, 2024
9b3d8e5
Update Readme
ashu26jha Jun 19, 2024
0adf4b9
Fixed linux
ashu26jha Jul 5, 2024
fe54b79
apple fi
ashu26jha Jul 5, 2024
d7ca69e
Minor fix
ashu26jha Jul 5, 2024
1c10d33
Change module name
ashu26jha Jul 5, 2024
c243451
rustdocs
ashu26jha Jul 5, 2024
0aa332e
readme license name fix
ashu26jha Jul 6, 2024
e96b2d3
Rename Readme.md to README.md
ashu26jha Jul 6, 2024
f7212b5
Change crate name
ashu26jha Jul 7, 2024
aba5c24
Added comment for calendar in linux.rs
ashu26jha Jul 19, 2024
72c0404
Re-export modules
ashu26jha Jul 19, 2024
6dc1973
Fix bug in locale retrieval
ashu26jha Jul 19, 2024
7cc7c65
Added enum in linux.rs
ashu26jha Jul 19, 2024
4c594be
tools/graveyard/Cargo.lock removed
ashu26jha Jul 19, 2024
514f4a0
fmt
ashu26jha Jul 19, 2024
6fd9d43
Added safety comments
ashu26jha Jul 20, 2024
3534091
Refactor linux calendar
ashu26jha Jul 23, 2024
62fe2c5
Fixed macos suffixes
ashu26jha Jul 23, 2024
efd5263
Added safety comments, changed function name to plural
ashu26jha Jul 23, 2024
f59808e
fmt
ashu26jha Jul 23, 2024
ad1a293
Windows fix
ashu26jha Jul 23, 2024
68c8775
removed cfg_if
ashu26jha Jul 26, 2024
e072f7f
added ergonomic names in enum
ashu26jha Jul 26, 2024
1a786f9
one unsafe function call per block
ashu26jha Jul 26, 2024
d73f432
added safety comments for apple.rs
ashu26jha Jul 26, 2024
489c6e6
added safety comments for linux.rs
ashu26jha Jul 26, 2024
52a6bc9
remove .clone()
ashu26jha Jul 30, 2024
87dcefc
Release memory
ashu26jha Jul 30, 2024
00efc98
cargo fmt
ashu26jha Jul 30, 2024
3f41cd9
suggestion: remove mod wrapper
ashu26jha Aug 2, 2024
6be1eae
suggestion: added result type in linux.rs
ashu26jha Aug 2, 2024
de9e217
suggestion: refactor to remove .clone()
ashu26jha Aug 2, 2024
daa9ddb
remove test, will be added in further commits
ashu26jha Aug 2, 2024
f1c9367
improved error format
ashu26jha Aug 2, 2024
c3f5da0
windows error propogation fixed
ashu26jha Aug 2, 2024
b5d5a38
remove unnecessary enum in linux.rs
ashu26jha Aug 2, 2024
8a4eea4
suggestion: improve error propagation macos
ashu26jha Aug 2, 2024
c31695f
updated linux.rs to new error enum
ashu26jha Aug 2, 2024
66eacc6
remove unnecessary return in linux.rs
ashu26jha Aug 2, 2024
60c4c58
impl from UTF8Error
ashu26jha Aug 2, 2024
a3d7393
updated apple.rs error propagation
ashu26jha Aug 2, 2024
d348fc4
updated linux.rs error propagation
ashu26jha Aug 2, 2024
3a07b75
linux.rs test covering get_locales
ashu26jha Aug 3, 2024
8ace9df
corrected license
ashu26jha Aug 4, 2024
8b71b4d
added calendar test for linux
ashu26jha Aug 4, 2024
bc2987a
clippy fix
ashu26jha Aug 4, 2024
af6c742
fix: linux test assert statements
ashu26jha Aug 4, 2024
ae87b00
added apple tests
ashu26jha Aug 4, 2024
835cf07
linux test changes
ashu26jha Aug 4, 2024
439de02
change error enum to test in CI
ashu26jha Aug 4, 2024
fdf3b45
error log ci
ashu26jha Aug 4, 2024
c86f7b4
remove return
ashu26jha Aug 4, 2024
a1fe396
handle lang_str NULL case
ashu26jha Aug 4, 2024
668245f
added safety comments and remove null chars
ashu26jha Aug 4, 2024
e0451f9
fix apple calendar
ashu26jha Aug 4, 2024
a40e2c5
refactor apple.rs
ashu26jha Aug 5, 2024
549b9a5
add timezone for apple
ashu26jha Aug 5, 2024
b45216f
added safety comments
ashu26jha Aug 5, 2024
52a491a
added timezone windows.rs
ashu26jha Aug 5, 2024
6865a0b
added windows rs tests
ashu26jha Aug 5, 2024
fdbd74f
optimise: get_string()
ashu26jha Aug 6, 2024
f51575c
error propagation improve
ashu26jha Aug 6, 2024
5586b47
change order of enum
ashu26jha Aug 6, 2024
f4bd006
added comments for get_string()
ashu26jha Aug 6, 2024
1e3a9fb
refactor: get_string()
ashu26jha Aug 6, 2024
82aaf5a
error.rs and variable name changes
ashu26jha Aug 6, 2024
5d4482b
update comment
ashu26jha Aug 6, 2024
feed347
suggestion: use NullLocale instead of NullPointer
ashu26jha Aug 6, 2024
edc9855
remove nullptr error enum
ashu26jha Aug 6, 2024
d9f65d7
updated tests
ashu26jha Aug 7, 2024
de77db0
ci fix: remove into_iter()
ashu26jha Aug 7, 2024
8f3a6c0
nit: fixed ascii assertion in macos test
ashu26jha Aug 7, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
added apple tests
  • Loading branch information
ashu26jha committed Aug 4, 2024
commit ae87b005b5bbc60a75007fa03b73e908f024025c
41 changes: 40 additions & 1 deletion utils/env_preferences/tests/test.rs
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ mod linux_tests {
}

#[test]
fn test_converting() {
fn test_converting_locales() {
let locale_res: std::collections::HashMap<LocaleCategory, String> = get_locales().unwrap();
for locale in locale_res.into_values() {
let parts: Vec<&str> = locale.split('.').collect();
Expand Down Expand Up @@ -70,3 +70,42 @@ mod linux_tests {
assert_eq!(test_calendar_locale.to_string(), calendar_locale);
}
}

#[cfg(target_os = "macos")]
#[cfg(test)]
mod macos_test {
use env_preferences::{get_locales, get_system_calendars};
use icu_locale::Locale;

#[test]
fn test_get_locales() {
let locales_res = get_locales();
match locales_res {
Ok(locales) => {
assert!(!locales.is_empty(), "Unable to retrieve locales for Apple")
}
Err(e) => {
panic!("{:?}", e)
}
}
}

#[test]
fn test_converting_locales() {
let locales = get_locales().unwrap();
for locale in locales {
#[allow(unused_variables)]
let locale_converted: Locale = locale.parse().unwrap();
}
}

#[test]
fn test_calendar() {
let calendar_res = get_system_calendars().unwrap();
for calendar in calendar_res {
#[allow(unused_variables)]
let calendar_locale: Locale = calendar.0.parse().unwrap();
assert!(!calendar.1.is_empty(), "Received empty calendar");
}
}
}
Loading