Mark applicable Scope's methods as default

Signed-off-by: Yanming Zhou <[email protected]>

Author: quaff

spring-beans/src/main/java/org/springframework/beans/factory/config/Scope.java

@@ -130,7 +130,9 @@ public interface Scope {
 	 * @return the corresponding object, or {@code null} if none found
 	 * @throws IllegalStateException if the underlying scope is not currently active
 	 */
-	@Nullable Object resolveContextualObject(String key);
+	default @Nullable Object resolveContextualObject(String key) {
+		return null;
+	}
 
 	/**
 	 * Return the <em>conversation ID</em> for the current underlying scope, if any.
@@ -147,6 +149,8 @@ public interface Scope {
 	 * conversation ID for the current scope
 	 * @throws IllegalStateException if the underlying scope is not currently active
 	 */
-	@Nullable String getConversationId();
+	default @Nullable String getConversationId() {
+		return null;
+	}
 
 }

spring-beans/src/test/java/org/springframework/beans/factory/config/TestTypes.java

@@ -44,14 +44,4 @@ public Object remove(String name) {
 	public void registerDestructionCallback(String name, Runnable callback) {
 	}
 
-	@Override
-	public Object resolveContextualObject(String key) {
-		return null;
-	}
-
-	@Override
-	public String getConversationId() {
-		return null;
-	}
-
 }

spring-context/src/main/java/org/springframework/context/support/SimpleThreadScope.java

@@ -83,11 +83,6 @@ public void registerDestructionCallback(String name, Runnable callback) {
 				"Consider using RequestScope in a web environment.");
 	}
 
-	@Override
-	public @Nullable Object resolveContextualObject(String key) {
-		return null;
-	}
-
 	@Override
 	public String getConversationId() {
 		return Thread.currentThread().getName();

spring-context/src/test/java/org/springframework/context/annotation/configuration/ScopingTests.java

@@ -353,11 +353,6 @@ public Object get(String name, ObjectFactory<?> objectFactory) {
 			return beans.get(name);
 		}
 
-		@Override
-		public String getConversationId() {
-			return null;
-		}
-
 		@Override
 		public void registerDestructionCallback(String name, Runnable callback) {
 			throw new IllegalStateException("Not supposed to be called");
@@ -368,10 +363,6 @@ public Object remove(String name) {
 			return beans.remove(name);
 		}
 
-		@Override
-		public Object resolveContextualObject(String key) {
-			return null;
-		}
 	}
 
 }

spring-context/src/test/java/org/springframework/context/annotation/configuration/Spr10744Tests.java

@@ -83,16 +83,6 @@ public Object remove(String name) {
 		@Override
 		public void registerDestructionCallback(String name, Runnable callback) {
 		}
-
-		@Override
-		public Object resolveContextualObject(String key) {
-			return null;
-		}
-
-		@Override
-		public String getConversationId() {
-			return null;
-		}
 	}
 
 

spring-context/src/test/java/org/springframework/context/event/AnnotationDrivenEventListenerTests.java

@@ -1090,16 +1090,6 @@ public Object remove(String name) {
 		@Override
 		public void registerDestructionCallback(String name, Runnable callback) {
 		}
-
-		@Override
-		public Object resolveContextualObject(String key) {
-			return null;
-		}
-
-		@Override
-		public String getConversationId() {
-			return null;
-		}
 	}
 
 

spring-context/src/test/java/org/springframework/context/expression/ApplicationContextExpressionTests.java

@@ -94,10 +94,6 @@ public Object resolveContextualObject(String key) {
 					return null;
 				}
 			}
-			@Override
-			public String getConversationId() {
-				return null;
-			}
 		});
 
 		ac.getBeanFactory().setConversionService(new DefaultConversionService());

spring-context/src/test/java/org/springframework/context/groovy/GroovyBeanDefinitionReaderTests.java

@@ -1214,21 +1214,11 @@ public Object remove(String name) {
 	public void registerDestructionCallback(String name, Runnable callback) {
 	}
 
-	@Override
-	public String getConversationId() {
-		return null;
-	}
-
 	@Override
 	public Object get(String name, ObjectFactory<?> objectFactory) {
 		instanceCount++;
 		return objectFactory.getObject();
 	}
-
-	@Override
-	public Object resolveContextualObject(String s) {
-		return null;
-	}
 }
 
 class BirthdayCardSender {

spring-context/src/testFixtures/java/org/springframework/context/testfixture/SimpleMapScope.java

@@ -68,20 +68,10 @@ public void registerDestructionCallback(String name, Runnable callback) {
 		this.callbacks.add(callback);
 	}
 
-	@Override
-	public Object resolveContextualObject(String key) {
-		return null;
-	}
-
 	public void close() {
 		for (Runnable runnable : this.callbacks) {
 			runnable.run();
 		}
 	}
 
-	@Override
-	public String getConversationId() {
-		return null;
-	}
-
 }

spring-messaging/src/main/java/org/springframework/messaging/simp/SimpSessionScope.java

@@ -70,11 +70,6 @@ public void registerDestructionCallback(String name, Runnable callback) {
 		SimpAttributesContextHolder.currentAttributes().registerDestructionCallback(name, callback);
 	}
 
-	@Override
-	public @Nullable Object resolveContextualObject(String key) {
-		return null;
-	}
-
 	@Override
 	public String getConversationId() {
 		return SimpAttributesContextHolder.currentAttributes().getSessionId();