Added a few more tests

Author: IhorShevchuk

libpiper/src/main/CMakeLists.txt

@@ -22,4 +22,9 @@ target_sources(
 
 target_link_libraries(main_utils PRIVATE piper)
 
-target_link_libraries(piper_exe PRIVATE piper main_utils)
+target_link_libraries(piper_exe PRIVATE piper main_utils)
+
+install(
+  TARGETS piper_exe
+  RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR}
+)

libpiper/tests/CMakeLists.txt

@@ -59,8 +59,9 @@ endif()
 
 include(GoogleTest)
 get_target_property(PIPER_BINARY_DIR piper BINARY_DIR)
+get_target_property(PIPER_EXE_BINARY_DIR piper_exe BINARY_DIR)
 
 gtest_discover_tests(piper_test
     DISCOVERY_MODE PRE_TEST
-    PROPERTIES ENVIRONMENT "PATH=${PIPER_BINARY_DIR}/$<CONFIG>;${ONNXRUNTIME_DIR}/lib;$ENV{PATH}"
+    PROPERTIES ENVIRONMENT "PATH=${PIPER_EXE_BINARY_DIR}/$<CONFIG>;${PIPER_BINARY_DIR}/$<CONFIG>;${ONNXRUNTIME_DIR}/lib;$ENV{PATH}"
 )

libpiper/tests/test_main_utils.cpp

@@ -108,4 +108,21 @@ TEST_F(MainUtilsTest, ParseArgsMissingModel) {
         throw;
       },
       std::runtime_error);
+}
+
+TEST_F(MainUtilsTest, ParseArgsMissingArgument) {
+  piper::RunConfig runConfig;
+
+  const char *argv[] = {"test_program", "--model"};
+  int argc = sizeof(argv) / sizeof(argv[0]);
+
+  EXPECT_THROW(
+      try {
+        parseArgsLogic(argc, const_cast<char **>(argv), runConfig);
+      } catch (const piper::ArgError &e) {
+        EXPECT_STREQ(e.what(), "Missing argument for --model");
+        throw;
+      },
+      piper::ArgError
+  );
 }

libpiper/tests/test_process.cpp

@@ -106,6 +106,32 @@ TEST_F(ProcessTest, ProcessInputStreamFileOutput) {
   std::filesystem::remove(outputPath);
 }
 
+TEST_F(ProcessTest, ProcessInputStreamDirectoryOutput) {
+  piper::RunConfig runConfig;
+  auto outputDir = std::filesystem::temp_directory_path() / "piper_test_output";
+  std::filesystem::create_directory(outputDir);
+  runConfig.outputPath = outputDir;
+  runConfig.outputType = piper::OUTPUT_DIRECTORY;
+
+  piper_synthesize_options options = piper_default_synthesize_options(synth);
+  options.speaker_id = 0;
+
+  StdinRedirect redirect("This is a test for directory output.");
+
+  std::stringstream cout_buffer;
+  std::streambuf* old_cout = std::cout.rdbuf(cout_buffer.rdbuf());
+
+  processInputStream(runConfig, synth, &options);
+
+  std::cout.rdbuf(old_cout);
+
+  std::string created_path = cout_buffer.str();
+  created_path.erase(created_path.find_last_not_of(" \n\r\t")+1);
+  ASSERT_TRUE(std::filesystem::exists(created_path));
+  ASSERT_GT(std::filesystem::file_size(created_path), 44);
+  std::filesystem::remove_all(outputDir);
+}
+
 TEST_F(ProcessTest, ProcessInputStreamJson) {
   piper::RunConfig runConfig;
   runConfig.outputType = piper::OUTPUT_STDOUT;