fix: trim space markdown renderer (runatlantis#2947)

* chore: trim space rendered templates

* regen e2e results

* revert parallel output substring check test case

* follow master branch

* follow main branch

* Update approve_policies_command_runner_test.go

Co-authored-by: nitrocode <[email protected]>

server/controllers/events/testfixtures/test-repos/automerge/exp-output-apply-dir1.txt

@@ -5,7 +5,4 @@ null_resource.automerge[0]: Creating...
 null_resource.automerge[0]: Creation complete after *s [id=*******************]
 
 Apply complete! Resources: 1 added, 0 changed, 0 destroyed.
-
-
-```
-
+```

server/controllers/events/testfixtures/test-repos/automerge/exp-output-apply-dir2.txt

@@ -5,7 +5,4 @@ null_resource.automerge[0]: Creating...
 null_resource.automerge[0]: Creation complete after *s [id=*******************]
 
 Apply complete! Resources: 1 added, 0 changed, 0 destroyed.
-
-
-```
-
+```

server/controllers/events/testfixtures/test-repos/automerge/exp-output-autoplan.txt

@@ -4,10 +4,7 @@ Ran Plan for 2 projects:
 1. dir: `dir2` workspace: `default`
 
 ### 1. dir: `dir1` workspace: `default`
-<details><summary>Show Output</summary>
-
 ```diff
-
 Terraform used the selected providers to generate the following execution
 plan. Resource actions are indicated with the following symbols:
 + create
@@ -20,24 +17,17 @@ Terraform will perform the following actions:
     }
 
 Plan: 1 to add, 0 to change, 0 to destroy.
-
-
 ```
 
 * :arrow_forward: To **apply** this plan, comment:
     * `atlantis apply -d dir1`
 * :put_litter_in_its_place: To **delete** this plan click [here](lock-url)
 * :repeat: To **plan** this project again, comment:
     * `atlantis plan -d dir1`
-</details>
-Plan: 1 to add, 0 to change, 0 to destroy.
 
 ---
 ### 2. dir: `dir2` workspace: `default`
-<details><summary>Show Output</summary>
-
 ```diff
-
 Terraform used the selected providers to generate the following execution
 plan. Resource actions are indicated with the following symbols:
 + create
@@ -50,20 +40,16 @@ Terraform will perform the following actions:
     }
 
 Plan: 1 to add, 0 to change, 0 to destroy.
-
-
 ```
 
 * :arrow_forward: To **apply** this plan, comment:
     * `atlantis apply -d dir2`
 * :put_litter_in_its_place: To **delete** this plan click [here](lock-url)
 * :repeat: To **plan** this project again, comment:
     * `atlantis plan -d dir2`
-</details>
-Plan: 1 to add, 0 to change, 0 to destroy.
 
 ---
 * :fast_forward: To **apply** all unapplied plans from this pull request, comment:
     * `atlantis apply`
 * :put_litter_in_its_place: To delete all plans and locks for the PR, comment:
-    * `atlantis unlock`
+    * `atlantis unlock`

server/controllers/events/testfixtures/test-repos/import-multiple-project/exp-output-autoplan.txt

@@ -7,7 +7,6 @@ Ran Plan for 2 projects:
 <details><summary>Show Output</summary>
 
 ```diff
-
 Terraform used the selected providers to generate the following execution
 plan. Resource actions are indicated with the following symbols:
 + create
@@ -26,8 +25,6 @@ Terraform will perform the following actions:
     }
 
 Plan: 1 to add, 0 to change, 0 to destroy.
-
-
 ```
 
 * :arrow_forward: To **apply** this plan, comment:
@@ -43,7 +40,6 @@ Plan: 1 to add, 0 to change, 0 to destroy.
 <details><summary>Show Output</summary>
 
 ```diff
-
 Terraform used the selected providers to generate the following execution
 plan. Resource actions are indicated with the following symbols:
 + create
@@ -62,8 +58,6 @@ Terraform will perform the following actions:
     }
 
 Plan: 1 to add, 0 to change, 0 to destroy.
-
-
 ```
 
 * :arrow_forward: To **apply** this plan, comment:
@@ -78,4 +72,4 @@ Plan: 1 to add, 0 to change, 0 to destroy.
 * :fast_forward: To **apply** all unapplied plans from this pull request, comment:
     * `atlantis apply`
 * :put_litter_in_its_place: To delete all plans and locks for the PR, comment:
-    * `atlantis unlock`
+    * `atlantis unlock`

server/controllers/events/testfixtures/test-repos/import-multiple-project/exp-output-import-dummy1.txt

@@ -10,11 +10,7 @@ Import successful!
 
 The resources that were imported are shown above. These resources are now in
 your Terraform state and will henceforth be managed by Terraform.
-
-
 ```
 
 * :repeat: To **plan** this project again, comment:
-  * `atlantis plan -d dir1`
-
-
+  * `atlantis plan -d dir1`

server/controllers/events/testfixtures/test-repos/import-multiple-project/exp-output-plan-again.txt

@@ -11,7 +11,6 @@ No changes. Your infrastructure matches the configuration.
 
 Terraform has compared your real infrastructure against your configuration
 and found no differences, so no changes are needed.
-
 ```
 
 * :arrow_forward: To **apply** this plan, comment:
@@ -25,7 +24,6 @@ and found no differences, so no changes are needed.
 <details><summary>Show Output</summary>
 
 ```diff
-
 Terraform used the selected providers to generate the following execution
 plan. Resource actions are indicated with the following symbols:
 + create
@@ -44,8 +42,6 @@ Terraform will perform the following actions:
     }
 
 Plan: 1 to add, 0 to change, 0 to destroy.
-
-
 ```
 
 * :arrow_forward: To **apply** this plan, comment:
@@ -60,4 +56,4 @@ Plan: 1 to add, 0 to change, 0 to destroy.
 * :fast_forward: To **apply** all unapplied plans from this pull request, comment:
     * `atlantis apply`
 * :put_litter_in_its_place: To delete all plans and locks for the PR, comment:
-    * `atlantis unlock`
+    * `atlantis unlock`

server/controllers/events/testfixtures/test-repos/import-single-project-var/exp-output-autoplan.txt

@@ -3,7 +3,6 @@ Ran Plan for dir: `.` workspace: `default`
 <details><summary>Show Output</summary>
 
 ```diff
-
 Terraform used the selected providers to generate the following execution
 plan. Resource actions are indicated with the following symbols:
 + create
@@ -33,8 +32,6 @@ Terraform will perform the following actions:
     }
 
 Plan: 2 to add, 0 to change, 0 to destroy.
-
-
 ```
 
 * :arrow_forward: To **apply** this plan, comment:
@@ -49,4 +46,4 @@ Plan: 2 to add, 0 to change, 0 to destroy.
 * :fast_forward: To **apply** all unapplied plans from this pull request, comment:
     * `atlantis apply`
 * :put_litter_in_its_place: To delete all plans and locks for the PR, comment:
-    * `atlantis unlock`
+    * `atlantis unlock`

server/controllers/events/testfixtures/test-repos/import-single-project-var/exp-output-import-count.txt

@@ -10,11 +10,7 @@ Import successful!
 
 The resources that were imported are shown above. These resources are now in
 your Terraform state and will henceforth be managed by Terraform.
-
-
 ```
 
 * :repeat: To **plan** this project again, comment:
-  * `atlantis plan -d .`
-
-
+  * `atlantis plan -d .`

server/controllers/events/testfixtures/test-repos/import-single-project-var/exp-output-import-foreach.txt

@@ -10,11 +10,7 @@ Import successful!
 
 The resources that were imported are shown above. These resources are now in
 your Terraform state and will henceforth be managed by Terraform.
-
-
 ```
 
 * :repeat: To **plan** this project again, comment:
-  * `atlantis plan -d .`
-
-
+  * `atlantis plan -d .`

server/controllers/events/testfixtures/test-repos/import-single-project-var/exp-output-plan-again.txt

@@ -1,12 +1,10 @@
 Ran Plan for dir: `.` workspace: `default`
 
 ```diff
-
 No changes. Your infrastructure matches the configuration.
 
 Terraform has compared your real infrastructure against your configuration
 and found no differences, so no changes are needed.
-
 ```
 
 * :arrow_forward: To **apply** this plan, comment:
@@ -19,4 +17,4 @@ and found no differences, so no changes are needed.
 * :fast_forward: To **apply** all unapplied plans from this pull request, comment:
     * `atlantis apply`
 * :put_litter_in_its_place: To delete all plans and locks for the PR, comment:
-    * `atlantis unlock`
+    * `atlantis unlock`

server/controllers/events/testfixtures/test-repos/import-single-project/exp-output-apply-no-projects.txt

@@ -1,4 +1 @@
-Ran Apply for 0 projects:
-
-
-
+Ran Apply for 0 projects:

server/controllers/events/testfixtures/test-repos/import-single-project/exp-output-autoplan.txt

@@ -3,7 +3,6 @@ Ran Plan for dir: `.` workspace: `default`
 <details><summary>Show Output</summary>
 
 ```diff
-
 Terraform used the selected providers to generate the following execution
 plan. Resource actions are indicated with the following symbols:
 + create
@@ -33,8 +32,6 @@ Terraform will perform the following actions:
     }
 
 Plan: 2 to add, 0 to change, 0 to destroy.
-
-
 ```
 
 * :arrow_forward: To **apply** this plan, comment:
@@ -49,4 +46,4 @@ Plan: 2 to add, 0 to change, 0 to destroy.
 * :fast_forward: To **apply** all unapplied plans from this pull request, comment:
     * `atlantis apply`
 * :put_litter_in_its_place: To delete all plans and locks for the PR, comment:
-    * `atlantis unlock`
+    * `atlantis unlock`

server/controllers/events/testfixtures/test-repos/import-single-project/exp-output-import-dummy1.txt

@@ -10,11 +10,7 @@ Import successful!
 
 The resources that were imported are shown above. These resources are now in
 your Terraform state and will henceforth be managed by Terraform.
-
-
 ```
 
 * :repeat: To **plan** this project again, comment:
-  * `atlantis plan -d .`
-
-
+  * `atlantis plan -d .`

server/controllers/events/testfixtures/test-repos/import-single-project/exp-output-import-dummy2.txt

@@ -10,11 +10,7 @@ Import successful!
 
 The resources that were imported are shown above. These resources are now in
 your Terraform state and will henceforth be managed by Terraform.
-
-
 ```
 
 * :repeat: To **plan** this project again, comment:
-  * `atlantis plan -d .`
-
-
+  * `atlantis plan -d .`

server/controllers/events/testfixtures/test-repos/import-single-project/exp-output-plan-again.txt

@@ -1,12 +1,10 @@
 Ran Plan for dir: `.` workspace: `default`
 
 ```diff
-
 No changes. Your infrastructure matches the configuration.
 
 Terraform has compared your real infrastructure against your configuration
 and found no differences, so no changes are needed.
-
 ```
 
 * :arrow_forward: To **apply** this plan, comment:
@@ -19,4 +17,4 @@ and found no differences, so no changes are needed.
 * :fast_forward: To **apply** all unapplied plans from this pull request, comment:
     * `atlantis apply`
 * :put_litter_in_its_place: To delete all plans and locks for the PR, comment:
-    * `atlantis unlock`
+    * `atlantis unlock`

server/controllers/events/testfixtures/test-repos/modules-yaml/exp-output-apply-production.txt

@@ -9,6 +9,4 @@ Apply complete! Resources: 1 added, 0 changed, 0 destroyed.
 Outputs:
 
 var = "production"
-
-```
-
+```

server/controllers/events/testfixtures/test-repos/modules-yaml/exp-output-apply-staging.txt

@@ -9,6 +9,4 @@ Apply complete! Resources: 1 added, 0 changed, 0 destroyed.
 Outputs:
 
 var = "staging"
-
-```
-
+```

server/controllers/events/testfixtures/test-repos/modules-yaml/exp-output-autoplan.txt

@@ -7,7 +7,6 @@ Ran Plan for 2 projects:
 <details><summary>Show Output</summary>
 
 ```diff
-
 Terraform used the selected providers to generate the following execution
 plan. Resource actions are indicated with the following symbols:
 + create
@@ -23,7 +22,6 @@ Plan: 1 to add, 0 to change, 0 to destroy.
 
 Changes to Outputs:
 + var = "staging"
-
 ```
 
 * :arrow_forward: To **apply** this plan, comment:
@@ -39,7 +37,6 @@ Plan: 1 to add, 0 to change, 0 to destroy.
 <details><summary>Show Output</summary>
 
 ```diff
-
 Terraform used the selected providers to generate the following execution
 plan. Resource actions are indicated with the following symbols:
 + create
@@ -55,7 +52,6 @@ Plan: 1 to add, 0 to change, 0 to destroy.
 
 Changes to Outputs:
 + var = "production"
-
 ```
 
 * :arrow_forward: To **apply** this plan, comment:
@@ -70,4 +66,4 @@ Plan: 1 to add, 0 to change, 0 to destroy.
 * :fast_forward: To **apply** all unapplied plans from this pull request, comment:
     * `atlantis apply`
 * :put_litter_in_its_place: To delete all plans and locks for the PR, comment:
-    * `atlantis unlock`
+    * `atlantis unlock`

server/controllers/events/testfixtures/test-repos/modules/exp-output-apply-production.txt

@@ -9,6 +9,4 @@ Apply complete! Resources: 1 added, 0 changed, 0 destroyed.
 Outputs:
 
 var = "production"
-
-```
-
+```

server/controllers/events/testfixtures/test-repos/modules/exp-output-apply-staging.txt

@@ -9,6 +9,4 @@ Apply complete! Resources: 1 added, 0 changed, 0 destroyed.
 Outputs:
 
 var = "staging"
-
-```
-
+```