Make sure manager is Sync+Send (#573)

paritytech · Jul 30, 2020 · 8e7ddc9 · 8e7ddc9
1 parent 254b630
commit 8e7ddc9
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/pubsub/src/manager.rs b/pubsub/src/manager.rs
@@ -34,7 +34,7 @@ use rand::distributions::Alphanumeric;
 use rand::{thread_rng, Rng};
 
 /// Cloneable `Spawn` handle.
-pub type TaskExecutor = Arc<dyn futures::task::Spawn>;
+pub type TaskExecutor = Arc<dyn futures::task::Spawn + Send + Sync>;
 
 type ActiveSubscriptions = Arc<Mutex<HashMap<SubscriptionId, oneshot::Sender<()>>>>;
 
@@ -360,4 +360,11 @@ mod tests {
 
 		assert!(is_not_cancelled);
 	}
+
+	#[test]
+	fn is_send_sync() {
+		fn send_sync<T: Send + Sync>() {}
+
+		send_sync::<SubscriptionManager>();
+	}
 }