Fix some things

Author: Kaiden42

core/src/passwords.rs

@@ -123,8 +123,7 @@ mod tests {
 
         symbols.append(&Symbols::LOWER_ALPHA);
 
-        let password =
-            generate_password(32, &symbols).expect("Password generation should not fail.");
+        let password = generate_password(32, &symbols);
         assert_eq!(password.len(), 32);
 
         assert!(password.chars().into_iter().all(|c| symbols.contains(&c)));
@@ -145,8 +144,7 @@ mod tests {
         symbols.append(&Symbols::NUMBERS);
         symbols.append(&Symbols::SPECIAL);
 
-        let password =
-            generate_password(64, &symbols).expect("Password generation should not fail.");
+        let password = generate_password(64, &symbols);
         assert_eq!(password.len(), 64);
 
         assert!(password.chars().into_iter().all(|c| symbols.contains(&c)));

gui/src/lib.rs

@@ -3,7 +3,7 @@
 //! It uses the [core](pwduck_core) module internally to manage the passwords of the user.
 #![deny(missing_docs)]
 #![deny(missing_debug_implementations)]
-#![deny(unused_results)]
+#![cfg_attr(not(test), deny(unused_results))]
 #![cfg_attr(not(test), forbid(unsafe_code))]
 #![cfg_attr(coverage, feature(no_coverage))]
 #![warn(

gui/src/pw_modal/password_generator/mod.rs

@@ -118,7 +118,9 @@ impl PasswordGeneratorState {
                 _,
                 VaultTabMessage::Container(VaultContainerMessage::ModifyEntry(_)),
             ) => Target::EntryModifier,
-            crate::Message::VaultTab(_, VaultTabMessage::Creator(_)) => todo!(),
+            crate::Message::VaultTab(_, VaultTabMessage::Creator(_)) => {
+                unimplemented!("Maybe add the generator to the vault creator too")
+            }
             _ => Target::None,
         });
 

gui/src/vault/container/list.rs

@@ -172,7 +172,7 @@ impl ListView {
                 } else {
                     self.split_state.divider_position().map_or_else(
                         || viewport.width / 2,
-                        |position| viewport.width - u32::from(position),
+                        |position| viewport.width - u32::from(position).min(viewport.width),
                     )
                 },
                 height: viewport.height,
@@ -197,6 +197,8 @@ impl ListView {
             )
             .style(theme.split())
             .padding(0.0)
+            .min_size_first(100)
+            .min_size_second(300)
             .into()
         };
 

gui/src/vault/container/mod.rs

@@ -513,7 +513,9 @@ impl Component for VaultContainer {
     type ConstructorParam = Box<Vault>;
 
     fn new(vault: Self::ConstructorParam) -> Self {
-        let root_uuid = vault.get_root_uuid().unwrap();
+        let root_uuid = vault
+            .get_root_uuid()
+            .expect("There should always be a root uuid.");
         let list_view = ListView::new(root_uuid, &vault);
 
         Self {

src/main.rs

@@ -63,10 +63,7 @@ mod desktop {
     use enigo::KeyboardControllable;
     use rfd::AsyncFileDialog;
 
-    use pwduck_gui::{
-        error::{NfdError, PWDuckGuiError},
-        Platform, Sequence,
-    };
+    use pwduck_gui::{error::NfdError, Platform, Sequence};
 
     /// An empty placeholder struct to implement the [`Platform`](Platform) trait for.
     #[derive(Default)]
@@ -126,7 +123,7 @@ mod desktop {
         }
 
         async fn auto_type(sequence: Sequence) -> Result<(), pwduck_gui::error::PWDuckGuiError> {
-            async_std::task::sleep(std::time::Duration::from_millis(1000)).await;
+            async_std::task::sleep(std::time::Duration::from_millis(5000)).await;
 
             let mut enigo = enigo::Enigo::new();
             #[cfg(target_os = "linux")]
@@ -135,7 +132,7 @@ mod desktop {
 
                 // Check if xdotools is available
                 drop(which::which("xdotool")
-                    .map_err(|err| PWDuckGuiError::String(format!("xdotool could not be found. Maybe it is not installed on your system? ({})", err)))?);
+                    .map_err(|err| pwduck_gui::error::PWDuckGuiError::String(format!("xdotool could not be found. Maybe it is not installed on your system? ({})", err)))?);
             }
 
             for part in sequence.iter() {