Merge pull request #59 from hashicorp/github-actions

move to GitHub Actions

Author: kmoe

.github/workflows/test.yml

@@ -0,0 +1,77 @@
+name: Tests
+on:
+  pull_request:
+    branches: [ main ]
+    paths-ignore:
+      - 'README.md'
+      - 'CHANGELOG.md'
+      - 'website/*'
+  push:
+    branches: [ main ]
+    paths-ignore:
+      - 'README.md'
+      - 'CHANGELOG.md'
+      - 'website/*'
+jobs:
+  build:
+    name: Build
+    runs-on: ubuntu-latest
+    timeout-minutes: 5
+    steps:
+
+    - name: Set up Go
+      uses: actions/[email protected]
+      with:
+        go-version: '1.16'
+      id: go
+
+    - name: Check out code into the Go module directory
+      uses: actions/[email protected]
+
+    - name: Go fmt
+      run: |
+        make fmt
+
+    - name: Go vet
+      run: |
+        make vet
+
+    - name: Build
+      run: |
+        go build -v .
+
+
+# run acceptance tests in a matrix with Terraform core versions
+  test:
+    name: Matrix Test
+    needs: build
+    runs-on: ${{ matrix.os }}
+    timeout-minutes: 15
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [macos-latest, windows-latest, ubuntu-latest]
+        terraform:
+          - '0.12.31'
+          - '0.13.7'
+          - '0.14.11'
+          - '0.15.5'
+          - '1.0.1'
+    steps:
+
+    - name: Set up Go
+      uses: actions/[email protected]
+      with:
+        go-version: '1.16'
+      id: go
+
+    - name: Check out code into the Go module directory
+      uses: actions/[email protected]
+
+    - name: TF acceptance tests
+      timeout-minutes: 10
+      env:
+        TF_ACC: "1"
+        TF_ACC_TERRAFORM_VERSION: ${{ matrix.terraform }}
+      run: |
+        go test -v -cover ./internal/provider/

.travis.yml

@@ -10,45 +10,31 @@ import (
 )
 
 func TestLocalFileDataSource(t *testing.T) {
-	var tests = []struct {
-		content string
-		config  string
-	}{
-		{
-			"This is some content",
-			`
-				resource "local_file" "file" {
-					content  = "This is some content"
-					filename = "local_file"
-				}
-				data "local_file" "file" {
-					filename = "${local_file.file.filename}"
-				}
-			`,
-		},
-	}
+	content := "This is some content"
+
+	config := `
+data "local_file" "file" {
+  filename = "./testdata/local_file"
+}
+`
 
-	for i, test := range tests {
-		t.Run(fmt.Sprintf("%d", i), func(t *testing.T) {
-			resource.UnitTest(t, resource.TestCase{
-				Providers: testProviders,
-				Steps: []resource.TestStep{
-					{
-						Config: test.config,
-						Check: func(s *terraform.State) error {
-							m := s.RootModule()
-							i := m.Resources["data.local_file.file"].Primary
-							if got, want := i.Attributes["content"], test.content; got != want {
-								return fmt.Errorf("wrong content %q; want %q", got, want)
-							}
-							if got, want := i.Attributes["content_base64"], base64.StdEncoding.EncodeToString([]byte(test.content)); got != want {
-								return fmt.Errorf("wrong content_base64 %q; want %q", got, want)
-							}
-							return nil
-						},
-					},
+	resource.UnitTest(t, resource.TestCase{
+		Providers: testProviders,
+		Steps: []resource.TestStep{
+			{
+				Config: config,
+				Check: func(s *terraform.State) error {
+					m := s.RootModule()
+					i := m.Resources["data.local_file.file"].Primary
+					if got, want := i.Attributes["content"], content; got != want {
+						return fmt.Errorf("wrong content %q; want %q", got, want)
+					}
+					if got, want := i.Attributes["content_base64"], base64.StdEncoding.EncodeToString([]byte(content)); got != want {
+						return fmt.Errorf("wrong content_base64 %q; want %q", got, want)
+					}
+					return nil
 				},
-			})
-		})
-	}
+			},
+		},
+	})
 }

internal/provider/data_source_local_file_test.go

@@ -8,6 +8,7 @@ import (
 	"path"
 	"path/filepath"
 	"runtime"
+	"strings"
 	"testing"
 
 	r "github.com/hashicorp/terraform-plugin-sdk/v2/helper/resource"
@@ -19,6 +20,7 @@ func TestLocalFile_Basic(t *testing.T) {
 	defer os.RemoveAll(td)
 
 	f := filepath.Join(td, "local_file")
+	f = strings.ReplaceAll(f, `\`, `\\`)
 
 	var cases = []struct {
 		path    string
@@ -140,7 +142,8 @@ func TestLocalFile_Permissions(t *testing.T) {
 	defer os.RemoveAll(td)
 
 	destinationDirPath := td
-	destinationFilePath := destinationDirPath + "/local_file"
+	destinationFilePath := filepath.Join(destinationDirPath, "local_file")
+	destinationFilePath = strings.ReplaceAll(destinationFilePath, `\`, `\\`)
 	filePermission := os.FileMode(0600)
 	directoryPermission := os.FileMode(0700)
 	skipDirCheck := false

internal/provider/resource_local_file_test.go

@@ -0,0 +1 @@
+This is some content