fix: make the result errors even if outputs will be changed (#909)

Author: suzuki-shunsuke

pkg/terraform/parser.go

@@ -120,6 +120,7 @@ func (p *PlanParser) Parse(body string) ParseResult { //nolint:cyclop
 	endChangeOutput := -1
 	startWarning := -1
 	endWarning := -1
+	startErrorIndex := -1
 	for i, line := range lines {
 		if line == "Note: Objects have changed outside of Terraform" { // https://github.com/hashicorp/terraform/blob/332045a4e4b1d256c45f98aac74e31102ace7af7/internal/command/views/plan.go#L403
 			startOutsideTerraform = i + 1
@@ -147,8 +148,15 @@ func (p *PlanParser) Parse(body string) ParseResult { //nolint:cyclop
 				endChangeOutput = i - 1
 			}
 		}
+		if startErrorIndex == -1 {
+			if p.Fail.MatchString(line) {
+				startErrorIndex = i
+				firstMatchLineIndex = i
+				firstMatchLine = line
+			}
+		}
 		if firstMatchLineIndex == -1 {
-			if p.Pass.MatchString(line) || p.Fail.MatchString(line) || p.OutputsChanges.MatchString(line) {
+			if p.Pass.MatchString(line) || p.OutputsChanges.MatchString(line) {
 				firstMatchLineIndex = i
 				firstMatchLine = line
 			}