tests: merge TcpChannel and UdpChannel test suites using test case templates

Change-Id: Id56bb036db53494d4d250f9ea68a3ded78b33ddb
Refs: #3514
diff --git a/tests/daemon/face/factory-test-common.hpp b/tests/daemon/face/factory-test-common.hpp
index e842ced..f7d9535 100644
--- a/tests/daemon/face/factory-test-common.hpp
+++ b/tests/daemon/face/factory-test-common.hpp
@@ -48,7 +48,7 @@
            const CreateFaceExpectedResult& expected)
 {
   factory.createFace(uri, persistency, wantLocalFieldsEnabled,
-                     [expected] (const shared_ptr<Face>& newFace) {
+                     [expected] (const shared_ptr<Face>&) {
                        BOOST_CHECK_EQUAL(CreateFaceExpectedResult::SUCCESS, expected.result);
                      },
                      [expected] (uint32_t actualStatus, const std::string& actualReason) {
@@ -58,11 +58,6 @@
                      });
 }
 
-inline void
-failIfError(uint32_t status, const std::string& reason) {
-  BOOST_FAIL("No error expected, but got: [" << status << ": " << reason << "]");
-}
-
 } // namespace tests
 } // namespace nfd