Correct testing bugs:

either changing assert(...) or BOOST_ASSERT(...) to BOOST_TEST
    (in my code only)

    or adding "return boost::report_errors();" where it was clearly
    missing (and a pure bug, in anyone's code).


[SVN r37057]

test/filter_iterator_test.cpp

@@ -268,6 +268,5 @@ int main()
         dummyT(1), dummyT(4));
 
 
-  std::cout << "test successful " << std::endl;
-  return boost::exit_success;
+    return boost::report_errors();
 }

test/iterator_facade.cpp

@@ -6,7 +6,6 @@
 
 #include <boost/iterator/iterator_facade.hpp>
 #include <boost/iterator/new_iterator_tests.hpp>
-#include <boost/assert.hpp>
 
 // This is a really, really limited test so far.  All we're doing
 // right now is checking that the postfix++ proxy for single-pass
@@ -95,7 +94,7 @@ int main()
     state = 3;
     boost::readable_iterator_test(counter_iterator<proxy>(&state), 3);
     boost::writable_iterator_test(counter_iterator<proxy>(&state), 9, 7);
-    BOOST_ASSERT(state == 8);
+    BOOST_TEST(state == 8);
 
     // test for a fix to http://tinyurl.com/zuohe
     // These two lines should be equivalent (and both compile)
@@ -103,5 +102,5 @@ int main()
     (*p).mutator();
     p->mutator();
     
-    return 0;
+    return boost::report_errors();
 }

test/reverse_iterator_test.cpp

@@ -170,6 +170,5 @@ int main()
 #endif
   }
 
-  std::cout << "test successful " << std::endl;
-  return boost::exit_success;
+  return boost::report_errors();
 }

test/transform_iterator_test.cpp

@@ -244,5 +244,5 @@ main()
     );
   }
 
-  return 0;
+  return boost::report_errors();
 }