Skip to content

Commit 7f4bedb

Browse files
committed
f Have insert_or_update take object by value
1 parent 748a664 commit 7f4bedb

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

src/data_store.rs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ where
7272
Ok(updated)
7373
}
7474

75-
pub(crate) fn insert_or_update(&self, object: &SO) -> Result<bool, Error> {
75+
pub(crate) fn insert_or_update(&self, object: SO) -> Result<bool, Error> {
7676
let mut locked_objects = self.objects.lock().unwrap();
7777

7878
let updated;
@@ -274,14 +274,14 @@ mod tests {
274274
// Check `insert_or_update` inserts unknown objects
275275
let iou_id = TestObjectId { id: [55u8; 4] };
276276
let iou_object = TestObject { id: iou_id, data: [34u8; 3] };
277-
assert_eq!(Ok(true), data_store.insert_or_update(&iou_object));
277+
assert_eq!(Ok(true), data_store.insert_or_update(iou_object.clone()));
278278

279279
// Check `insert_or_update` doesn't update the same object
280-
assert_eq!(Ok(false), data_store.insert_or_update(&iou_object));
280+
assert_eq!(Ok(false), data_store.insert_or_update(iou_object.clone()));
281281

282282
// Check `insert_or_update` updates if object changed
283-
let mut new_iou_object = iou_object.clone();
283+
let mut new_iou_object = iou_object;
284284
new_iou_object.data[0] += 1;
285-
assert_eq!(Ok(true), data_store.insert_or_update(&new_iou_object));
285+
assert_eq!(Ok(true), data_store.insert_or_update(new_iou_object));
286286
}
287287
}

0 commit comments

Comments
 (0)