5 files changed, 151 insertions, 0 deletions

diff --git a/spec/fixtures/invalid_patch.patch b/spec/fixtures/invalid_patch.patch
new file mode 100644
index 00000000..75d05c58
--- /dev/null
+++ b/spec/fixtures/invalid_patch.patch
@@ -0,0 +1,20 @@
+--- 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,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!
diff --git a/spec/fixtures/lao b/spec/fixtures/lao
new file mode 100644
index 00000000..635ef2c4
--- /dev/null
+++ b/spec/fixtures/lao
@@ -0,0 +1,11 @@
+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.
diff --git a/spec/fixtures/tzu b/spec/fixtures/tzu
new file mode 100644
index 00000000..5af88a8f
--- /dev/null
+++ b/spec/fixtures/tzu
@@ -0,0 +1,13 @@
+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!
diff --git a/spec/fixtures/valid_patch.patch b/spec/fixtures/valid_patch.patch
new file mode 100644
index 00000000..042c7d22
--- /dev/null
+++ b/spec/fixtures/valid_patch.patch
@@ -0,0 +1,19 @@
+--- 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!
diff --git a/spec/tools_spec.lua b/spec/tools_spec.lua
new file mode 100644
index 00000000..7717be08
--- /dev/null
+++ b/spec/tools_spec.lua
@@ -0,0 +1,88 @@
+local test_env = require("spec.util.test_env")
+local testing_paths = test_env.testing_paths
+local get_tmp_path = test_env.get_tmp_path
+local write_file = test_env.write_file
+
+test_env.unload_luarocks()
+local fs = require("luarocks.fs")
+local patch = package.loaded["luarocks.tools.patch"]
+
+describe("Luarocks patch test #unit", function()
+   local runner
+   
+   setup(function()
+      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
+         
+         t, result = patch.read_patch(testing_paths.fixtures_dir .. "/valid_patch.patch")
+         assert.truthy(result)
+         
+         t, result = patch.read_patch(testing_paths.fixtures_dir .. "/invalid_patch.patch")
+         assert.falsy(result)
+      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)
+         
+         local fd = assert(io.open(testing_paths.fixtures_dir .. "/lao"))
+         local laocontent = assert(fd:read("*a"))
+         fd:close()
+         write_file("lao", laocontent, finally)
+         
+         fd = assert(io.open(testing_paths.fixtures_dir .. "/tzu"))
+         local tzucontent = assert(fd:read("*a"))
+         fd:close()
+         write_file("tzu", tzucontent, 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 true if the patch is valid", function()
+         local p = patch.read_patch(testing_paths.fixtures_dir .. "/valid_patch.patch")
+         local result = patch.apply_patch(p)
+         assert.truthy(result)
+      end)
+      
+      it("returns false if the files to be patched are not valid or doesn't exist", function()
+         os.remove("lao")
+         os.remove("tzu")
+         local p = patch.read_patch(testing_paths.fixtures_dir .. "/invalid_patch.patch")
+         local result = patch.apply_patch(p)
+         assert.falsy(result)
+      end)
+      
+      it("returns false if the target file is already patched", function()
+         local p = patch.read_patch(testing_paths.fixtures_dir .. "/valid_patch.patch")
+         local result = patch.apply_patch(p)
+         assert.truthy(result)
+         
+         result = patch.apply_patch(p)
+         assert.falsy(result)
+      end)
+   end)
+end)