From f03c738a9e39053ad185084bf83bf6e526011b71 Mon Sep 17 00:00:00 2001 From: Joel Klinghed Date: Wed, 25 Sep 2024 22:16:26 +0200 Subject: samba & sftp: Share path utils methods Add tests for them, and add basename, currently unused, for completeness --- .../main/java/org/the_jk/cleversync/PathUtils.kt | 49 ++++++++++++++++++ .../java/org/the_jk/cleversync/PathUtilsTest.kt | 58 ++++++++++++++++++++++ 2 files changed, 107 insertions(+) create mode 100644 libs/utils/src/main/java/org/the_jk/cleversync/PathUtils.kt create mode 100644 libs/utils/src/test/java/org/the_jk/cleversync/PathUtilsTest.kt (limited to 'libs/utils') diff --git a/libs/utils/src/main/java/org/the_jk/cleversync/PathUtils.kt b/libs/utils/src/main/java/org/the_jk/cleversync/PathUtils.kt new file mode 100644 index 0000000..4202b24 --- /dev/null +++ b/libs/utils/src/main/java/org/the_jk/cleversync/PathUtils.kt @@ -0,0 +1,49 @@ +package org.the_jk.cleversync + +object PathUtils { + fun dirname(path: String): String { + if (path.isEmpty()) return "." + var start = path.lastIndex + while (start > -1 && path[start] == '/') start-- + if (start == -1) return "/" + val index = path.lastIndexOf('/', startIndex = start) + if (index == 0) return "/" + if (index > -1) return path.substring(0, index) + return "." + } + + fun basename(path: String): String { + if (path.isEmpty()) return "" + var start = path.lastIndex + while (start > -1 && path[start] == '/') start-- + if (start == -1) return "/" + val index = path.lastIndexOf('/', startIndex = start) + return path.substring(index + 1, start + 1) + } + + fun join(path1: String, path2: String): String { + if (path1.isEmpty() || path2.startsWith("/")) return path2 + if (path2.isEmpty()) return path1 + return if (path1.endsWith("/")) path1 + path2 else "${path1}/${path2}" + } + + fun resolve(path: String): String { + if (path.isEmpty()) return "" + val prefixed = path.startsWith("/") + val parts = path.split('/').filter { part -> + part.isNotEmpty() && part != "." + }.toMutableList() + var i = 0 + while (i < parts.size) { + if (parts[i] == "..") { + parts.removeAt(i) + if (i > 0) { + parts.removeAt(i - 1) + } + } else { + i++ + } + } + return parts.joinToString("/", prefix = if (prefixed) "/" else "") + } +} diff --git a/libs/utils/src/test/java/org/the_jk/cleversync/PathUtilsTest.kt b/libs/utils/src/test/java/org/the_jk/cleversync/PathUtilsTest.kt new file mode 100644 index 0000000..eb1d63f --- /dev/null +++ b/libs/utils/src/test/java/org/the_jk/cleversync/PathUtilsTest.kt @@ -0,0 +1,58 @@ +package org.the_jk.cleversync + +import com.google.common.truth.Truth.assertThat +import org.junit.Test + +class PathUtilsTest { + @Test + fun dirname() { + assertThat(PathUtils.dirname("")).isEqualTo(".") + assertThat(PathUtils.dirname("foo")).isEqualTo(".") + assertThat(PathUtils.dirname("foo/")).isEqualTo(".") + assertThat(PathUtils.dirname("foo/bar")).isEqualTo("foo") + assertThat(PathUtils.dirname("/")).isEqualTo("/") + assertThat(PathUtils.dirname("//")).isEqualTo("/") + assertThat(PathUtils.dirname("/foo")).isEqualTo("/") + assertThat(PathUtils.dirname("/foo/")).isEqualTo("/") + } + + @Test + fun basename() { + assertThat(PathUtils.basename("")).isEmpty() + assertThat(PathUtils.basename("foo")).isEqualTo("foo") + assertThat(PathUtils.basename("foo/")).isEqualTo("foo") + assertThat(PathUtils.basename("foo/bar")).isEqualTo("bar") + assertThat(PathUtils.basename("/")).isEqualTo("/") + assertThat(PathUtils.basename("//")).isEqualTo("/") + assertThat(PathUtils.basename("/foo")).isEqualTo("foo") + assertThat(PathUtils.basename("/foo/")).isEqualTo("foo") + } + + @Test + fun join() { + assertThat(PathUtils.join("", "")).isEmpty() + assertThat(PathUtils.join("/", "")).isEqualTo("/") + assertThat(PathUtils.join("", "/")).isEqualTo("/") + assertThat(PathUtils.join("/", "/")).isEqualTo("/") + assertThat(PathUtils.join("foo", "bar")).isEqualTo("foo/bar") + assertThat(PathUtils.join("foo/", "bar")).isEqualTo("foo/bar") + assertThat(PathUtils.join("foo", "/bar")).isEqualTo("/bar") + assertThat(PathUtils.join("/foo", "/bar")).isEqualTo("/bar") + assertThat(PathUtils.join("/foo", "bar/")).isEqualTo("/foo/bar/") + assertThat(PathUtils.join("/foo/", "bar/")).isEqualTo("/foo/bar/") + } + + @Test + fun resolve() { + assertThat(PathUtils.resolve("")).isEmpty() + assertThat(PathUtils.resolve("/")).isEqualTo("/") + assertThat(PathUtils.resolve("../foo")).isEqualTo("foo") + assertThat(PathUtils.resolve("/../foo")).isEqualTo("/foo") + assertThat(PathUtils.resolve("foo/../bar")).isEqualTo("bar") + assertThat(PathUtils.resolve("/foo/../bar")).isEqualTo("/bar") + assertThat(PathUtils.resolve("foo/./bar")).isEqualTo("foo/bar") + assertThat(PathUtils.resolve("/foo/./bar")).isEqualTo("/foo/bar") + assertThat(PathUtils.resolve("foo/../../../bar/")).isEqualTo("bar") + assertThat(PathUtils.resolve("/foo/../../../bar/")).isEqualTo("/bar") + } +} -- cgit v1.2.3-70-g09d2