test: use github actions instead of travis

also fix some compatibility issues

Author: hirochachacha

.travis/client_conf.json renamed to .github/client_conf.json

@@ -11,8 +11,8 @@
 	},
 	"session": {
 		"type": "ntlm",
-		"user": "travis",
-		"passwd": "travis"
+		"user": "smbuser",
+		"passwd": "Smbpasswd12345"
 	},
 	"tree_conn": {
 	    "share1": "tmp",

.travis/smb.conf renamed to .github/smb.conf

@@ -0,0 +1,58 @@
+on: [push, pull_request]
+jobs:
+  test_linux:
+    strategy:
+      matrix:
+        go: [1.11, 1.12, 1.13, 1.14, 1.15, 1.16, 1.17]
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/setup-go@v3
+      with:
+        go-version: ${{ matrix.go }}
+    - uses: actions/checkout@v3
+    - run: |
+        sudo apt install -y samba
+        sudo groupadd smbgroup
+        sudo useradd smbuser -G smbgroup
+        (echo Smbpasswd12345; echo Smbpasswd12345) | sudo smbpasswd -a -s smbuser
+        sudo cp ./.github/smb.conf /etc/samba/smb.conf
+        sudo smbd
+    - uses: actions/cache@v2
+      with:
+        path: |
+          ~/go/pkg/mod
+          ~/.cache/go-build
+          ~/Library/Caches/go-build
+          ~\AppData\Local\go-build
+        key: ${{ runner.os }}-go-${{ matrix.go }}-${{ hashFiles('**/go.sum') }}
+        restore-keys: ${{ runner.os }}-go-${{ matrix.go }}-
+    - run: |
+        cp ./.github/client_conf.json ./
+        go test -v -race ./...
+  test_windows:
+    strategy:
+      matrix:
+        go: [1.15, 1.16, 1.17]
+    runs-on: windows-latest
+    steps:
+    - uses: actions/setup-go@v3
+      with:
+        go-version: ${{ matrix.go }}
+    - uses: actions/checkout@v3
+    - run: |
+        mkdir C:\tmp
+        net user smbuser Smbpasswd12345 /add
+        net share tmp="C:\tmp" /GRANT:smbuser,full
+        net share tmp2="C:\tmp" /GRANT:smbuser,read
+    - uses: actions/cache@v2
+      with:
+        path: |
+          ~/go/pkg/mod
+          ~/.cache/go-build
+          ~/Library/Caches/go-build
+          ~\AppData\Local\go-build
+        key: ${{ runner.os }}-go-${{ matrix.go }}-${{ hashFiles('**/go.sum') }}
+        restore-keys: ${{ runner.os }}-go-${{ matrix.go }}-
+    - run: |
+        cp ./.github/client_conf.json ./
+        go test -v -race ./...

.github/workflows/go.yml

@@ -4,7 +4,6 @@ import (
 	"context"
 	"fmt"
 	"io"
-	"math"
 	"math/rand"
 	"net"
 	"os"
@@ -797,6 +796,11 @@ func (fs *Share) ReadDir(dirname string) ([]os.FileInfo, error) {
 	return fis, nil
 }
 
+const (
+	intSize = 32 << (^uint(0) >> 63) // 32 or 64
+	maxInt  = 1<<(intSize-1) - 1
+)
+
 func (fs *Share) ReadFile(filename string) ([]byte, error) {
 	f, err := fs.Open(filename)
 	if err != nil {
@@ -808,7 +812,7 @@ func (fs *Share) ReadFile(filename string) ([]byte, error) {
 
 	var size int
 
-	if size64 <= math.MaxInt {
+	if size64 <= maxInt {
 		size = int(size64)
 
 		// If a file claims a small size, read at least 512 bytes.
@@ -819,7 +823,7 @@ func (fs *Share) ReadFile(filename string) ([]byte, error) {
 			size = 512
 		}
 	} else {
-		size = math.MaxInt
+		size = maxInt
 	}
 
 	data := make([]byte, 0, size)

.travis.yml

@@ -1,4 +1,4 @@
-// +build go1.6
+// +build go1.16
 
 package smb2
 
@@ -48,6 +48,34 @@ func (fs *wfs) ReadFile(name string) ([]byte, error) {
 	return fs.share.ReadFile(fs.path(name))
 }
 
+// dirInfo is a DirEntry based on a FileInfo.
+type dirInfo struct {
+	fileInfo fs.FileInfo
+}
+
+func (di dirInfo) IsDir() bool {
+	return di.fileInfo.IsDir()
+}
+
+func (di dirInfo) Type() fs.FileMode {
+	return di.fileInfo.Mode().Type()
+}
+
+func (di dirInfo) Info() (fs.FileInfo, error) {
+	return di.fileInfo, nil
+}
+
+func (di dirInfo) Name() string {
+	return di.fileInfo.Name()
+}
+
+func fileInfoToDirEntry(info fs.FileInfo) fs.DirEntry {
+	if info == nil {
+		return nil
+	}
+	return dirInfo{fileInfo: info}
+}
+
 type wfile struct {
 	*File
 }
@@ -59,7 +87,7 @@ func (f *wfile) ReadDir(n int) (dirents []fs.DirEntry, err error) {
 	}
 	dirents = make([]fs.DirEntry, len(infos))
 	for i, info := range infos {
-		dirents[i] = fs.FileInfoToDirEntry(info)
+		dirents[i] = fileInfoToDirEntry(info)
 	}
 	return dirents, nil
 }

client.go

@@ -1,4 +1,4 @@
-// +build go1.6
+// +build go1.16
 
 package smb2_test