Add tests. Correct path.

Author: abellgithub

pdal/plang/Redirector.cpp

@@ -171,7 +171,7 @@ PyObject* Redirector::init()
 
 void Redirector::set_stdout(std::ostream* ostr)
 {
-    stdout_write_type writeFunc = [ostr](std::string msg) { std::cerr << "Ostr = " << ostr << "!\n"; *ostr << msg; };
+    stdout_write_type writeFunc = [ostr](std::string msg) { *ostr << msg; };
     stdout_flush_type flushFunc = [ostr]{ ostr->flush(); };
 
     this->set_stdout(writeFunc, flushFunc);

pdal/test/PythonFilterTest.cpp

@@ -838,23 +838,17 @@ TEST(PLangTest, log)
         EXPECT_EQ(view->size(), 750u);
     }
 
-    if (FileUtils::fileExists(logfile))
-        std::cerr << "File exists!\n";
-    else
-        std::cerr << "File doesn't exist!\n";
-    /**
     bool ok = Support::compare_text_files(logfile,
         Support::datapath("logs/log_py.txt"));
-    **/
 
     // TODO: fails on Windows
     // unknown file: error: C++ exception with description "pdalboost::filesystem::remove:
     // The process cannot access the file because it is being used by another process:
     // "C:/projects/pdal/test/data/../temp/mylog_three.txt"" thrown in the test body.
-    //if (ok)
-    //    FileUtils::deleteFile(Support::temppath("mylog_three.txt"));
+    if (ok)
+        FileUtils::deleteFile(Support::temppath("mylog_three.txt"));
 
-//    EXPECT_TRUE(ok);
+    EXPECT_TRUE(ok);
 }
 
 

pdal/test/TestConfig.hpp

@@ -35,8 +35,8 @@
 #ifndef UNITTEST_TESTCONFIG_INCLUDED
 #define UNITTEST_TESTCONFIG_INCLUDED
 
-#define UNITTEST_TESTCONFIG_DATA_PATH "./pdal/test/data/"
-#define UNITTEST_TESTCONFIG_CONFIGURED_PATH "./pdal/test/data/"
+#define UNITTEST_TESTCONFIG_DATA_PATH "./test/data/"
+#define UNITTEST_TESTCONFIG_CONFIGURED_PATH "./test/data/"
 #define UNITTEST_TESTCONFIG_BINARY_PATH ""
 #define UNITTEST_TESTCONFIG_OCI_CONNECTION ""