[WIP] coverage increased due new malloc() interceptor.

Author: Gonzalo Diaz

src/tests/unit/lib/hackerrank/warmup/time_conversion.test.cpp

@@ -1,41 +1,57 @@
 #include <catch2/catch_test_macros.hpp>
 
+#include "../../../../tools/memory_injector.hpp"
 #include <exercises/hackerrank/warmup/time_conversion.h>
 #include <filesystem>
 #include <fstream>
 #include <nlohmann/json.hpp>
-#include <vector>
 
 using json = nlohmann::json;
 
 TEST_CASE("time_conversion JSON Test Cases",
           "[hackerrank] [jsontestcase] [warmup]") {
-  std::filesystem::path cwd = std::filesystem::current_path();
-  std::string path =
-      cwd.string() +
-      "/unit/lib/hackerrank/warmup/time_conversion.testcases.json";
 
-  INFO("time_conversion JSON test cases FILE: " << path);
+#ifdef __linux__
+  SECTION("time_conversion Failure handling during Out of Memory (OOM)") {
+    MemoryInjector::enable_oom_fault();
 
-  std::ifstream f(path);
-  json data = json::parse(f);
+    const char *input = "12:01:00PM";
 
-  for (auto testcase : data) {
-    char *result = HACKERRANK_WARMUP_timeConversion(
-        testcase["input"].get<std::string>().c_str());
+    REQUIRE(HACKERRANK_WARMUP_timeConversion(input) == nullptr);
 
-    std::string result_as_string(result);
+    // Clean up the state after finishing the test section
+    MemoryInjector::disable_fault();
+  }
+#endif
+
+  SECTION("Normal memory allocation succeeds") {
+    std::filesystem::path cwd = std::filesystem::current_path();
+    std::string path =
+        cwd.string() +
+        "/unit/lib/hackerrank/warmup/time_conversion.testcases.json";
+
+    INFO("time_conversion JSON test cases FILE: " << path);
+
+    std::ifstream f(path);
+    json data = json::parse(f);
 
-    free(result);
+    for (auto testcase : data) {
+      char *result = HACKERRANK_WARMUP_timeConversion(
+          testcase["input"].get<std::string>().c_str());
 
-    CHECK(result_as_string == testcase["expected"]);
+      std::string result_as_string(result);
+
+      free(result);
+
+      CHECK(result_as_string == testcase["expected"]);
+    }
   }
-}
 
-TEST_CASE("time_conversion edge cases", "[hackerrank] [helper] [warmup]") {
-  CHECK(HACKERRANK_WARMUP_timeConversion(nullptr) == nullptr);
+  SECTION("time_conversion edge cases", "[hackerrank] [helper] [warmup]") {
+    CHECK(HACKERRANK_WARMUP_timeConversion(nullptr) == nullptr);
 
-  CHECK(HACKERRANK_WARMUP_timeConversion("") == nullptr);
+    CHECK(HACKERRANK_WARMUP_timeConversion("") == nullptr);
 
-  CHECK(HACKERRANK_WARMUP_timeConversion("aa:bb:ccXM") == nullptr);
+    CHECK(HACKERRANK_WARMUP_timeConversion("aa:bb:ccXM") == nullptr);
+  }
 }