fix around samePath in testing

Author: Songmu

.github/workflows/test.yaml

@@ -3,7 +3,6 @@ on:
   push:
     branches:
     - "**"
-  pull_request: {}
 jobs:
   test:
     runs-on: ${{ matrix.os }}

cmd_root_test.go

@@ -12,13 +12,7 @@ import (
 )
 
 func samePath(lhs, rhs string) bool {
-	if runtime.GOOS != "windows" {
-		return lhs == rhs
-	}
-
-	lhs, _ = filepath.Abs(filepath.Clean(lhs))
-	rhs, _ = filepath.Abs(filepath.Clean(rhs))
-	return strings.EqualFold(lhs, rhs)
+	return lhs == rhs
 }
 
 func samePaths(lhs, rhs string) bool {
@@ -27,12 +21,7 @@ func samePaths(lhs, rhs string) bool {
 	}
 	lhss := strings.Split(lhs, "\n")
 	rhss := strings.Split(rhs, "\n")
-	for i := range lhss {
-		if !samePath(lhss[i], rhss[i]) {
-			return false
-		}
-	}
-	return true
+	return samePathSlice(lhss, rhss)
 }
 
 func TestDoRoot(t *testing.T) {

local_repository_test.go

@@ -6,17 +6,31 @@ import (
 	"reflect"
 	"runtime"
 	"sort"
+	"strings"
 	"sync"
 	"testing"
 
 	"github.com/Songmu/gitconfig"
 )
 
 func samePathSlice(lhss, rhss []string) bool {
-	sort.Strings(lhss)
-	sort.Strings(rhss)
-	for i := range lhss {
-		if !samePath(lhss[i], rhss[i]) {
+	if len(lhss) != len(rhss) {
+		return false
+	}
+	lhssAbs := make([]string, len(lhss))
+	rhssAbs := make([]string, len(rhss))
+
+	for i, p := range lhss {
+		lhsAbs, _ := filepath.Abs(filepath.Clean(p))
+		lhssAbs[i] = strings.ToLower(lhsAbs)
+
+		rhsAbs, _ := filepath.Abs(filepath.Clean(rhss[i]))
+		rhssAbs[i] = strings.ToLower(rhsAbs)
+	}
+	sort.Strings(lhssAbs)
+	sort.Strings(rhssAbs)
+	for i := range lhssAbs {
+		if !samePath(lhssAbs[i], rhssAbs[i]) {
 			return false
 		}
 	}