Use matches! macro (#553)

Apparently our Windows runner has been updated, so this should work now.
paritytech · Jun 2, 2020 · ef92a48 · ef92a48
1 parent e2569b9
commit ef92a48
Showing 1 changed file with 3 additions and 15 deletions.
diff --git a/pubsub/src/manager.rs b/pubsub/src/manager.rs
@@ -304,11 +304,7 @@ mod tests {
 			sink.sink_map_err(|_| ()).send_all(stream).map(|_| ())
 		});
 
-		if let SubscriptionId::String(_) = id {
-			assert!(true)
-		} else {
-			assert!(false, "Expected SubscriptionId::String");
-		}
+		assert!(matches!(id, SubscriptionId::String(_)))
 	}
 
 	#[test]
@@ -325,11 +321,7 @@ mod tests {
 			sink.sink_map_err(|_| ()).send_all(stream).map(|_| ())
 		});
 
-		if let SubscriptionId::Number(_) = id {
-			assert!(true)
-		} else {
-			assert!(false, "Expected SubscriptionId::Number");
-		}
+		assert!(matches!(id, SubscriptionId::Number(_)))
 	}
 
 	#[test]
@@ -346,11 +338,7 @@ mod tests {
 			sink.sink_map_err(|_| ()).send_all(stream).map(|_| ())
 		});
 
-		if let SubscriptionId::String(_) = id {
-			assert!(true)
-		} else {
-			assert!(false, "Expected SubscriptionId::String");
-		}
+		assert!(matches!(id, SubscriptionId::String(_)))
 	}
 
 	#[test]