Fix problems in path normalisation algorithm (#1541)

Current implementation of path normalisation contains various flaws:

- Trailing `.` and `..` at the end of path don't get normalised,
- Path `/` turns into an empty string after normalisation.

This patch changes implementation of normalisation to fix these.

1 files changed, 8 insertions, 0 deletions

diff --git a/spec/dir_spec.lua b/spec/dir_spec.lua
index 9f05c664..f6ef172e 100644
--- a/spec/dir_spec.lua
+++ b/spec/dir_spec.lua
@@ -52,12 +52,20 @@ describe("luarocks.dir #unit", function()
       it("strips unneeded /../ and /./", function()
          assert.are.same("/some/dir/file.txt", dir.normalize("/../../../some/./foo/bar/.././../dir/bla/../file.txt"))
          assert.are.same("/some/dir/file.txt", dir.normalize("/../../../some/./foo/bar/.././../dir/bla/../file.txt"))
+         assert.are.same("/some/dir", dir.normalize("/../../../some/./foo/bar/.././../dir/./some/subdir/../.."))
+         assert.are.same("/some/dir", dir.normalize("/../../../some/./foo/bar/.././../dir/./."))
       end)
       it("respects relative paths", function()
+         assert.are.same(".", dir.normalize("."))
          assert.are.same("boo", dir.normalize("./boo"))
          assert.are.same("/boo", dir.normalize("/./boo"))
          assert.are.same("../../../../boo", dir.normalize("../../../hello/world/../../../boo"))
       end)
+      it("respects root directory", function()
+         assert.are.same("/", dir.normalize("/"))
+         assert.are.same("/", dir.normalize("/////"))
+         assert.are.same("/", dir.normalize("/a/b/.././../c/./../../"))
+      end)
    end)
 
 end)