tests: fix for trailing slashes

This commit is contained in:
Jonathan Goren 2021-11-15 10:40:54 +02:00
parent a26bd3232c
commit 1153e3e155

View file

@ -72,7 +72,7 @@ fn test_simple() {
./one/two/c.foo ./one/two/c.foo
./one/two/C.Foo2 ./one/two/C.Foo2
./one/two/three/d.foo ./one/two/three/d.foo
./one/two/three/directory_foo", ./one/two/three/directory_foo/",
); );
} }
@ -82,14 +82,14 @@ fn test_empty_pattern() {
let te = TestEnv::new(DEFAULT_DIRS, DEFAULT_FILES); let te = TestEnv::new(DEFAULT_DIRS, DEFAULT_FILES);
let expected = "./a.foo let expected = "./a.foo
./e1 e2 ./e1 e2
./one ./one/
./one/b.foo ./one/b.foo
./one/two ./one/two/
./one/two/c.foo ./one/two/c.foo
./one/two/C.Foo2 ./one/two/C.Foo2
./one/two/three ./one/two/three/
./one/two/three/d.foo ./one/two/three/d.foo
./one/two/three/directory_foo ./one/two/three/directory_foo/
./symlink"; ./symlink";
te.assert_output(&["--regex"], expected); te.assert_output(&["--regex"], expected);
@ -171,24 +171,24 @@ fn test_explicit_root_path() {
one/two/c.foo one/two/c.foo
one/two/C.Foo2 one/two/C.Foo2
one/two/three/d.foo one/two/three/d.foo
one/two/three/directory_foo", one/two/three/directory_foo/",
); );
te.assert_output( te.assert_output(
&["foo", "one/two/three"], &["foo", "one/two/three"],
"one/two/three/d.foo "one/two/three/d.foo
one/two/three/directory_foo", one/two/three/directory_foo/",
); );
te.assert_output_subdirectory( te.assert_output_subdirectory(
"one/two", "one/two/",
&["foo", "../../"], &["foo", "../../"],
"../../a.foo "../../a.foo
../../one/b.foo ../../one/b.foo
../../one/two/c.foo ../../one/two/c.foo
../../one/two/C.Foo2 ../../one/two/C.Foo2
../../one/two/three/d.foo ../../one/two/three/d.foo
../../one/two/three/directory_foo", ../../one/two/three/directory_foo/",
); );
te.assert_output_subdirectory( te.assert_output_subdirectory(
@ -196,9 +196,9 @@ fn test_explicit_root_path() {
&["", ".."], &["", ".."],
"../c.foo "../c.foo
../C.Foo2 ../C.Foo2
../three ../three/
../three/d.foo ../three/d.foo
../three/directory_foo", ../three/directory_foo/",
); );
} }
@ -387,7 +387,7 @@ fn test_full_path() {
&format!("^{prefix}.*three.*foo$", prefix = prefix), &format!("^{prefix}.*three.*foo$", prefix = prefix),
], ],
"./one/two/three/d.foo "./one/two/three/d.foo
./one/two/three/directory_foo", ./one/two/three/directory_foo/",
); );
} }
@ -404,7 +404,7 @@ fn test_hidden() {
./one/two/c.foo ./one/two/c.foo
./one/two/C.Foo2 ./one/two/C.Foo2
./one/two/three/d.foo ./one/two/three/d.foo
./one/two/three/directory_foo", ./one/two/three/directory_foo/",
); );
} }
@ -444,7 +444,7 @@ fn test_no_ignore() {
./one/two/c.foo ./one/two/c.foo
./one/two/C.Foo2 ./one/two/C.Foo2
./one/two/three/d.foo ./one/two/three/d.foo
./one/two/three/directory_foo", ./one/two/three/directory_foo/",
); );
te.assert_output( te.assert_output(
@ -457,7 +457,7 @@ fn test_no_ignore() {
./one/two/c.foo ./one/two/c.foo
./one/two/C.Foo2 ./one/two/C.Foo2
./one/two/three/d.foo ./one/two/three/d.foo
./one/two/three/directory_foo", ./one/two/three/directory_foo/",
); );
} }
@ -610,7 +610,7 @@ fn test_no_ignore_vcs() {
./one/two/c.foo ./one/two/c.foo
./one/two/C.Foo2 ./one/two/C.Foo2
./one/two/three/d.foo ./one/two/three/d.foo
./one/two/three/directory_foo", ./one/two/three/directory_foo/",
); );
} }
@ -647,7 +647,7 @@ fn test_no_ignore_aliases() {
./one/two/c.foo ./one/two/c.foo
./one/two/C.Foo2 ./one/two/C.Foo2
./one/two/three/d.foo ./one/two/three/d.foo
./one/two/three/directory_foo", ./one/two/three/directory_foo/",
); );
te.assert_output( te.assert_output(
@ -660,7 +660,7 @@ fn test_no_ignore_aliases() {
./one/two/c.foo ./one/two/c.foo
./one/two/C.Foo2 ./one/two/C.Foo2
./one/two/three/d.foo ./one/two/three/d.foo
./one/two/three/directory_foo", ./one/two/three/directory_foo/",
); );
} }
@ -759,7 +759,7 @@ fn test_print0() {
./one/two/C.Foo2NULL ./one/two/C.Foo2NULL
./one/two/c.fooNULL ./one/two/c.fooNULL
./one/two/three/d.fooNULL ./one/two/three/d.fooNULL
./one/two/three/directory_fooNULL", ./one/two/three/directory_foo/NULL",
); );
} }
@ -772,12 +772,12 @@ fn test_max_depth() {
&["--max-depth", "3"], &["--max-depth", "3"],
"./a.foo "./a.foo
./e1 e2 ./e1 e2
./one ./one/
./one/b.foo ./one/b.foo
./one/two ./one/two/
./one/two/c.foo ./one/two/c.foo
./one/two/C.Foo2 ./one/two/C.Foo2
./one/two/three ./one/two/three/
./symlink", ./symlink",
); );
@ -785,9 +785,9 @@ fn test_max_depth() {
&["--max-depth", "2"], &["--max-depth", "2"],
"./a.foo "./a.foo
./e1 e2 ./e1 e2
./one ./one/
./one/b.foo ./one/b.foo
./one/two ./one/two/
./symlink", ./symlink",
); );
@ -795,7 +795,7 @@ fn test_max_depth() {
&["--max-depth", "1"], &["--max-depth", "1"],
"./a.foo "./a.foo
./e1 e2 ./e1 e2
./one ./one/
./symlink", ./symlink",
); );
} }
@ -809,15 +809,15 @@ fn test_min_depth() {
&["--min-depth", "3"], &["--min-depth", "3"],
"./one/two/c.foo "./one/two/c.foo
./one/two/C.Foo2 ./one/two/C.Foo2
./one/two/three ./one/two/three/
./one/two/three/d.foo ./one/two/three/d.foo
./one/two/three/directory_foo", ./one/two/three/directory_foo/",
); );
te.assert_output( te.assert_output(
&["--min-depth", "4"], &["--min-depth", "4"],
"./one/two/three/d.foo "./one/two/three/d.foo
./one/two/three/directory_foo", ./one/two/three/directory_foo/",
); );
} }
@ -830,7 +830,7 @@ fn test_exact_depth() {
&["--exact-depth", "3"], &["--exact-depth", "3"],
"./one/two/c.foo "./one/two/c.foo
./one/two/C.Foo2 ./one/two/C.Foo2
./one/two/three", ./one/two/three/",
); );
} }
@ -850,19 +850,19 @@ fn test_prune() {
te.assert_output( te.assert_output(
&["foo"], &["foo"],
"./foo "./foo/
./foo/foo.file ./foo/foo.file
./foo/bar/foo.file ./foo/bar/foo.file
./bar/foo.file ./bar/foo.file
./bar/foo ./bar/foo/
./bar/foo/foo.file ./bar/foo/foo.file
./baz/foo.file", ./baz/foo.file",
); );
te.assert_output( te.assert_output(
&["--prune", "foo"], &["--prune", "foo"],
"./foo "./foo/
./bar/foo ./bar/foo/
./bar/foo.file ./bar/foo.file
./baz/foo.file", ./baz/foo.file",
); );
@ -878,14 +878,14 @@ fn test_absolute_path() {
&format!( &format!(
"{abs_path}/a.foo "{abs_path}/a.foo
{abs_path}/e1 e2 {abs_path}/e1 e2
{abs_path}/one {abs_path}/one/
{abs_path}/one/b.foo {abs_path}/one/b.foo
{abs_path}/one/two {abs_path}/one/two/
{abs_path}/one/two/c.foo {abs_path}/one/two/c.foo
{abs_path}/one/two/C.Foo2 {abs_path}/one/two/C.Foo2
{abs_path}/one/two/three {abs_path}/one/two/three/
{abs_path}/one/two/three/d.foo {abs_path}/one/two/three/d.foo
{abs_path}/one/two/three/directory_foo {abs_path}/one/two/three/directory_foo/
{abs_path}/symlink", {abs_path}/symlink",
abs_path = &abs_path abs_path = &abs_path
), ),
@ -899,7 +899,7 @@ fn test_absolute_path() {
{abs_path}/one/two/c.foo {abs_path}/one/two/c.foo
{abs_path}/one/two/C.Foo2 {abs_path}/one/two/C.Foo2
{abs_path}/one/two/three/d.foo {abs_path}/one/two/three/d.foo
{abs_path}/one/two/three/directory_foo", {abs_path}/one/two/three/directory_foo/",
abs_path = &abs_path abs_path = &abs_path
), ),
); );
@ -918,7 +918,7 @@ fn test_implicit_absolute_path() {
{abs_path}/one/two/c.foo {abs_path}/one/two/c.foo
{abs_path}/one/two/C.Foo2 {abs_path}/one/two/C.Foo2
{abs_path}/one/two/three/d.foo {abs_path}/one/two/three/d.foo
{abs_path}/one/two/three/directory_foo", {abs_path}/one/two/three/directory_foo/",
abs_path = &abs_path abs_path = &abs_path
), ),
); );
@ -938,7 +938,7 @@ fn test_normalized_absolute_path() {
{abs_path}/one/two/c.foo {abs_path}/one/two/c.foo
{abs_path}/one/two/C.Foo2 {abs_path}/one/two/C.Foo2
{abs_path}/one/two/three/d.foo {abs_path}/one/two/three/d.foo
{abs_path}/one/two/three/directory_foo", {abs_path}/one/two/three/directory_foo/",
abs_path = &abs_path abs_path = &abs_path
), ),
); );
@ -963,18 +963,18 @@ fn test_type() {
te.assert_output( te.assert_output(
&["--type", "d"], &["--type", "d"],
"./one "./one/
./one/two ./one/two/
./one/two/three ./one/two/three/
./one/two/three/directory_foo", ./one/two/three/directory_foo/",
); );
te.assert_output( te.assert_output(
&["--type", "d", "--type", "l"], &["--type", "d", "--type", "l"],
"./one "./one/
./one/two ./one/two/
./one/two/three ./one/two/three/
./one/two/three/directory_foo ./one/two/three/directory_foo/
./symlink", ./symlink",
); );
@ -1001,10 +1001,10 @@ fn test_type_executable() {
te.assert_output( te.assert_output(
&["--type", "executable", "--type", "directory"], &["--type", "executable", "--type", "directory"],
"./executable-file.sh "./executable-file.sh
./one ./one/
./one/two ./one/two/
./one/two/three ./one/two/three/
./one/two/three/directory_foo", ./one/two/three/directory_foo/",
); );
} }
@ -1021,18 +1021,18 @@ fn test_type_empty() {
te.assert_output( te.assert_output(
&["--type", "empty"], &["--type", "empty"],
"./0_bytes.foo "./0_bytes.foo
./dir_empty", ./dir_empty/",
); );
te.assert_output( te.assert_output(
&["--type", "empty", "--type", "file", "--type", "directory"], &["--type", "empty", "--type", "file", "--type", "directory"],
"./0_bytes.foo "./0_bytes.foo
./dir_empty", ./dir_empty/",
); );
te.assert_output(&["--type", "empty", "--type", "file"], "./0_bytes.foo"); te.assert_output(&["--type", "empty", "--type", "file"], "./0_bytes.foo");
te.assert_output(&["--type", "empty", "--type", "directory"], "./dir_empty"); te.assert_output(&["--type", "empty", "--type", "directory"], "./dir_empty/");
} }
/// File extension (--extension) /// File extension (--extension)
@ -1103,12 +1103,12 @@ fn test_no_extension() {
te.assert_output( te.assert_output(
&["^[^.]+$"], &["^[^.]+$"],
"./aa "./aa
./one ./one/
./one/bb ./one/bb
./one/two ./one/two/
./one/two/three ./one/two/three/
./one/two/three/d ./one/two/three/d
./one/two/three/directory_foo ./one/two/three/directory_foo/
./symlink", ./symlink",
); );
@ -1148,14 +1148,14 @@ fn test_symlink_as_root() {
"{dir}/a.foo "{dir}/a.foo
{dir}/broken_symlink {dir}/broken_symlink
{dir}/e1 e2 {dir}/e1 e2
{dir}/one {dir}/one/
{dir}/one/b.foo {dir}/one/b.foo
{dir}/one/two {dir}/one/two/
{dir}/one/two/c.foo {dir}/one/two/c.foo
{dir}/one/two/C.Foo2 {dir}/one/two/C.Foo2
{dir}/one/two/three {dir}/one/two/three/
{dir}/one/two/three/d.foo {dir}/one/two/three/d.foo
{dir}/one/two/three/directory_foo {dir}/one/two/three/directory_foo/
{dir}/symlink", {dir}/symlink",
dir = &parent_parent dir = &parent_parent
), ),
@ -1174,9 +1174,9 @@ fn test_symlink_and_absolute_path() {
&format!( &format!(
"{abs_path}/{expected_path}/c.foo "{abs_path}/{expected_path}/c.foo
{abs_path}/{expected_path}/C.Foo2 {abs_path}/{expected_path}/C.Foo2
{abs_path}/{expected_path}/three {abs_path}/{expected_path}/three/
{abs_path}/{expected_path}/three/d.foo {abs_path}/{expected_path}/three/d.foo
{abs_path}/{expected_path}/three/directory_foo", {abs_path}/{expected_path}/three/directory_foo/",
abs_path = &abs_path, abs_path = &abs_path,
expected_path = expected_path expected_path = expected_path
), ),
@ -1192,9 +1192,9 @@ fn test_symlink_as_absolute_root() {
&format!( &format!(
"{abs_path}/symlink/c.foo "{abs_path}/symlink/c.foo
{abs_path}/symlink/C.Foo2 {abs_path}/symlink/C.Foo2
{abs_path}/symlink/three {abs_path}/symlink/three/
{abs_path}/symlink/three/d.foo {abs_path}/symlink/three/d.foo
{abs_path}/symlink/three/directory_foo", {abs_path}/symlink/three/directory_foo/",
abs_path = &abs_path abs_path = &abs_path
), ),
); );
@ -1216,9 +1216,9 @@ fn test_symlink_and_full_path() {
&format!("^{prefix}.*three", prefix = prefix), &format!("^{prefix}.*three", prefix = prefix),
], ],
&format!( &format!(
"{abs_path}/{expected_path}/three "{abs_path}/{expected_path}/three/
{abs_path}/{expected_path}/three/d.foo {abs_path}/{expected_path}/three/d.foo
{abs_path}/{expected_path}/three/directory_foo", {abs_path}/{expected_path}/three/directory_foo/",
abs_path = &abs_path, abs_path = &abs_path,
expected_path = expected_path expected_path = expected_path
), ),
@ -1237,9 +1237,9 @@ fn test_symlink_and_full_path_abs_path() {
&format!("{abs_path}/symlink", abs_path = abs_path), &format!("{abs_path}/symlink", abs_path = abs_path),
], ],
&format!( &format!(
"{abs_path}/symlink/three "{abs_path}/symlink/three/
{abs_path}/symlink/three/d.foo {abs_path}/symlink/three/d.foo
{abs_path}/symlink/three/directory_foo", {abs_path}/symlink/three/directory_foo/",
abs_path = &abs_path abs_path = &abs_path
), ),
); );
@ -1251,32 +1251,32 @@ fn test_excludes() {
te.assert_output( te.assert_output(
&["--exclude", "*.foo"], &["--exclude", "*.foo"],
"./one "./one/
./one/two ./one/two/
./one/two/C.Foo2 ./one/two/C.Foo2
./one/two/three ./one/two/three/
./one/two/three/directory_foo ./one/two/three/directory_foo/
./e1 e2 ./e1 e2
./symlink", ./symlink",
); );
te.assert_output( te.assert_output(
&["--exclude", "*.foo", "--exclude", "*.Foo2"], &["--exclude", "*.foo", "--exclude", "*.Foo2"],
"./one "./one/
./one/two ./one/two/
./one/two/three ./one/two/three/
./one/two/three/directory_foo ./one/two/three/directory_foo/
./e1 e2 ./e1 e2
./symlink", ./symlink",
); );
te.assert_output( te.assert_output(
&["--exclude", "*.foo", "--exclude", "*.Foo2", "foo"], &["--exclude", "*.foo", "--exclude", "*.Foo2", "foo"],
"./one/two/three/directory_foo", "./one/two/three/directory_foo/",
); );
te.assert_output( te.assert_output(
&["--exclude", "one/two", "foo"], &["--exclude", "one/two/", "foo"],
"./a.foo "./a.foo
./one/b.foo", ./one/b.foo",
); );
@ -1285,11 +1285,11 @@ fn test_excludes() {
&["--exclude", "one/**/*.foo"], &["--exclude", "one/**/*.foo"],
"./a.foo "./a.foo
./e1 e2 ./e1 e2
./one ./one/
./one/two ./one/two/
./one/two/C.Foo2 ./one/two/C.Foo2
./one/two/three ./one/two/three/
./one/two/three/directory_foo ./one/two/three/directory_foo/
./symlink", ./symlink",
); );
} }
@ -1786,7 +1786,7 @@ fn test_custom_path_separator() {
one=two=c.foo one=two=c.foo
one=two=C.Foo2 one=two=C.Foo2
one=two=three=d.foo one=two=three=d.foo
one=two=three=directory_foo", one=two=three=directory_foo=",
); );
} }
@ -1797,20 +1797,20 @@ fn test_base_directory() {
te.assert_output( te.assert_output(
&["--base-directory", "one"], &["--base-directory", "one"],
"./b.foo "./b.foo
./two ./two/
./two/c.foo ./two/c.foo
./two/C.Foo2 ./two/C.Foo2
./two/three ./two/three/
./two/three/d.foo ./two/three/d.foo
./two/three/directory_foo", ./two/three/directory_foo/",
); );
te.assert_output( te.assert_output(
&["--base-directory", "one/two", "foo"], &["--base-directory", "one/two/", "foo"],
"./c.foo "./c.foo
./C.Foo2 ./C.Foo2
./three/d.foo ./three/d.foo
./three/directory_foo", ./three/directory_foo/",
); );
// Explicit root path // Explicit root path
@ -1819,12 +1819,12 @@ fn test_base_directory() {
"two/c.foo "two/c.foo
two/C.Foo2 two/C.Foo2
two/three/d.foo two/three/d.foo
two/three/directory_foo", two/three/directory_foo/",
); );
// Ignore base directory when absolute path is used // Ignore base directory when absolute path is used
let (te, abs_path) = get_test_env_with_abs_path(DEFAULT_DIRS, DEFAULT_FILES); let (te, abs_path) = get_test_env_with_abs_path(DEFAULT_DIRS, DEFAULT_FILES);
let abs_base_dir = &format!("{abs_path}/one/two", abs_path = &abs_path); let abs_base_dir = &format!("{abs_path}/one/two/", abs_path = &abs_path);
te.assert_output( te.assert_output(
&["--base-directory", abs_base_dir, "foo", &abs_path], &["--base-directory", abs_base_dir, "foo", &abs_path],
&format!( &format!(
@ -1833,7 +1833,7 @@ fn test_base_directory() {
{abs_path}/one/two/c.foo {abs_path}/one/two/c.foo
{abs_path}/one/two/C.Foo2 {abs_path}/one/two/C.Foo2
{abs_path}/one/two/three/d.foo {abs_path}/one/two/three/d.foo
{abs_path}/one/two/three/directory_foo", {abs_path}/one/two/three/directory_foo/",
abs_path = &abs_path abs_path = &abs_path
), ),
); );
@ -1989,14 +1989,14 @@ fn test_strip_cwd_prefix() {
&["--strip-cwd-prefix", "."], &["--strip-cwd-prefix", "."],
"a.foo "a.foo
e1 e2 e1 e2
one one/
one/b.foo one/b.foo
one/two one/two/
one/two/c.foo one/two/c.foo
one/two/C.Foo2 one/two/C.Foo2
one/two/three one/two/three/
one/two/three/d.foo one/two/three/d.foo
one/two/three/directory_foo one/two/three/directory_foo/
symlink", symlink",
); );
} }