Merge pull request #685 from cppalliance/test_coverage

Improve testing coverage

Author: mborland

test/compare_dec128_and_fast.cpp

@@ -205,6 +205,8 @@ void test_mul()
 
         if (!BOOST_TEST_EQ(static_cast<double>(dec128_res), static_cast<double>(dec128_fast_res)))
         {
+            // LCOV_EXCL_START
+
             std::cerr << std::setprecision(35)
                       << "Val 1: " << val1
                       << "\nVal 2: " << val2
@@ -214,6 +216,8 @@ void test_mul()
                       << "\nDecfast 1: " << dec128_fast_1
                       << "\nDecfast 2: " << dec128_fast_2
                       << "\nDecfast res: " << dec128_fast_res << std::endl;
+
+            // LCOV_EXCL_STOP
         }
     }
 
@@ -234,6 +238,8 @@ void test_mul()
 
         if (!BOOST_TEST_EQ(static_cast<double>(dec128_res), static_cast<double>(dec128_fast_res)))
         {
+            // LCOV_EXCL_START
+
             std::cerr << std::setprecision(35)
                       << "Val 1: " << val1
                       << "\nVal 2: " << val2
@@ -243,6 +249,8 @@ void test_mul()
                       << "\nDecfast 1: " << dec128_fast_1
                       << "\nDecfast 2: " << dec128_fast_2
                       << "\nDecfast res: " << dec128_fast_res << std::endl;
+
+            // LCOV_EXCL_STOP
         }
     }
 }
@@ -266,6 +274,8 @@ void test_div()
 
         if (!BOOST_TEST_EQ(static_cast<double>(dec128_res), static_cast<double>(dec128_fast_res)))
         {
+            // LCOV_EXCL_START
+
             std::cerr << std::setprecision(35)
                       << "Val 1: " << val1
                       << "\nVal 2: " << val2
@@ -275,6 +285,8 @@ void test_div()
                       << "\nDecfast 1: " << dec128_fast_1
                       << "\nDecfast 2: " << dec128_fast_2
                       << "\nDecfast res: " << dec128_fast_res << std::endl;
+
+            // LCOV_EXCL_STOP
         }
     }
 
@@ -295,6 +307,8 @@ void test_div()
 
         if (!BOOST_TEST_EQ(static_cast<double>(dec128_res), static_cast<double>(dec128_fast_res)))
         {
+            // LCOV_EXCL_START
+
             std::cerr << std::setprecision(35)
                       << "Val 1: " << val1
                       << "\nVal 2: " << val2
@@ -304,6 +318,8 @@ void test_div()
                       << "\nDecfast 1: " << dec128_fast_1
                       << "\nDecfast 2: " << dec128_fast_2
                       << "\nDecfast res: " << dec128_fast_res << std::endl;
+
+            // LCOV_EXCL_STOP
         }
     }
 }

test/random_decimal128_fast_math.cpp

@@ -241,7 +241,7 @@ void random_multiplication(T lower, T upper)
 
         if (val1 * val2 == 0)
         {
-            continue;
+            continue; // LCOV_EXCL_LINE
         }
 
         if (!BOOST_TEST_EQ(res, res_int))

test/test_cmath.cpp

@@ -802,6 +802,8 @@ void test_lrint()
         // Difference in rounding mode at 0.5
         if (abs(ret_dec) == abs(ret_val) + 1)
         {
+            // LCOV_EXCL_START
+
             float iptr;
             const auto frac = std::modf(val1, &iptr);
             if (abs(abs(frac) - 0.5F) < 0.01F)
@@ -810,11 +812,11 @@ void test_lrint()
             }
             else
             {
-                // LCOV_EXCL_START
                 std::cerr << "Frac: " << frac
                           << "\nDist: " << std::fabs(frac - 0.5F) / std::numeric_limits<float>::epsilon() << std::endl;
-                // LCOV_EXCL_STOP
             }
+
+            // LCOV_EXCL_STOP
         }
 
         if (!BOOST_TEST_EQ(ret_val, ret_dec))