Fix setOutput to honor passed in variable rather than get from stack.

Author: scr-oath

log/api.go

@@ -58,7 +58,7 @@ func setOutput(L *lua.LState, logger *luaLogger, output string) error {
 	case "STDERR":
 		logger.SetOutput(os.Stderr)
 	default:
-		fd, err := os.OpenFile(L.CheckString(1), os.O_WRONLY|os.O_APPEND|os.O_CREATE, 0644)
+		fd, err := os.OpenFile(output, os.O_WRONLY|os.O_APPEND|os.O_CREATE, 0644)
 		if err != nil {
 			return err
 		}

log/test/test_api.lua

@@ -1,9 +1,9 @@
 local log = require("log")
 local ioutil = require("ioutil")
 
-os.remove("./test/test.log")
-
 function Test_log(t)
+    os.remove("./test/test.log")
+
     local info, err = log.new("./test/test.log")
     assert(not err, err)
 
@@ -37,3 +37,15 @@ function Test_log(t)
         assert(expect_result == get_result, string.format("expected %s; got %s", expect_result, get_result))
     end)
 end
+
+function TestSetOutput(t)
+    -- Ensure that set_output does not result in an error
+    os.remove("./test/test.log")
+    local info = log.new('STDERR')
+    info:set_output("./test/test.log")
+    info:print("foo bar")
+    info:close()
+    local get_result, err = ioutil.read_file("./test/test.log")
+    assert(not err, err)
+    assert(get_result == 'foo bar\n')
+end