From d7450ce8e4d7a6413cfa7fa6658b75967679a023 Mon Sep 17 00:00:00 2001
From: Sam Brannen <104798+sbrannen@users.noreply.github.com>
Date: Mon, 13 Apr 2026 12:58:36 +0200
Subject: [PATCH] Polish SpelCompilationCoverageTests

---
 .../spel/SpelCompilationCoverageTests.java           | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/spring-expression/src/test/java/org/springframework/expression/spel/SpelCompilationCoverageTests.java b/spring-expression/src/test/java/org/springframework/expression/spel/SpelCompilationCoverageTests.java
index 0ecf23aca17..e690c46d188 100644
--- a/spring-expression/src/test/java/org/springframework/expression/spel/SpelCompilationCoverageTests.java
+++ b/spring-expression/src/test/java/org/springframework/expression/spel/SpelCompilationCoverageTests.java
@@ -6426,24 +6426,24 @@ public class SpelCompilationCoverageTests extends AbstractExpressionTests {
 		return (SpelNodeImpl)ast;
 	}
 
-	private void assertCanCompile(Expression expression) {
+	private Expression parse(String expression) {
+		return parser.parseExpression(expression);
+	}
+
+	private static void assertCanCompile(Expression expression) {
 		assertThat(SpelCompiler.compile(expression))
 				.as(() -> "Expression <%s> should be compilable"
 						.formatted(((SpelExpression) expression).toStringAST()))
 				.isTrue();
 	}
 
-	private void assertCannotCompile(Expression expression) {
+	private static void assertCannotCompile(Expression expression) {
 		assertThat(SpelCompiler.compile(expression))
 				.as(() -> "Expression <%s> should not be compilable"
 						.formatted(((SpelExpression) expression).toStringAST()))
 				.isFalse();
 	}
 
-	private Expression parse(String expression) {
-		return parser.parseExpression(expression);
-	}
-
 	private static void assertNotPublic(Class<?> clazz) {
 		assertThat(Modifier.isPublic(clazz.getModifiers())).as("%s must be private", clazz.getName()).isFalse();
 	}