[lldb] Go back to process-directed signals in MainLoopTest.cpp
authorPavel Labath <pavel@labath.sk>
Tue, 6 Sep 2022 12:00:53 +0000 (14:00 +0200)
committerPavel Labath <pavel@labath.sk>
Tue, 6 Sep 2022 12:00:53 +0000 (14:00 +0200)
Thread-directed signals are not caught by kqueue (used on Mac). This
reverts half of D133181.

lldb/unittests/Host/MainLoopTest.cpp

index 5ebe009..00e8551 100644 (file)
@@ -197,7 +197,7 @@ TEST_F(MainLoopTest, Signal) {
 
   auto handle = loop.RegisterSignal(SIGUSR1, make_callback(), error);
   ASSERT_TRUE(error.Success());
-  pthread_kill(pthread_self(), SIGUSR1);
+  kill(getpid(), SIGUSR1);
   ASSERT_TRUE(loop.Run().Success());
   ASSERT_EQ(1u, callback_count);
 }
@@ -215,8 +215,8 @@ TEST_F(MainLoopTest, UnmonitoredSignal) {
 
   auto handle = loop.RegisterSignal(SIGUSR1, make_callback(), error);
   ASSERT_TRUE(error.Success());
-  pthread_kill(pthread_self(), SIGUSR2);
-  pthread_kill(pthread_self(), SIGUSR1);
+  kill(getpid(), SIGUSR2);
+  kill(getpid(), SIGUSR1);
   ASSERT_TRUE(loop.Run().Success());
   ASSERT_EQ(1u, callback_count);
 }
@@ -238,7 +238,7 @@ TEST_F(MainLoopTest, TwoSignalCallbacks) {
         SIGUSR1, [&](MainLoopBase &loop) { ++callback2_count; }, error);
     ASSERT_TRUE(error.Success());
 
-    pthread_kill(pthread_self(), SIGUSR1);
+    kill(getpid(), SIGUSR1);
     ASSERT_TRUE(loop.Run().Success());
     ASSERT_EQ(1u, callback_count);
     ASSERT_EQ(1u, callback2_count);
@@ -251,7 +251,7 @@ TEST_F(MainLoopTest, TwoSignalCallbacks) {
         SIGUSR1, [&](MainLoopBase &loop) { ++callback3_count; }, error);
     ASSERT_TRUE(error.Success());
 
-    pthread_kill(pthread_self(), SIGUSR1);
+    kill(getpid(), SIGUSR1);
     ASSERT_TRUE(loop.Run().Success());
     ASSERT_EQ(2u, callback_count);
     ASSERT_EQ(1u, callback2_count);
@@ -259,7 +259,7 @@ TEST_F(MainLoopTest, TwoSignalCallbacks) {
   }
 
   // Both extra callbacks should be unregistered now.
-  pthread_kill(pthread_self(), SIGUSR1);
+  kill(getpid(), SIGUSR1);
   ASSERT_TRUE(loop.Run().Success());
   ASSERT_EQ(3u, callback_count);
   ASSERT_EQ(1u, callback2_count);