X-Git-Url: http://git.lttng.org./?a=blobdiff_plain;ds=sidebyside;f=tests%2Funit%2Ftest_utils_expand_path.c;h=ff4394b684d95d195bd43f151b0109ebce14eaff;hb=2e96442dcb6d400164d43ca8c09dc480f123385a;hp=44173f6f023e2c18ae8eb848ef15965c0978ca7c;hpb=890d8fe47755c3bad936389cf48ffa141cff41c9;p=lttng-tools.git

diff --git a/tests/unit/test_utils_expand_path.c b/tests/unit/test_utils_expand_path.c
index 44173f6f0..ff4394b68 100644
--- a/tests/unit/test_utils_expand_path.c
+++ b/tests/unit/test_utils_expand_path.c
@@ -26,8 +26,7 @@
 
 #include <tap/tap.h>
 
-#include <src/common/utils.h>
-
+#include <common/utils.h>
 #include <common/common.h>
 
 /* For error.h */
@@ -281,8 +280,8 @@ error:
 static void test_utils_expand_path(void)
 {
 	char *result;
-	char name[100], tmppath[PATH_MAX];
-	int i;
+	char name[100], tmppath[PATH_MAX], real_tree_origin[PATH_MAX];
+	int i, treelen;
 
 	/* Test valid cases */
 	for (i = 0; i < num_valid_tests; i++) {
@@ -295,14 +294,24 @@ static void test_utils_expand_path(void)
 		free(result);
 	}
 
+	/*
+	 * Get the realpath for the tree_origin since it can itself be a
+	 * symlink.
+	 */
+	result = realpath(tree_origin, real_tree_origin);
+	if (!result) {
+		fail("realpath failed.");
+		return;
+	}
+
 	/* Test symlink tree cases */
-	int treelen = strlen(tree_origin) + 1;
+	treelen = strlen(real_tree_origin) + 1;
 	for (i = 0; i < num_symlink_tests; i++) {
 		sprintf(name, "symlink tree test case: [tmppath/]%s",
 				symlink_tests_inputs[i].input);
 
 		snprintf(tmppath, PATH_MAX, "%s/%s",
-				tree_origin, symlink_tests_inputs[i].input);
+				real_tree_origin, symlink_tests_inputs[i].input);
 		result = utils_expand_path(tmppath);
 		ok(result != NULL && strcmp(result + treelen,
 					symlink_tests_inputs[i].expected_result) == 0, name);
@@ -312,9 +321,12 @@ static void test_utils_expand_path(void)
 
 	/* Test invalid cases */
 	for (i = 0; i < num_invalid_tests; i++) {
-		sprintf(name, "invalid test case: %s", invalid_tests_inputs[i]);
+		const char *test_input = invalid_tests_inputs[i];
+
+		sprintf(name, "invalid test case: %s", test_input ?
+				test_input : "NULL");
 
-		result = utils_expand_path(invalid_tests_inputs[i]);
+		result = utils_expand_path(test_input);
 		if (result != NULL) {
 			free(result);
 		}