Update Collection::new()

Updated how Collection objectpath is generated.
Introduced created: u64, parameter to supply the date, rather than create
it inside new()

Signed-off-by: Dhanuka Warusadura <dhanuka@gnome.org>

server/src/daemon/collection.rs

@@ -16,6 +16,8 @@ use zvariant::{ObjectPath, OwnedObjectPath};
 
 use super::{error::ServiceError, prompt::Prompt, Result, Service};
 
+const SECRET_COLLECTION_OBJECTPATH: &str = "/org/freedesktop/secrets.Devel/collection/";
+
 #[derive(Debug)]
 pub struct Collection {
     keyring: Arc<Keyring>,
@@ -123,20 +125,16 @@ impl Collection {
 }
 
 impl Collection {
-    // temporarily creates a generic Collection object
-    pub fn new(label: &str, alias: &str, keyring: Arc<Keyring>) -> Self {
+    pub fn new(label: &str, alias: &str, created: u64, keyring: Arc<Keyring>) -> Self {
         Self {
             items: Default::default(),
             label: label.to_owned(),
             alias: RwLock::new(alias.to_owned()),
             locked: AtomicBool::new(false),
-            created: Duration::from_secs(23123),
-            modified: Duration::from_secs(23123),
-            path: OwnedObjectPath::try_from(format!(
-                "/org/freedesktop/secrets/collection/{}",
-                label
-            ))
-            .unwrap(),
+            created: Duration::from_secs(created),
+            modified: Duration::from_secs(created),
+            path: OwnedObjectPath::try_from(format!("{}{}", SECRET_COLLECTION_OBJECTPATH, alias))
+                .unwrap(),
             keyring,
         }
     }

server/src/daemon/service.rs

@@ -3,6 +3,7 @@
 use std::{
     collections::HashMap,
     sync::{Arc, Mutex},
+    time::SystemTime,
 };
 
 use oo7::{
@@ -57,7 +58,15 @@ impl Service {
         alias: &str,
         #[zbus(object_server)] object_server: &ObjectServer,
     ) -> Result<(OwnedObjectPath, Prompt)> {
-        let collection = Collection::new(properties.label(), alias, Arc::clone(&self.keyring)); // temporarily
+        let collection = Collection::new(
+            properties.label(),
+            alias,
+            SystemTime::now()
+                .duration_since(SystemTime::UNIX_EPOCH)
+                .unwrap()
+                .as_secs(),
+            Arc::clone(&self.keyring),
+        );
         let path = OwnedObjectPath::from(collection.path());
         object_server.at(&path, collection).await?;
         let prompt = Prompt::default(); // temp Prompt