Feature: differentiate between (+/-) and (-/+) in output (#59)

* Feature: differentiate between (+/-) and (-/+) in output

* Apply linter suggestions

* Update go to v1.21

* Update gh actions to go v1.21

* Fix build

* Whoops

.github/workflows/build.yml

@@ -21,7 +21,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v4
       with:
-        go-version: '1.20'
+        go-version: '1.21'
 
     - name: golangci-lint
       uses: golangci/golangci-lint-action@v3

.github/workflows/release.yml

@@ -18,7 +18,7 @@ jobs:
         name: Set up Go
         uses: actions/setup-go@v4
         with:
-          go-version: '1.20'
+          go-version: '1.21'
 
       -
         name: Import GPG key

go.mod

@@ -1,6 +1,6 @@
 module github.com/dineshba/tf-summarize
 
-go 1.20
+go 1.21
 
 require (
 	github.com/fatih/color v1.15.0

terraformstate/terraform_state.go

@@ -3,6 +3,7 @@ package terraformstate
 import (
 	"encoding/json"
 	"fmt"
+	"slices"
 )
 
 const ColorReset = "\033[0m"
@@ -19,15 +20,19 @@ type ResourceChange struct {
 	Type          string `json:"type"`
 	Name          string `json:"name"`
 	ProviderName  string `json:"provider_name"`
-	Change        struct {
-		Actions   []string        `json:"actions"`
-		Before    json.RawMessage `json:"before,omitempty"`
-		After     json.RawMessage `json:"after,omitempty"`
-		Importing struct {
-			ID string `json:"id"`
-		} `json:"importing"`
-	} `json:"change"`
-	ActionReason string `json:"action_reason,omitempty"`
+	Change        Change `json:"change"`
+	ActionReason  string `json:"action_reason,omitempty"`
+}
+
+type Change struct {
+	Actions   []string        `json:"actions"`
+	Before    json.RawMessage `json:"before,omitempty"`
+	After     json.RawMessage `json:"after,omitempty"`
+	Importing Importing       `json:"importing"`
+}
+
+type Importing struct {
+	ID string `json:"id"`
 }
 
 type OutputValues struct {
@@ -53,7 +58,10 @@ func (rc ResourceChange) ColorPrefixAndSuffixText() (string, string) {
 	} else if rc.Change.Importing.ID != "" {
 		colorPrefix = ColorCyan
 		suffix = "(i)"
-	} else {
+	} else if slices.Equal(actions, []string{"delete", "create"}) {
+		colorPrefix = ColorMagenta
+		suffix = "(-/+)"
+	} else if slices.Equal(actions, []string{"create", "delete"}) {
 		colorPrefix = ColorMagenta
 		suffix = "(+/-)"
 	}

terraformstate/terraform_state_test.go

@@ -0,0 +1,59 @@
+package terraformstate
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func TestResourceChangeColor(t *testing.T) {
+	ExpectedColors := map[string]string{
+		"create": ColorGreen,
+		"delete": ColorRed,
+		"update": ColorYellow,
+	}
+
+	for action, expectedColor := range ExpectedColors {
+		create := ResourceChange{Change: Change{Actions: []string{action}}}
+		color, _ := create.ColorPrefixAndSuffixText()
+
+		assert.Equal(t, color, expectedColor)
+	}
+	create := ResourceChange{Change: Change{Actions: []string{"create", "delete"}}}
+	color, _ := create.ColorPrefixAndSuffixText()
+	assert.Equal(t, color, ColorMagenta)
+
+	create = ResourceChange{Change: Change{Actions: []string{"delete", "create"}}}
+	color, _ = create.ColorPrefixAndSuffixText()
+	assert.Equal(t, color, ColorMagenta)
+}
+
+func TestResourceChangeSuffix(t *testing.T) {
+	ExpectedSuffix := map[string]string{
+		"create": "(+)",
+		"delete": "(-)",
+		"update": "(~)",
+	}
+
+	for action, expectedSuffix := range ExpectedSuffix {
+		create := ResourceChange{Change: Change{Actions: []string{action}}}
+		_, suffix := create.ColorPrefixAndSuffixText()
+
+		assert.Equal(t, suffix, expectedSuffix)
+	}
+	create := ResourceChange{Change: Change{Actions: []string{"create", "delete"}}}
+	_, suffix := create.ColorPrefixAndSuffixText()
+	assert.Equal(t, suffix, "(+/-)")
+
+	create = ResourceChange{Change: Change{Actions: []string{"delete", "create"}}}
+	_, suffix = create.ColorPrefixAndSuffixText()
+	assert.Equal(t, suffix, "(-/+)")
+}
+
+func TestResourceChangeColorAndSuffixImport(t *testing.T) {
+	importing := ResourceChange{Change: Change{Importing: Importing{ID: "id"}}}
+	color, suffix := importing.ColorPrefixAndSuffixText()
+
+	assert.Equal(t, color, ColorCyan)
+	assert.Equal(t, suffix, "(i)")
+}