Function TestExpressionBreakingWithQuotes

pkg/workflow/expressions_test.go:1193–1247

TestExpressionBreakingWithQuotes tests that quotes are handled correctly during line breaking

(t *testing.T)

// TestExpressionBreakingWithQuotes tests that quotes are handled correctly during line breaking
func TestExpressionBreakingWithQuotes(t *testing.T) {
	tests := []struct {
		name       string
		expression string
	}{
		{
			name:       "single quoted strings",
			expression: "contains(github.event.issue.body, 'this is a very long string that should not be broken even though it contains || and && operators') && github.event.action == 'opened'",
		},
		{
			name:       "double quoted strings",
			expression: `contains(github.event.issue.body, "this is a very long string that should not be broken even though it contains || and && operators") && github.event.action == "opened"`,
		},
		{
			name:       "mixed quotes",
			expression: `contains(github.event.issue.body, 'single quoted || string') && contains(github.event.comment.body, "double quoted && string")`,
		},
		{
			name:       "escaped quotes",
			expression: `contains(github.event.issue.body, 'string with \\'escaped\\' quotes || and operators') && github.event.action == 'opened'`,
		},
	}

	for _, tt := range tests {
		t.Run(tt.name, func(t *testing.T) {
			lines := BreakLongExpression(tt.expression)

			// Verify that quotes are preserved and no breaking happens inside quoted strings
			joined := strings.Join(lines, " ")
			originalNorm := strings.Join(strings.Fields(tt.expression), " ")
			joinedNorm := strings.Join(strings.Fields(joined), " ")

			if joinedNorm != originalNorm {
				t.Errorf("Expression with quotes not preserved correctly\nOriginal: %s\nJoined:   %s", originalNorm, joinedNorm)
			}

			// Check that no line contains half of a quoted string
			for _, line := range lines {
				singleQuotes := strings.Count(line, "'")
				doubleQuotes := strings.Count(line, `"`)

				// Count non-escaped quotes
				nonEscapedSingle := singleQuotes - strings.Count(line, `\'`)
				nonEscapedDouble := doubleQuotes - strings.Count(line, `\"`)

				if nonEscapedSingle%2 != 0 {
					t.Errorf("Line has unmatched single quotes: %s", line)
				}
				if nonEscapedDouble%2 != 0 {
					t.Errorf("Line has unmatched double quotes: %s", line)
				}
			}
		})
	}
}