Split Signal unit tests (one expectation per test)

lairworks · Jan 2, 2025 · 9e58ab1 · 9e58ab1
1 parent 1bfa17b
commit 9e58ab1
Showing 1 changed file with 23 additions and 6 deletions.
diff --git a/test/Signal/Signal.test.cpp b/test/Signal/Signal.test.cpp
@@ -12,21 +12,38 @@ namespace {
 }
 
 
-TEST(Signal, ConnectEmitDisconnect) {
-	MockHandler handler{};
-	auto delegate = NAS2D::Delegate{&handler, &MockHandler::MockMethod};
+TEST(Signal, InitEmpty) {
 	NAS2D::Signal<> signal;
-
 	EXPECT_TRUE(signal.empty());
+}
+
+TEST(Signal, ConnectNonEmpty) {
+	NAS2D::Signal<> signal;
+	MockHandler handler{};
+	auto delegate = NAS2D::Delegate{&handler, &MockHandler::MockMethod};
 
 	signal.connect(delegate);
 	EXPECT_FALSE(signal.empty());
+}
 
-	EXPECT_CALL(handler, MockMethod());
-	signal.emit();
+TEST(Signal, DisconnectEmpty) {
+	NAS2D::Signal<> signal;
+	MockHandler handler{};
+	auto delegate = NAS2D::Delegate{&handler, &MockHandler::MockMethod};
 
+	signal.connect(delegate);
 	signal.disconnect(delegate);
 	EXPECT_TRUE(signal.empty());
+}
+
+TEST(Signal, ConnectEmitDisconnect) {
+	NAS2D::Signal<> signal;
+	MockHandler handler{};
+	auto delegate = NAS2D::Delegate{&handler, &MockHandler::MockMethod};
 
+	EXPECT_CALL(handler, MockMethod()).Times(1);
+	signal.connect(delegate);
+	signal.emit();
+	signal.disconnect(delegate);
 	signal.emit();
 }