From 0297e9db976aa2c3f17e237c6c5c37f3fde454a9 Mon Sep 17 00:00:00 2001
From: dzwdz
Date: Sun, 8 Aug 2021 23:23:27 +0200
Subject: path_simplify now returns int, has better tests

---
 src/kernel/tests/vfs.c | 57 ++++++++++++++++++++++++++++++++------------------
 1 file changed, 37 insertions(+), 20 deletions(-)

(limited to 'src/kernel/tests')

diff --git a/src/kernel/tests/vfs.c b/src/kernel/tests/vfs.c
index a5a6848..017f36d 100644
--- a/src/kernel/tests/vfs.c
+++ b/src/kernel/tests/vfs.c
@@ -3,31 +3,48 @@
 #include <kernel/vfs/path.h>
 
 TEST(path_simplify) {
-	char buf[256];
+#define TEST_WRAPPER(argument, result) do { \
+		int len = path_simplify(argument, buf, sizeof(argument) - 1); \
+		if (result == 0) { \
+			TEST_COND(len < 0); \
+		} else { \
+			TEST_COND(len > 0); \
+			/* TODO check equality */ \
+		} \
+	} while (0)
 
-	// some easy valid cases
-	TEST_COND( path_simplify("/asdf", buf, 5));
-	TEST_COND( path_simplify("/asd/", buf, 5));
-	TEST_COND( path_simplify("/a/./", buf, 5));
-	TEST_COND( path_simplify("/a/..", buf, 5));
-	TEST_COND( path_simplify("/a//.", buf, 5));
+	char buf[256];
 
-	// .. going under the root or close to it
-	TEST_COND(!path_simplify("/../123456", buf, 10));
-	TEST_COND(!path_simplify("/./a/../..", buf, 10));
-	TEST_COND( path_simplify("/a/a/../..", buf, 10));
-	TEST_COND(!path_simplify("/////../..", buf, 10));
-	TEST_COND(!path_simplify("//a//../..", buf, 10));
+	// some easy cases first
+	TEST_WRAPPER("/",         "/");
+	TEST_WRAPPER("/asdf",     "/asdf");
+	TEST_WRAPPER("/asdf/",    "/asdf/");
+	TEST_WRAPPER("/asdf//",   "/asdf/");
+	TEST_WRAPPER("/asdf/./",  "/asdf/");
+	TEST_WRAPPER("/a/./b",    "/a/b");
+	TEST_WRAPPER("/a/./b/",   "/a/b/");
 
-	// relative paths aren't allowed
-	TEST_COND(!path_simplify("apath", buf, 5));
-	TEST_COND(!path_simplify("a/pth", buf, 5));
-	TEST_COND(!path_simplify("../th", buf, 5));
+	// some slightly less easy cases
+	TEST_WRAPPER("/asdf/..",  "/");
+	TEST_WRAPPER("/asdf/../", "/");
+	TEST_WRAPPER("/asdf/.",   "/asdf/");
+	TEST_WRAPPER("/asdf//.",  "/asdf/");
 
-	// this includes empty paths
-	TEST_COND(!path_simplify("", buf, 1));
+	// going under the root or close to it
+	TEST_WRAPPER("/../asdf",   0);
+	TEST_WRAPPER("/../asdf/",  0);
+	TEST_WRAPPER("/./a/../..", 0);
+	TEST_WRAPPER("/a/a/../..", "/");
+	TEST_WRAPPER("/////../..", 0);
+	TEST_WRAPPER("//a//../..", 0);
 
-	// TODO test if the paths are simplified correctly
+	// relative paths aren't allowed
+	TEST_WRAPPER("relative",   0);
+	TEST_WRAPPER("some/stuff", 0);
+	TEST_WRAPPER("./stuff",    0);
+	TEST_WRAPPER("../stuff",   0);
+	TEST_WRAPPER("",           0);
+#undef TEST_WRAPPER
 }
 
 void tests_vfs() {
-- 
cgit v1.2.3