mirror of
https://github.com/danbulant/Portfolio
synced 2026-07-03 01:50:52 +00:00
fix: random 12 character string tests
This commit is contained in:
parent
db0d72a4f2
commit
0ef7844bcb
2 changed files with 4 additions and 4 deletions
|
|
@ -287,6 +287,6 @@ pub mod tests {
|
||||||
let cookies = admin_login(&client);
|
let cookies = admin_login(&client);
|
||||||
let response = create_candidate(&client, cookies, 1031511, "0".to_string());
|
let response = create_candidate(&client, cookies, 1031511, "0".to_string());
|
||||||
|
|
||||||
assert_eq!(response.password.len(), 8);
|
assert_eq!(response.password.len(), 12);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
@ -13,7 +13,7 @@ use std::str::FromStr;
|
||||||
|
|
||||||
use crate::error::ServiceError;
|
use crate::error::ServiceError;
|
||||||
|
|
||||||
/// Foolproof random 8 char string
|
/// Foolproof random 12 char string
|
||||||
/// only uppercase letters (except for 0 and O) and numbers
|
/// only uppercase letters (except for 0 and O) and numbers
|
||||||
pub fn random_12_char_string() -> String {
|
pub fn random_12_char_string() -> String {
|
||||||
let random_chars_12: Vec<char> = rand::thread_rng()
|
let random_chars_12: Vec<char> = rand::thread_rng()
|
||||||
|
|
@ -339,11 +339,11 @@ pub async fn decrypt_file_with_private_key_as_buffer<P: AsRef<Path>>(
|
||||||
#[cfg(test)]
|
#[cfg(test)]
|
||||||
mod tests {
|
mod tests {
|
||||||
#[test]
|
#[test]
|
||||||
fn test_random_8_char_string() {
|
fn test_random_12_char_string() {
|
||||||
for _ in 0..1000 {
|
for _ in 0..1000 {
|
||||||
let s = super::random_12_char_string();
|
let s = super::random_12_char_string();
|
||||||
// Is 8 chars long
|
// Is 8 chars long
|
||||||
assert_eq!(s.len(), 8);
|
assert_eq!(s.len(), 12);
|
||||||
// Does not contain possibly confusing characters
|
// Does not contain possibly confusing characters
|
||||||
assert!(!s.contains('0'));
|
assert!(!s.contains('0'));
|
||||||
assert!(!s.contains('O'));
|
assert!(!s.contains('O'));
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue