Draft: fix tests for log2_approx and log2_approx64

• skip the test log2_approx.complete when using ctest
  • this test takes too much time to run
• fix test log2_approx64.boundaries
  • The test was actually never executed correctly (unsigned -10 is never <= 10), and log2_approx64_ref() was not accurate enough.
  • Therefore replace it by directly comparing one value above and one value below each rounding boundary.
• add test log2_approx.boundaries

openair1/PHY/TOOLS/tests/test_log2_approx.cpp

@@ -10,25 +10,27 @@ uint8_t log2_approx_ref(uint32_t x)
   return std::round(std::log2(x));
 }
 
-uint8_t log2_approx64_ref(unsigned long long int x)
-{
-  return std::round(std::log2(static_cast<long double>(x)));
-}
-
 TEST(log2_approx, complete)
 {
   for (uint32_t i = 0; i < UINT32_MAX; i++)
     EXPECT_EQ(log2_approx(i), log2_approx_ref(i));
 }
 
+TEST(log2_approx, boundaries)
+{
+  for (int i = 0; i < 32; i++) {
+    uint32_t i2 = std::pow(2.0, i + 0.5);
+    EXPECT_EQ(log2_approx(i2), i) << "log2(" << i2 << ")";
+    EXPECT_EQ(log2_approx(i2 + 1), i + 1) << "log2(" << i2 + 1 << ")";
+  }
+}
+
 TEST(log2_approx64, boundaries)
 {
-  for (long double i = 0; i < 64; i++) {
-    unsigned long long i2 = std::pow(2.0L, i + 0.5L);
-    for (unsigned long long j = -10; j <= 10; j++) {
-      unsigned long long x = i2 + j;
-      EXPECT_EQ(log2_approx64(x), log2_approx64_ref(x));
-    }
+  for (int i = 0; i < 64; i++) {
+    unsigned long long int i2 = std::pow(2.0L, i + 0.5L);
+    EXPECT_EQ(log2_approx64(i2), i) << "log2(" << i2 << ")";
+    EXPECT_EQ(log2_approx64(i2 + 1), i + 1) << "log2(" << i2 + 1 << ")";
   }
 }