From f122b3bbdf6159ec6c87a435cac3bce381f30c3e Mon Sep 17 00:00:00 2001
From: Makpoc <makpoc@gmail.com>
Date: Tue, 13 Oct 2015 23:35:24 +0300
Subject: [PATCH] Fix failing test (windows) - simulate an error by executing
 stat on a filename with zero-byte in it. Fix cleanup of created files after
 the tests.

---
 config/setup/root_test.go | 22 ++++++++++++++++++----
 1 file changed, 18 insertions(+), 4 deletions(-)

diff --git a/config/setup/root_test.go b/config/setup/root_test.go
index f34e05d2..ff3dad3f 100644
--- a/config/setup/root_test.go
+++ b/config/setup/root_test.go
@@ -7,6 +7,7 @@ import (
 	"path/filepath"
 	"strings"
 	"testing"
+	"runtime"
 )
 
 func TestRoot(t *testing.T) {
@@ -26,10 +27,13 @@ func TestRoot(t *testing.T) {
 	if err != nil {
 		t.Fatalf("BeforeTest: Failed to create temp file for testing! Error was: %v", err)
 	}
-	defer os.Remove(existingFile.Name())
-
-	unaccessiblePath := filepath.Join(existingFile.Name(), "some_name")
+	defer func () {
+		existingFile.Close()
+		os.Remove(existingFile.Name())
+	}()
 
+	unaccessiblePath := getInaccessibleOsDependentPath(existingFile.Name())
+	
 	tests := []struct {
 		input              string
 		shouldErr          bool
@@ -60,8 +64,9 @@ func TestRoot(t *testing.T) {
 	for i, test := range tests {
 		c := NewTestController(test.input)
 		mid, err := Root(c)
+
 		if test.shouldErr && err == nil {
-			t.Errorf("Test %d: Expected error but found nil for input %s", i, test.input)
+			t.Errorf("Test %d: Expected error but found %s for input %s", i, err, test.input)
 		}
 
 		if err != nil {
@@ -97,3 +102,12 @@ func getTempDirPath() (string, error) {
 
 	return tempDir, nil
 }
+
+func getInaccessibleOsDependentPath(file string) string{
+	if runtime.GOOS == "windows"{
+		return filepath.Join("C:", "file\x00name")// 0 byte breaks the lstat syscall
+	} else {
+		// TODO - check if the zero-byte filename works for linux only. If it does - use it instead
+ 	   return filepath.Join(file, "some_name")
+	}
+}
\ No newline at end of file