perf(session): fix tests after push() return type change

Replace assert!(messages.push(...)) with plain push() calls
now that push() returns () instead of bool.

Signed-off-by: Sam Betts <1769706+Tehsmash@users.noreply.github.com>

Author: Tehsmash

data-plane/core/session/src/producer_buffer.rs

@@ -170,32 +170,32 @@ mod tests {
             .build_publish()
             .unwrap();
 
-        assert!(messages.push(p0.clone()));
+        messages.push(p0.clone());
 
         assert_eq!(messages.get(0).unwrap(), p0);
         assert_eq!(messages.get(0).unwrap(), p0);
         assert_eq!(messages.get(0).unwrap(), p0);
         assert_eq!(messages.get(1), None);
 
-        assert!(messages.push(p0.clone()));
-        assert!(messages.push(p1.clone()));
-        assert!(messages.push(p2.clone()));
+        messages.push(p0.clone());
+        messages.push(p1.clone());
+        messages.push(p2.clone());
 
         assert_eq!(messages.get(0).unwrap(), p0);
         assert_eq!(messages.get(1).unwrap(), p1);
         assert_eq!(messages.get(2).unwrap(), p2);
         assert_eq!(messages.get(3), None);
 
         // now the messages is full, add a new element will remote the elem 0
-        assert!(messages.push(p3.clone()));
+        messages.push(p3.clone());
         assert_eq!(messages.get(0), None);
         assert_eq!(messages.get(1).unwrap(), p1);
         assert_eq!(messages.get(2).unwrap(), p2);
         assert_eq!(messages.get(3).unwrap(), p3);
         assert_eq!(messages.get(4), None);
 
         // now the messages is full, add a new element will remote the elem 1
-        assert!(messages.push(p4.clone()));
+        messages.push(p4.clone());
         assert_eq!(messages.get(0), None);
         assert_eq!(messages.get(1), None);
         assert_eq!(messages.get(2).unwrap(), p2);
@@ -211,11 +211,11 @@ mod tests {
         assert_eq!(messages.get(4), None);
 
         // add all msgs and check again
-        assert!(messages.push(p0.clone()));
-        assert!(messages.push(p1.clone()));
-        assert!(messages.push(p2.clone()));
-        assert!(messages.push(p3.clone()));
-        assert!(messages.push(p4.clone()));
+        messages.push(p0.clone());
+        messages.push(p1.clone());
+        messages.push(p2.clone());
+        messages.push(p3.clone());
+        messages.push(p4.clone());
         assert_eq!(messages.get(0), None);
         assert_eq!(messages.get(1), None);
         assert_eq!(messages.get(2).unwrap(), p2);