lmi-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lmi-commits] [lmi] master cd18136 4/5: Test material rather than exact


From: Greg Chicares
Subject: [lmi-commits] [lmi] master cd18136 4/5: Test material rather than exact equality of doubles
Date: Mon, 28 Aug 2017 09:27:45 -0400 (EDT)

branch: master
commit cd18136995fc49f1205cc1fd7a3db98c561d8252
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>

    Test material rather than exact equality of doubles
    
    Relaxed an exact-equality test that had happened to pass by accident,
    but no longer did with '-frounding-math'.
---
 currency_test.cpp | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/currency_test.cpp b/currency_test.cpp
index 29aca90..0e7a4d0 100644
--- a/currency_test.cpp
+++ b/currency_test.cpp
@@ -23,6 +23,7 @@
 
 #include "currency.hpp"
 
+#include "materially_equal.hpp"
 #include "test_tools.hpp"
 #include "timer.hpp"
 
@@ -151,9 +152,9 @@ void currency_test::test_double()
     BOOST_TEST_EQUAL(currency::from_value( 0.005).total_cents(),  1);
     BOOST_TEST_EQUAL(currency::from_value(-0.005).total_cents(), -1);
 
-    auto c = currency::from_value(    14857345.859999999404);
-    BOOST_TEST_EQUAL(c.total_cents() ,1485734586);
-    BOOST_TEST_EQUAL(c.value()       ,14857345.86);
+    auto c = currency::from_value(         14857345.859999999404);
+    BOOST_TEST_EQUAL(c.total_cents()     , 1485734586);
+    BOOST_TEST(materially_equal(c.value(), 14857345.86));
 }
 
 void test_stream_roundtrip



reply via email to

[Prev in Thread] Current Thread [Next in Thread]