remove duplicate tests
This commit is contained in:
Adrien Delorme
parent
a115b428ac
commit
7d30a5d79d
|
|
@ -2,14 +2,7 @@ package hcl2template
|
|||
|
||||
import (
|
||||
"bytes"
|
||||
"io/ioutil"
|
||||
"os"
|
||||
"os/exec"
|
||||
"path/filepath"
|
||||
"strings"
|
||||
"testing"
|
||||
|
||||
"github.com/google/go-cmp/cmp"
|
||||
)
|
||||
|
||||
func TestHCL2Formatter_Format(t *testing.T) {
|
||||
|
|
@ -38,471 +31,3 @@ func TestHCL2Formatter_Format(t *testing.T) {
|
|||
}
|
||||
}
|
||||
}
|
||||
|
||||
func TestHCL2Formatter_Recursive(t *testing.T) {
|
||||
unformattedData := `
|
||||
// starts resources to provision them.
|
||||
build {
|
||||
sources = [
|
||||
"source.amazon-ebs.ubuntu-1604",
|
||||
"source.virtualbox-iso.ubuntu-1204",
|
||||
]
|
||||
|
||||
provisioner "shell" {
|
||||
string = coalesce(null, "", "string")
|
||||
int = "${41 + 1}"
|
||||
int64 = "${42 + 1}"
|
||||
bool = "true"
|
||||
trilean = true
|
||||
duration = "${9 + 1}s"
|
||||
map_string_string = {
|
||||
a = "b"
|
||||
c = "d"
|
||||
}
|
||||
slice_string = [
|
||||
"a",
|
||||
"b",
|
||||
"c",
|
||||
]
|
||||
slice_slice_string = [
|
||||
["a","b"],
|
||||
["c","d"]
|
||||
]
|
||||
|
||||
nested {
|
||||
string = "string"
|
||||
int = 42
|
||||
int64 = 43
|
||||
bool = true
|
||||
trilean = true
|
||||
duration = "10s"
|
||||
map_string_string = {
|
||||
a = "b"
|
||||
c = "d"
|
||||
}
|
||||
slice_string = [
|
||||
"a",
|
||||
"b",
|
||||
"c",
|
||||
]
|
||||
slice_slice_string = [
|
||||
["a","b"],
|
||||
["c","d"]
|
||||
]
|
||||
}
|
||||
|
||||
nested_slice {
|
||||
}
|
||||
}
|
||||
|
||||
provisioner "file" {
|
||||
string = "string"
|
||||
int = 42
|
||||
int64 = 43
|
||||
bool = true
|
||||
trilean = true
|
||||
duration = "10s"
|
||||
map_string_string = {
|
||||
a = "b"
|
||||
c = "d"
|
||||
}
|
||||
slice_string = [
|
||||
"a",
|
||||
"b",
|
||||
"c",
|
||||
]
|
||||
slice_slice_string = [
|
||||
["a","b"],
|
||||
["c","d"]
|
||||
]
|
||||
|
||||
nested {
|
||||
string = "string"
|
||||
int = 42
|
||||
int64 = 43
|
||||
bool = true
|
||||
trilean = true
|
||||
duration = "10s"
|
||||
map_string_string = {
|
||||
a = "b"
|
||||
c = "d"
|
||||
}
|
||||
slice_string = [
|
||||
"a",
|
||||
"b",
|
||||
"c",
|
||||
]
|
||||
slice_slice_string = [
|
||||
["a","b"],
|
||||
["c","d"]
|
||||
]
|
||||
}
|
||||
|
||||
nested_slice {
|
||||
}
|
||||
}
|
||||
|
||||
post-processor "amazon-import" {
|
||||
string = "string"
|
||||
int = 42
|
||||
int64 = 43
|
||||
bool = true
|
||||
trilean = true
|
||||
duration = "10s"
|
||||
map_string_string = {
|
||||
a = "b"
|
||||
c = "d"
|
||||
}
|
||||
slice_string = [
|
||||
"a",
|
||||
"b",
|
||||
"c",
|
||||
]
|
||||
slice_slice_string = [
|
||||
["a","b"],
|
||||
["c","d"]
|
||||
]
|
||||
|
||||
nested {
|
||||
string = "string"
|
||||
int = 42
|
||||
int64 = 43
|
||||
bool = true
|
||||
trilean = true
|
||||
duration = "10s"
|
||||
map_string_string = {
|
||||
a = "b"
|
||||
c = "d"
|
||||
}
|
||||
slice_string = [
|
||||
"a",
|
||||
"b",
|
||||
"c",
|
||||
]
|
||||
slice_slice_string = [
|
||||
["a","b"],
|
||||
["c","d"]
|
||||
]
|
||||
}
|
||||
|
||||
nested_slice {
|
||||
}
|
||||
}
|
||||
}
|
||||
`
|
||||
|
||||
formattedData := `
|
||||
// starts resources to provision them.
|
||||
build {
|
||||
sources = [
|
||||
"source.amazon-ebs.ubuntu-1604",
|
||||
"source.virtualbox-iso.ubuntu-1204",
|
||||
]
|
||||
|
||||
provisioner "shell" {
|
||||
string = coalesce(null, "", "string")
|
||||
int = "${41 + 1}"
|
||||
int64 = "${42 + 1}"
|
||||
bool = "true"
|
||||
trilean = true
|
||||
duration = "${9 + 1}s"
|
||||
map_string_string = {
|
||||
a = "b"
|
||||
c = "d"
|
||||
}
|
||||
slice_string = [
|
||||
"a",
|
||||
"b",
|
||||
"c",
|
||||
]
|
||||
slice_slice_string = [
|
||||
["a", "b"],
|
||||
["c", "d"]
|
||||
]
|
||||
|
||||
nested {
|
||||
string = "string"
|
||||
int = 42
|
||||
int64 = 43
|
||||
bool = true
|
||||
trilean = true
|
||||
duration = "10s"
|
||||
map_string_string = {
|
||||
a = "b"
|
||||
c = "d"
|
||||
}
|
||||
slice_string = [
|
||||
"a",
|
||||
"b",
|
||||
"c",
|
||||
]
|
||||
slice_slice_string = [
|
||||
["a", "b"],
|
||||
["c", "d"]
|
||||
]
|
||||
}
|
||||
|
||||
nested_slice {
|
||||
}
|
||||
}
|
||||
|
||||
provisioner "file" {
|
||||
string = "string"
|
||||
int = 42
|
||||
int64 = 43
|
||||
bool = true
|
||||
trilean = true
|
||||
duration = "10s"
|
||||
map_string_string = {
|
||||
a = "b"
|
||||
c = "d"
|
||||
}
|
||||
slice_string = [
|
||||
"a",
|
||||
"b",
|
||||
"c",
|
||||
]
|
||||
slice_slice_string = [
|
||||
["a", "b"],
|
||||
["c", "d"]
|
||||
]
|
||||
|
||||
nested {
|
||||
string = "string"
|
||||
int = 42
|
||||
int64 = 43
|
||||
bool = true
|
||||
trilean = true
|
||||
duration = "10s"
|
||||
map_string_string = {
|
||||
a = "b"
|
||||
c = "d"
|
||||
}
|
||||
slice_string = [
|
||||
"a",
|
||||
"b",
|
||||
"c",
|
||||
]
|
||||
slice_slice_string = [
|
||||
["a", "b"],
|
||||
["c", "d"]
|
||||
]
|
||||
}
|
||||
|
||||
nested_slice {
|
||||
}
|
||||
}
|
||||
|
||||
post-processor "amazon-import" {
|
||||
string = "string"
|
||||
int = 42
|
||||
int64 = 43
|
||||
bool = true
|
||||
trilean = true
|
||||
duration = "10s"
|
||||
map_string_string = {
|
||||
a = "b"
|
||||
c = "d"
|
||||
}
|
||||
slice_string = [
|
||||
"a",
|
||||
"b",
|
||||
"c",
|
||||
]
|
||||
slice_slice_string = [
|
||||
["a", "b"],
|
||||
["c", "d"]
|
||||
]
|
||||
|
||||
nested {
|
||||
string = "string"
|
||||
int = 42
|
||||
int64 = 43
|
||||
bool = true
|
||||
trilean = true
|
||||
duration = "10s"
|
||||
map_string_string = {
|
||||
a = "b"
|
||||
c = "d"
|
||||
}
|
||||
slice_string = [
|
||||
"a",
|
||||
"b",
|
||||
"c",
|
||||
]
|
||||
slice_slice_string = [
|
||||
["a", "b"],
|
||||
["c", "d"]
|
||||
]
|
||||
}
|
||||
|
||||
nested_slice {
|
||||
}
|
||||
}
|
||||
}
|
||||
`
|
||||
var buf bytes.Buffer
|
||||
f := NewHCL2Formatter()
|
||||
f.Output = &buf
|
||||
f.Write = true
|
||||
|
||||
tests := []struct {
|
||||
name string
|
||||
recursive bool
|
||||
alreadyPresentContent map[string]string
|
||||
expectedContent map[string]string
|
||||
}{
|
||||
{
|
||||
name: "nested formats recursively",
|
||||
recursive: true,
|
||||
alreadyPresentContent: map[string]string{
|
||||
"foo/bar/baz": unformattedData,
|
||||
"foo/bar/baz/woo": unformattedData,
|
||||
"": unformattedData,
|
||||
},
|
||||
expectedContent: map[string]string{
|
||||
"foo/bar/baz": formattedData,
|
||||
"foo/bar/baz/woo": formattedData,
|
||||
"": formattedData,
|
||||
},
|
||||
},
|
||||
{
|
||||
name: "nested no recursive format",
|
||||
recursive: false,
|
||||
alreadyPresentContent: map[string]string{
|
||||
"foo/bar/baz": unformattedData,
|
||||
"foo/bar/baz/woo": unformattedData,
|
||||
"": unformattedData,
|
||||
},
|
||||
expectedContent: map[string]string{
|
||||
"foo/bar/baz": unformattedData,
|
||||
"foo/bar/baz/woo": unformattedData,
|
||||
"": formattedData,
|
||||
},
|
||||
},
|
||||
}
|
||||
|
||||
testDir := "testdata/format"
|
||||
|
||||
for _, tt := range tests {
|
||||
tempFileNames := make(map[string]string)
|
||||
|
||||
tempDirectory, err := ioutil.TempDir(testDir, "test-dir-*")
|
||||
if err != nil {
|
||||
t.Fatalf("Failed to create temp dir for test case: %s, error: %v", tt.name, err)
|
||||
}
|
||||
defer os.RemoveAll(tempDirectory)
|
||||
|
||||
for subDir, content := range tt.alreadyPresentContent {
|
||||
dir := filepath.Join(tempDirectory, subDir)
|
||||
err = os.MkdirAll(dir, 0700)
|
||||
if err != nil {
|
||||
t.Fatalf("Failed to create directory for test case: %s, error: %v", tt.name, err)
|
||||
}
|
||||
|
||||
tempFile, err := ioutil.TempFile(dir, "*.pkrvars.hcl")
|
||||
if err != nil {
|
||||
t.Fatalf("Failed to create temp file for test case: %s, error: %v", tt.name, err)
|
||||
}
|
||||
|
||||
_, err = tempFile.Write([]byte(content))
|
||||
if err != nil {
|
||||
t.Fatalf("Failed to write temp file for test case: %s, error: %v", tt.name, err)
|
||||
}
|
||||
tempFileNames[subDir] = tempFile.Name()
|
||||
tempFile.Close()
|
||||
}
|
||||
|
||||
f.Recursive = tt.recursive
|
||||
_, diags := f.Format(tempDirectory)
|
||||
if diags.HasErrors() {
|
||||
t.Fatalf("Call to Format failed unexpectedly for test case: %s, errors: %s", tt.name, diags.Error())
|
||||
}
|
||||
|
||||
for expectedPath, expectedContent := range tt.expectedContent {
|
||||
b, err := ioutil.ReadFile(tempFileNames[expectedPath])
|
||||
if err != nil {
|
||||
t.Fatalf("ReadFile failed for test case: %s, error : %v", tt.name, err)
|
||||
}
|
||||
got := string(b)
|
||||
if diff := cmp.Diff(got, expectedContent); diff != "" {
|
||||
t.Errorf(
|
||||
"format dir, unexpected result for test case: %s, path: %s, Expected: %s, Got: %s",
|
||||
tt.name,
|
||||
expectedPath,
|
||||
expectedContent,
|
||||
got)
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
func TestHCL2Formatter_Format_Write(t *testing.T) {
|
||||
|
||||
var buf bytes.Buffer
|
||||
f := NewHCL2Formatter()
|
||||
f.Output = &buf
|
||||
f.Write = true
|
||||
|
||||
unformattedData, err := ioutil.ReadFile("testdata/format/unformatted.pkr.hcl")
|
||||
if err != nil {
|
||||
t.Fatalf("failed to open the unformatted fixture %s", err)
|
||||
}
|
||||
|
||||
tf, err := ioutil.TempFile("", "*.pkr.hcl")
|
||||
if err != nil {
|
||||
t.Fatalf("failed to create tempfile for test %s", err)
|
||||
}
|
||||
defer os.Remove(tf.Name())
|
||||
|
||||
_, _ = tf.Write(unformattedData)
|
||||
tf.Close()
|
||||
|
||||
_, diags := f.Format(tf.Name())
|
||||
if diags.HasErrors() {
|
||||
t.Fatalf("the call to Format failed unexpectedly %s", diags.Error())
|
||||
}
|
||||
|
||||
//lets re-read the tempfile which should now be formatted
|
||||
data, err := ioutil.ReadFile(tf.Name())
|
||||
if err != nil {
|
||||
t.Fatalf("failed to open the newly formatted fixture %s", err)
|
||||
}
|
||||
|
||||
formattedData, err := ioutil.ReadFile("testdata/format/formatted.pkr.hcl")
|
||||
if err != nil {
|
||||
t.Fatalf("failed to open the formatted fixture %s", err)
|
||||
}
|
||||
|
||||
if diff := cmp.Diff(string(data), string(formattedData)); diff != "" {
|
||||
t.Errorf("Unexpected format output %s", diff)
|
||||
}
|
||||
}
|
||||
|
||||
func TestHCL2Formatter_Format_ShowDiff(t *testing.T) {
|
||||
|
||||
if _, err := exec.LookPath("diff"); err != nil {
|
||||
t.Skip("Skipping test because diff is not in the executable PATH")
|
||||
}
|
||||
|
||||
var buf bytes.Buffer
|
||||
f := HCL2Formatter{
|
||||
Output: &buf,
|
||||
ShowDiff: true,
|
||||
}
|
||||
|
||||
_, diags := f.Format("testdata/format/unformatted.pkr.hcl")
|
||||
if diags.HasErrors() {
|
||||
t.Fatalf("the call to Format failed unexpectedly %s", diags.Error())
|
||||
}
|
||||
|
||||
diffHeader := `
|
||||
--- old/testdata/format/unformatted.pkr.hcl
|
||||
+++ new/testdata/format/unformatted.pkr.hcl
|
||||
@@ -1,149 +1,149 @@
|
||||
`
|
||||
if !strings.Contains(buf.String(), diffHeader) {
|
||||
t.Errorf("expected buf to contain a file diff, but instead we got %s", buf.String())
|
||||
}
|
||||
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue