Make decap tc tests fail on missmatch

Summary: For tc tests we don't check result of comparison in unit tests, only print them. Fixing to fail unit test run when miss match.

Reviewed By: nikhildl12

Differential Revision: D71416328

fbshipit-source-id: 4952e22c947e4655ffd197496c3c2f5d4f8d005c

Author: avasylev

katran/lib/testing/BpfTester.cpp

@@ -129,14 +129,15 @@ bool BpfTester::testFromFixture() {
   return runBpfTesterFromFixtures(config_.bpfProgFd, kXdpCodes, {});
 }
 
-void BpfTester::testClsFromFixture(
+bool BpfTester::testClsFromFixture(
     int progFd,
     std::vector<struct __sk_buff> ctxs_in) {
   std::vector<void*> ctxs;
   for (auto& ctx : ctxs_in) {
     ctxs.push_back(&ctx);
   }
-  runBpfTesterFromFixtures(progFd, kTcCodes, ctxs, sizeof(struct __sk_buff));
+  return runBpfTesterFromFixtures(
+      progFd, kTcCodes, ctxs, sizeof(struct __sk_buff));
 }
 
 bool BpfTester::runBpfTesterFromFixtures(

katran/lib/testing/BpfTester.h

@@ -110,7 +110,7 @@ class BpfTester {
    * helper function to run tests on data from test fixtures
    * for clsact(tc) based bpf program. optional ctxs could be specified
    */
-  void testClsFromFixture(int progFd, std::vector<struct __sk_buff> ctxs_in);
+  bool testClsFromFixture(int progFd, std::vector<struct __sk_buff> ctxs_in);
 
   /**
    * @param vector<string, string> new input fixtures