DATAREDIS-1030 - Refine RedisScript creation.

Allow RedisScript.of(…) creation from a Resource.

Original Pull Request: #473

Author: mp911de

src/main/java/org/springframework/data/redis/core/script/DefaultRedisScript.java

@@ -75,7 +75,7 @@ public DefaultRedisScript(String script, @Nullable Class<T> resultType) {
 	 * (non-Javadoc)
 	 * @see org.springframework.beans.factory.InitializingBean#afterPropertiesSet()
 	 */
-	public void afterPropertiesSet() throws Exception {
+	public void afterPropertiesSet() {
 		Assert.state(this.scriptSource != null, "Either script, script location," + " or script source is required");
 	}
 

src/main/java/org/springframework/data/redis/core/script/RedisScript.java

@@ -15,15 +15,17 @@
  */
 package org.springframework.data.redis.core.script;
 
+import org.springframework.core.io.Resource;
 import org.springframework.lang.Nullable;
 import org.springframework.util.Assert;
 
 /**
- * A script to be executed using the <a href="https://redis.io/commands/eval">Redis scripting support</a> available as of
- * version 2.6
+ * A script to be executed using the <a href="https://redis.io/commands/eval">Redis scripting support</a> available as
+ * of version 2.6
  *
  * @author Jennifer Hickey
  * @author Christoph Strobl
+ * @author Mark Paluch
  * @param <T> The script result type. Should be one of Long, Boolean, List, or deserialized value type. Can be
  *          {@litearl null} if the script returns a throw-away status (i.e "OK")
  */
@@ -80,4 +82,43 @@ static <T> RedisScript of(String script, Class<T> resultType) {
 
 		return new DefaultRedisScript(script, resultType);
 	}
+
+	/**
+	 * Creates new {@link RedisScript} from {@link Resource}.
+	 *
+	 * @param resource must not be {@literal null}.
+	 * @return new instance of {@link RedisScript}.
+	 * @since 2.2
+	 */
+	static <T> RedisScript<T> of(Resource resource) {
+
+		Assert.notNull(resource, "Resource must not be null!");
+
+		DefaultRedisScript<T> script = new DefaultRedisScript<>();
+		script.setLocation(resource);
+		script.afterPropertiesSet();
+
+		return script;
+	}
+
+	/**
+	 * Creates new {@link RedisScript} from {@link Resource}.
+	 *
+	 * @param resource must not be {@literal null}.
+	 * @param resultType must not be {@literal null}.
+	 * @return new instance of {@link RedisScript}.
+	 * @since 2.2
+	 */
+	static <T> RedisScript<T> of(Resource resource, Class<T> resultType) {
+
+		Assert.notNull(resource, "Resource must not be null!");
+		Assert.notNull(resultType, "ResultType must not be null!");
+
+		DefaultRedisScript<T> script = new DefaultRedisScript<>();
+		script.setResultType(resultType);
+		script.setLocation(resource);
+		script.afterPropertiesSet();
+
+		return script;
+	}
 }

src/test/java/org/springframework/data/redis/core/script/DefaultRedisScriptTests.java

@@ -28,6 +28,7 @@
  *
  * @author Jennifer Hickey
  * @author Christoph Strobl
+ * @author Mark Paluch
  */
 public class DefaultRedisScriptTests {
 
@@ -55,6 +56,14 @@ public void testGetScriptAsString() {
 		assertThat(redisScript.getScriptAsString()).isEqualTo("return ARGS[1]");
 	}
 
+	@Test // DATAREDIS-1030
+	public void testGetScriptAsStringFromResource() {
+
+		RedisScript<String> redisScript = RedisScript
+				.of(new ClassPathResource("org/springframework/data/redis/core/script/cas.lua"));
+		assertThat(redisScript.getScriptAsString()).startsWith("local current = redis.call('GET', KEYS[1])");
+	}
+
 	@Test(expected = ScriptingException.class)
 	public void testGetScriptAsStringError() {