Check variables in unit tests

Avoid seemingly unused variables by checking something about them, even though
the main point of the test is to check something else.

Check accepted connections in socket test

2 files changed, 7 insertions, 1 deletions

diff --git a/test/maid_test.cpp b/test/maid_test.cpp
index 0456913..fbd278a 100644
--- a/test/maid_test.cpp
+++ b/test/maid_test.cpp
@@ -65,8 +65,10 @@ test()
   {
     assert(n_junk == 0);
     const Maid::managed_ptr<Junk> a = maid.make_managed<Junk>(1U);
+    assert(a->value() == 1U);
     assert(n_junk == 1);
     const Maid::managed_ptr<Junk> b = maid.make_managed<Junk>(2U);
+    assert(b->value() == 2U);
     assert(n_junk == 2);
   }
 
@@ -119,6 +121,7 @@ test()
 
   // Allocate a new object, then let it and the Maid go out of scope
   const Maid::managed_ptr<Junk> c = maid.make_managed<Junk>(5U);
+  assert(c->value() == 5U);
   assert(n_junk == 1);
 }
 
diff --git a/test/socket_test.cpp b/test/socket_test.cpp
index b29788c..1f2d736 100644
--- a/test/socket_test.cpp
+++ b/test/socket_test.cpp
@@ -60,11 +60,15 @@ main()
 
       if (pfds[0].revents & POLLIN) {
         const std::shared_ptr<Socket> conn = unix_server_sock.accept();
+        assert(conn);
+        assert(conn->fd());
         ++n_received;
       }
 
       if (pfds[1].revents & POLLIN) {
         const std::shared_ptr<Socket> conn = tcp_server_sock.accept();
+        assert(conn);
+        assert(conn->fd());
         ++n_received;
       }
     }
@@ -72,7 +76,6 @@ main()
     unix_server_sock.shutdown();
     tcp_server_sock.shutdown();
     unlink("/tmp/raul_test_sock");
-    fprintf(stderr, "n received: %u\n", n_received);
     return n_received != 2;
   }