Skip to content

Commit a0984c8

Browse files
committed
fix tests by removing use of take_loaned
1 parent 86e3007 commit a0984c8

File tree

1 file changed

+0
-21
lines changed

1 file changed

+0
-21
lines changed

rclrs/src/subscription.rs

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -658,8 +658,6 @@ mod tests {
658658

659659
assert!(subscriber.take()?.is_none());
660660
assert!(subscriber.take_with_info()?.is_none());
661-
assert!(subscriber.take_loaned()?.is_none());
662-
assert!(subscriber.take_loaned_with_info()?.is_none());
663661

664662
publisher.publish(msg::Empty::default())?;
665663
waitset.wait(timeout, |_| Ok(()))?;
@@ -668,29 +666,10 @@ mod tests {
668666

669667
assert!(subscriber.take()?.is_none());
670668
assert!(subscriber.take_with_info()?.is_none());
671-
assert!(subscriber.take_loaned()?.is_none());
672-
assert!(subscriber.take_loaned_with_info()?.is_none());
673669

674670
publisher.publish(msg::Empty::default())?;
675671
waitset.wait(timeout, |_| Ok(()))?;
676672

677-
assert!(subscriber.take_loaned()?.is_some());
678-
679-
assert!(subscriber.take()?.is_none());
680-
assert!(subscriber.take_with_info()?.is_none());
681-
assert!(subscriber.take_loaned()?.is_none());
682-
assert!(subscriber.take_loaned_with_info()?.is_none());
683-
684-
publisher.publish(msg::Empty::default())?;
685-
waitset.wait(timeout, |_| Ok(()))?;
686-
687-
assert!(subscriber.take_loaned_with_info()?.is_some());
688-
689-
assert!(subscriber.take()?.is_none());
690-
assert!(subscriber.take_with_info()?.is_none());
691-
assert!(subscriber.take_loaned()?.is_none());
692-
assert!(subscriber.take_loaned_with_info()?.is_none());
693-
694673
Ok(())
695674
}
696675
}

0 commit comments

Comments
 (0)