[c89stringutils_tests/test_c89stringutils/test_string_extras.h] Add jasprintf test; [c89stringutils/c89stringutils_string_extras.h] Whitespace; [CMakeLists.txt] Guard test runner properly

Author: SamuelMarks

CMakeLists.txt

@@ -37,8 +37,9 @@ configure_file(
 add_subdirectory("${PROJECT_NAME}")
 
 include(CTest)
-enable_testing()
-add_subdirectory("c89stringutils_tests")
+if (BUILD_TESTING)
+    add_subdirectory("c89stringutils_tests")
+endif (BUILD_TESTING)
 
 # header-only so including this will pollute the `find_path` suggestion in vcpkg
 #install(

c89stringutils/c89stringutils_string_extras.h

@@ -145,8 +145,8 @@ extern int strcasecmp(const char *, const char *);
 
 #if defined(C89STRINGUTILS_IMPLEMENTATION) && !defined(HAVE_STRNCASECMP_H)
 #define HAVE_STRNCASECMP_H
-#define strncasecmp _strnicmp
 
+#define strncasecmp _strnicmp
 #define strcasecmp _stricmp
 
 #endif /* defined(C89STRINGUTILS_IMPLEMENTATION) &&                            \

c89stringutils_tests/test_c89stringutils/test_string_extras.h

@@ -16,8 +16,18 @@ TEST x_strnstr_should_fail(void) {
   PASS();
 }
 
+TEST x_jasprintf_should_succeed(void) {
+  char *s=NULL;
+  jasprintf(&s, "foo%s", "bar");
+  jasprintf(&s, "can%s", "haz");
+  ASSERT_EQ(strcmp(s, "foobarcanhaz"), 0);
+  free(s);
+  PASS();
+}
+
 /* Suites can group multiple tests with common setup. */
 SUITE(strnstr_suite) {
   RUN_TEST(x_strnstr_should_succeed);
   RUN_TEST(x_strnstr_should_fail);
+  RUN_TEST(x_jasprintf_should_succeed);
 }