Merge pull request #122 from albertorm95/support-body-fmt

feat: Support body from fmt command
mercari · Dec 12, 2022 · 1a2db90 · 1a2db90
2 parents baca5ec + e9192df
commit 1a2db90
Show file tree

Hide file tree

Showing 2 changed files with 47 additions and 13 deletions.
diff --git a/terraform/template.go b/terraform/template.go
@@ -41,9 +41,15 @@ const (
 
 {{ .Message }}
 
-{{ .Result }}
+{{if .Result}}
+<pre><code>{{ .Result }}
+</code></pre>
+{{end}}
+
+<details><summary>Details (Click me)</summary>
 
-{{ .Body }}
+<pre><code>{{ .Body }}
+</code></pre></details>
 `
 
 	// DefaultPlanTemplate is a default template for terraform plan
@@ -242,8 +248,8 @@ func (t *FmtTemplate) Execute() (string, error) {
 	data := map[string]interface{}{
 		"Title":   t.Title,
 		"Message": t.Message,
-		"Result":  "",
-		"Body":    t.Result,
+		"Result":  t.Result,
+		"Body":    t.Body,
 		"Link":    t.Link,
 	}
 

diff --git a/terraform/template_test.go b/terraform/template_test.go
@@ -66,7 +66,6 @@ b
 </code></pre></details>
 `,
 		},
-
 		{
 			template: "",
 			value: CommonTemplate{
@@ -193,7 +192,10 @@ func TestFmtTemplateExecute(t *testing.T) {
 
 
 
+<details><summary>Details (Click me)</summary>
 
+<pre><code>
+</code></pre></details>
 `,
 		},
 		{
@@ -208,7 +210,10 @@ message
 
 
 
+<details><summary>Details (Click me)</summary>
 
+<pre><code>
+</code></pre></details>
 `,
 		},
 		{
@@ -225,11 +230,16 @@ a
 b
 
 
+<pre><code>c
+</code></pre>
+
 
-c
+<details><summary>Details (Click me)</summary>
+
+<pre><code>d
+</code></pre></details>
 `,
 		},
-
 		{
 			template: "",
 			value: CommonTemplate{
@@ -244,8 +254,14 @@ a
 b
 
 
+<pre><code>c
+</code></pre>
+
+
+<details><summary>Details (Click me)</summary>
 
-c
+<pre><code>d
+</code></pre></details>
 `,
 		},
 		{
@@ -262,8 +278,14 @@ a
 b
 
 
+<pre><code>This is a &#34;result&#34;.
+</code></pre>
+
+
+<details><summary>Details (Click me)</summary>
 
-This is a &#34;result&#34;.
+<pre><code>d
+</code></pre></details>
 `,
 		},
 		{
@@ -281,19 +303,25 @@ a
 b
 
 
+<pre><code>This is a "result".
+</code></pre>
+
 
-This is a "result".
+<details><summary>Details (Click me)</summary>
+
+<pre><code>d
+</code></pre></details>
 `,
 		},
 		{
 			template: `{{ .Title }}-{{ .Message }}-{{ .Result }}-{{ .Body }}`,
 			value: CommonTemplate{
 				Title:   "a",
 				Message: "b",
-				Result:  "should be used as body",
-				Body:    "should be empty",
+				Result:  "c",
+				Body:    "d",
 			},
-			resp: `a-b--should be used as body`,
+			resp: `a-b-c-d`,
 		},
 	}
 	for _, testCase := range testCases {