Minor fix to how Users are constructed

Signed-off-by: Max Dymond <max.dymond@metaswitch.com>

Author: maxdymond

src/environment.rs

@@ -13,6 +13,16 @@ pub struct User {
     pub gid: nix::unistd::Gid,
 }
 
+impl User {
+    /// Get the user and group ids of the current user
+    fn current() -> Self {
+        let uid = nix::unistd::getuid();
+        let gid = nix::unistd::getgid();
+        debug!("Current user has uid {} and group {}", uid, gid);
+        Self { uid, gid }
+    }
+}
+
 #[derive(Debug)]
 pub struct Environment {
     /// User uid and gid
@@ -35,7 +45,7 @@ impl Environment {
     /// Gather information on the environment floki is running in
     pub fn gather(config_file: &Option<path::PathBuf>) -> Result<Self, Error> {
         let (floki_root, config_path) = resolve_floki_root_and_config(config_file)?;
-        let user = get_user_details();
+        let user = User::current();
         Ok(Environment {
             user_details: user,
             current_directory: get_current_working_directory()?,
@@ -47,14 +57,6 @@ impl Environment {
     }
 }
 
-/// Get the user and group ids of the current user
-fn get_user_details() -> User {
-    let uid = nix::unistd::getuid();
-    let gid = nix::unistd::getgid();
-    debug!("Current user has uid {} and group {}", uid, gid);
-    User { uid, gid }
-}
-
 /// Get the current working directory as a String
 fn get_current_working_directory() -> Result<path::PathBuf, Error> {
     Ok(env::current_dir()?)