fix(fs): make current_dir always return 1 arg only

1 files changed, 253 insertions, 0 deletions

diff --git a/spec/unit/tools_spec.lua b/spec/unit/tools_spec.lua
new file mode 100644
index 00000000..29e21740
--- /dev/null
+++ b/spec/unit/tools_spec.lua
@@ -0,0 +1,253 @@
+local test_env = require("spec.util.test_env")
+local get_tmp_path = test_env.get_tmp_path
+local testing_paths = test_env.testing_paths
+local write_file = test_env.write_file
+
+test_env.unload_luarocks()
+local fs = require("luarocks.fs")
+local cfg = require("luarocks.core.cfg")
+local patch = require("luarocks.tools.patch")
+
+local lao =
+[[The Nameless is the origin of Heaven and Earth;
+The named is the mother of all things.
+
+Therefore let there always be non-being,
+  so we may see their subtlety,
+And let there always be being,
+  so we may see their outcome.
+The two are the same,
+But after they are produced,
+  they have different names.
+They both may be called deep and profound.
+Deeper and more profound,
+The door of all subtleties!]]
+
+local tzu =
+[[The Way that can be told of is not the eternal Way;
+The name that can be named is not the eternal name.
+The Nameless is the origin of Heaven and Earth;
+The Named is the mother of all things.
+Therefore let there always be non-being,
+  so we may see their subtlety,
+And let there always be being,
+  so we may see their outcome.
+The two are the same,
+But after they are produced,
+  they have different names.]]
+
+local valid_patch1 =
+[[--- lao       2002-02-21 23:30:39.942229878 -0800
++++ tzu 2002-02-21 23:30:50.442260588 -0800
+@@ -1,7 +1,6 @@
+-The Way that can be told of is not the eternal Way;
+-The name that can be named is not the eternal name.
+ The Nameless is the origin of Heaven and Earth;
+-The Named is the mother of all things.
++The named is the mother of all things.
++
+ Therefore let there always be non-being,
+   so we may see their subtlety,
+ And let there always be being,
+@@ -9,3 +8,6 @@
+ The two are the same,
+ But after they are produced,
+   they have different names.
++They both may be called deep and profound.
++Deeper and more profound,
++The door of all subtleties!]]
+
+local valid_patch2 =
+[[--- /dev/null 1969-02-21 23:30:39.942229878 -0800
++++ tzu 2002-02-21 23:30:50.442260588 -0800
+@@ -1,7 +1,6 @@
+-The Way that can be told of is not the eternal Way;
+-The name that can be named is not the eternal name.
+ The Nameless is the origin of Heaven and Earth;
+-The Named is the mother of all things.
++The named is the mother of all things.
++
+ Therefore let there always be non-being,
+   so we may see their subtlety,
+ And let there always be being,
+@@ -9,3 +8,6 @@
+ The two are the same,
+ But after they are produced,
+   they have different names.
++They both may be called deep and profound.
++Deeper and more profound,
++The door of all subtleties!]]
+
+local invalid_patch1 =
+[[--- lao       2002-02-21 23:30:39.942229878 -0800
++++ tzu 2002-02-21 23:30:50.442260588 -0800
+@@ -1,7 +1,6 @@
+-The Way that can be told of is not the eternal Way;
+-The name that can be named is not the eternal name.
+ The Nameless is the origin of Heaven and Earth;
+-The Named is the mother of all things.
+--- Extra
++The named is the mother of all things.
++
+ Therefore let there always be non-being,
+   so we may see their subtlety,
+ And let there always be being,
+--- Extra
+@@ -9,3 +8,7 @@
+ The two are the same,
+ But after they are produced,
+   they have different names.
++They both may be called deep and profound.
++Deeper and more profound,
++The door of all subtleties!]]
+
+local invalid_patch2 =
+[[--- lao       2002-02-21 23:30:39.942229878 -0800
++++   tzu       2002-02-21 23:30:50.442260588 -0800
+@@ -1,7 +1,6 @@
+-The Way that can be told of is not the eternal Way;
+-The name that can be named is not the eternal name.
+ The Nameless is the origin of Heaven and Earth;
+-The Named is the mother of all things.
++The named is the mother of all things.
++
+ Therefore let there always be non-being,
+   so we may see their subtlety,
+ And let there always be being,
+@@ -9,3 +8,6 @@
+ The two are the same,
+ But after they are produced,
+   they have different names.
++They both may be called deep and profound.
++Deeper and more profound,
+? ...
++The door of all subtleties!]]
+
+local invalid_patch3 =
+[[---     lao   2002-02-21 23:30:39.942229878 -0800
++++ tzu 2002-02-21 23:30:50.442260588 -0800
+@@ -1,7 +1,6 @@
+-The Way that can be told of is not the eternal Way;
+-The name that can be named is not the eternal name.
+ The Nameless is the origin of Heaven and Earth;
+-The Named is the mother of all things.
++The named is the mother of all things.
++
+ Therefore let there always be non-being,
+   so we may see their subtlety,
+ And let there always be being,
+@@ -9,3 +8,6 @@
+ The two are the same,
+ But after they are produced,
+   they have different names.
++They both may be called deep and profound.
++Deeper and more profound,
+? ...
++The door of all subtleties!]]
+
+describe("Luarocks patch test #unit", function()
+   local runner
+
+   setup(function()
+      cfg.init()
+      fs.init()
+      runner = require("luacov.runner")
+      runner.init(testing_paths.testrun_dir .. "/luacov.config")
+      runner.tick = true
+   end)
+
+   teardown(function()
+      runner.shutdown()
+   end)
+
+   describe("patch.read_patch", function()
+      it("returns a table with the patch file info and the result of parsing the file", function()
+         local t, result
+
+         write_file("test.patch", valid_patch1, finally)
+         t, result = patch.read_patch("test.patch")
+         assert.truthy(result)
+         assert.truthy(t)
+
+         write_file("test.patch", invalid_patch1, finally)
+         t, result = patch.read_patch("test.patch")
+         assert.falsy(result)
+         assert.truthy(t)
+
+         write_file("test.patch", invalid_patch2, finally)
+         t, result = patch.read_patch("test.patch")
+         assert.falsy(result)
+         assert.truthy(t)
+
+         write_file("test.patch", invalid_patch3, finally)
+         t, result = patch.read_patch("test.patch")
+         assert.falsy(result)
+         assert.truthy(t)
+      end)
+   end)
+
+   describe("patch.apply_patch", function()
+      local tmpdir
+      local olddir
+
+      before_each(function()
+         tmpdir = get_tmp_path()
+         olddir = lfs.currentdir()
+         lfs.mkdir(tmpdir)
+         lfs.chdir(tmpdir)
+
+         write_file("lao", tzu, finally)
+         write_file("tzu", lao, finally)
+      end)
+
+      after_each(function()
+         if olddir then
+            lfs.chdir(olddir)
+            if tmpdir then
+               lfs.rmdir(tmpdir)
+            end
+         end
+      end)
+
+      it("applies the given patch and returns the result of patching", function()
+         write_file("test.patch", valid_patch1, finally)
+         local p = patch.read_patch("test.patch")
+         local result = patch.apply_patch(p)
+         assert.truthy(result)
+      end)
+
+      it("applies the given patch with custom arguments and returns the result of patching", function()
+         write_file("test.patch", valid_patch2, finally)
+         local p = patch.read_patch("test.patch")
+         local result = patch.apply_patch(p, nil, true)
+         assert.truthy(result)
+      end)
+
+      it("fails if the patch file is invalid", function()
+         write_file("test.patch", invalid_patch1, finally)
+         local p = patch.read_patch("test.patch")
+         local result = pcall(patch.apply_patch, p)
+         assert.falsy(result)
+      end)
+
+      it("returns false if the files from the patch doesn't exist", function()
+         os.remove("lao")
+         os.remove("tzu")
+
+         write_file("test.patch", valid_patch1, finally)
+         local p = patch.read_patch("test.patch")
+         local result = patch.apply_patch(p)
+         assert.falsy(result)
+      end)
+
+      it("returns false if the target file was already patched", function()
+         write_file("test.patch", valid_patch1, finally)
+         local p = patch.read_patch("test.patch")
+         local result = patch.apply_patch(p)
+         assert.truthy(result)
+
+         result = patch.apply_patch(p)
+         assert.falsy(result)
+      end)
+   end)
+end)