You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ra...@apache.org on 2021/04/28 13:23:42 UTC
[sling-org-apache-sling-servlets-resolver] branch master updated:
SLING-10340 - The ScriptResource should unwrap the active resource before
adapting it to a Servlet
This is an automated email from the ASF dual-hosted git repository.
radu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-servlets-resolver.git
The following commit(s) were added to refs/heads/master by this push:
new 245ae4d SLING-10340 - The ScriptResource should unwrap the active resource before adapting it to a Servlet
245ae4d is described below
commit 245ae4d02af09fbf283944e6b49c66be4e1fe81e
Author: Radu Cotescu <ra...@apache.org>
AuthorDate: Wed Apr 28 15:22:37 2021 +0200
SLING-10340 - The ScriptResource should unwrap the active resource before adapting it to a Servlet
* unwrap wrapped resources to check if they're based on a ServletResource
* added unit tests for ScriptResource#adaptTo
---
pom.xml | 4 +-
.../servlets/resolver/internal/ScriptResource.java | 7 +-
.../resolver/internal/ScriptResourceTest.java | 165 +++++++++++++++++++++
3 files changed, 173 insertions(+), 3 deletions(-)
diff --git a/pom.xml b/pom.xml
index 83cb8d5..b057309 100644
--- a/pom.xml
+++ b/pom.xml
@@ -336,8 +336,8 @@
</dependency>
<dependency>
<groupId>org.mockito</groupId>
- <artifactId>mockito-all</artifactId>
- <version>1.10.19</version>
+ <artifactId>mockito-core</artifactId>
+ <version>3.9.0</version>
<scope>test</scope>
</dependency>
<dependency>
diff --git a/src/main/java/org/apache/sling/servlets/resolver/internal/ScriptResource.java b/src/main/java/org/apache/sling/servlets/resolver/internal/ScriptResource.java
index ff995bb..c5a75ce 100644
--- a/src/main/java/org/apache/sling/servlets/resolver/internal/ScriptResource.java
+++ b/src/main/java/org/apache/sling/servlets/resolver/internal/ScriptResource.java
@@ -28,6 +28,7 @@ import org.apache.sling.api.resource.Resource;
import org.apache.sling.api.resource.ResourceMetadata;
import org.apache.sling.api.resource.ResourceResolver;
import org.apache.sling.api.resource.ResourceUtil;
+import org.apache.sling.api.resource.ResourceWrapper;
import org.apache.sling.api.scripting.SlingScript;
import org.apache.sling.servlets.resolver.internal.resource.ServletResource;
@@ -101,7 +102,11 @@ public class ScriptResource extends AbstractResource {
@Override
public <AdapterType> AdapterType adaptTo(final Class<AdapterType> type) {
if ( type == Servlet.class ) {
- if (! (this.getActiveResource() instanceof ServletResource)) {
+ Resource activeResource = this.getActiveResource();
+ while (activeResource instanceof ResourceWrapper) {
+ activeResource = ((ResourceWrapper) activeResource).getResource();
+ }
+ if (! (activeResource instanceof ServletResource)) {
final Servlet s = (Servlet) super.adaptTo(type);
if ( s != null ) {
return (AdapterType)s;
diff --git a/src/test/java/org/apache/sling/servlets/resolver/internal/ScriptResourceTest.java b/src/test/java/org/apache/sling/servlets/resolver/internal/ScriptResourceTest.java
new file mode 100644
index 0000000..1448856
--- /dev/null
+++ b/src/test/java/org/apache/sling/servlets/resolver/internal/ScriptResourceTest.java
@@ -0,0 +1,165 @@
+/*~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ ~ Licensed to the Apache Software Foundation (ASF) under one
+ ~ or more contributor license agreements. See the NOTICE file
+ ~ distributed with this work for additional information
+ ~ regarding copyright ownership. The ASF licenses this file
+ ~ to you under the Apache License, Version 2.0 (the
+ ~ "License"); you may not use this file except in compliance
+ ~ with the License. You may obtain a copy of the License at
+ ~
+ ~ http://www.apache.org/licenses/LICENSE-2.0
+ ~
+ ~ Unless required by applicable law or agreed to in writing,
+ ~ software distributed under the License is distributed on an
+ ~ "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ ~ KIND, either express or implied. See the License for the
+ ~ specific language governing permissions and limitations
+ ~ under the License.
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~*/
+package org.apache.sling.servlets.resolver.internal;
+
+import javax.servlet.Servlet;
+
+import org.apache.sling.api.adapter.AdapterManager;
+import org.apache.sling.api.adapter.SlingAdaptable;
+import org.apache.sling.api.resource.Resource;
+import org.apache.sling.api.resource.ResourceResolver;
+import org.apache.sling.api.resource.ResourceWrapper;
+import org.apache.sling.api.scripting.SlingScript;
+import org.apache.sling.servlets.resolver.internal.resource.ServletResource;
+import org.junit.Test;
+
+import static org.junit.Assert.assertEquals;
+import static org.mockito.ArgumentMatchers.eq;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.spy;
+import static org.mockito.Mockito.when;
+
+public class ScriptResourceTest {
+
+ @Test
+ public void testAdaptToServletForAServletResource() {
+ final String resourcePath = "/sling/test/test.html";
+ ResourceResolver perThreadRR = mock(ResourceResolver.class);
+ ResourceResolver sharedRR = mock(ResourceResolver.class);
+ Resource resource = mock(Resource.class);
+ when(resource.getPath()).thenReturn(resourcePath);
+
+ Servlet servlet = mock(Servlet.class);
+ ServletResource servletResource = spy(new ServletResource(mock(ResourceResolver.class),servlet, "/sling/test/test" +
+ ".html"));
+
+ when(perThreadRR.getResource(resourcePath)).thenReturn(servletResource);
+ when(perThreadRR.isLive()).thenReturn(true);
+
+ ScriptResource scriptResource = new ScriptResource(resource, () -> perThreadRR, sharedRR);
+
+ AdapterManager adapterManager = mock(AdapterManager.class);
+ when(adapterManager.getAdapter(eq(scriptResource), eq(Servlet.class))).thenReturn(mock(Servlet.class));
+ SlingAdaptable.setAdapterManager(adapterManager);
+
+ Servlet adaptedServlet = scriptResource.adaptTo(Servlet.class);
+ assertEquals(servlet, adaptedServlet);
+ }
+
+ @Test
+ public void testAdaptToServletForAWrappedServletResource() {
+ final String resourcePath = "/sling/test/test.html";
+ ResourceResolver perThreadRR = mock(ResourceResolver.class);
+ ResourceResolver sharedRR = mock(ResourceResolver.class);
+ Resource resource = mock(Resource.class);
+ when(resource.getPath()).thenReturn(resourcePath);
+
+ Servlet servlet = mock(Servlet.class);
+ ServletResource servletResource = spy(new ServletResource(mock(ResourceResolver.class),servlet, "/sling/test/test" +
+ ".html"));
+ Resource wrappedResource = new ResourceWrapper(servletResource);
+
+ when(perThreadRR.getResource(resourcePath)).thenReturn(wrappedResource);
+ when(perThreadRR.isLive()).thenReturn(true);
+
+ ScriptResource scriptResource = new ScriptResource(resource, () -> perThreadRR, sharedRR);
+
+ AdapterManager adapterManager = mock(AdapterManager.class);
+ when(adapterManager.getAdapter(eq(scriptResource), eq(Servlet.class))).thenReturn(mock(Servlet.class));
+ SlingAdaptable.setAdapterManager(adapterManager);
+
+ Servlet adaptedServlet = scriptResource.adaptTo(Servlet.class);
+ assertEquals(servlet, adaptedServlet);
+ }
+
+ @Test
+ public void testAdaptToServletForADoubleWrappedServletResource() {
+ final String resourcePath = "/sling/test/test.html";
+ ResourceResolver perThreadRR = mock(ResourceResolver.class);
+ ResourceResolver sharedRR = mock(ResourceResolver.class);
+ Resource resource = mock(Resource.class);
+ when(resource.getPath()).thenReturn(resourcePath);
+
+ Servlet servlet = mock(Servlet.class);
+ ServletResource servletResource = new ServletResource(mock(ResourceResolver.class),servlet, "/sling/test/test.html");
+ Resource wrappedResource = new ResourceWrapper(servletResource);
+ Resource secondWrappedResource = new ResourceWrapper(wrappedResource);
+
+ when(perThreadRR.getResource(resourcePath)).thenReturn(secondWrappedResource);
+ when(perThreadRR.isLive()).thenReturn(true);
+
+ ScriptResource scriptResource = new ScriptResource(resource, () -> perThreadRR, sharedRR);
+
+ AdapterManager adapterManager = mock(AdapterManager.class);
+ when(adapterManager.getAdapter(eq(scriptResource), eq(Servlet.class))).thenReturn(mock(Servlet.class));
+ SlingAdaptable.setAdapterManager(adapterManager);
+
+ Servlet adaptedServlet = scriptResource.adaptTo(Servlet.class);
+ assertEquals(servlet, adaptedServlet);
+ }
+
+ @Test
+ public void testAdaptToServletForANonServletResource() {
+ final String resourcePath = "/sling/test/test.html";
+ ResourceResolver perThreadRR = mock(ResourceResolver.class);
+ ResourceResolver sharedRR = mock(ResourceResolver.class);
+ Resource resource = mock(Resource.class);
+ when(resource.getPath()).thenReturn(resourcePath);
+
+ Servlet servlet = mock(Servlet.class);
+ Resource wrappedResource = new ResourceWrapper(resource);
+
+ when(perThreadRR.getResource(resourcePath)).thenReturn(wrappedResource);
+ when(perThreadRR.isLive()).thenReturn(true);
+
+ ScriptResource scriptResource = new ScriptResource(resource, () -> perThreadRR, sharedRR);
+
+ AdapterManager adapterManager = mock(AdapterManager.class);
+ when(adapterManager.getAdapter(eq(scriptResource), eq(Servlet.class))).thenReturn(servlet);
+ SlingAdaptable.setAdapterManager(adapterManager);
+
+ Servlet adaptedServlet = scriptResource.adaptTo(Servlet.class);
+ assertEquals(servlet, adaptedServlet);
+ }
+
+ @Test
+ public void testAdaptToSlingScript() {
+ final String resourcePath = "/sling/test/test.html";
+ ResourceResolver perThreadRR = mock(ResourceResolver.class);
+ ResourceResolver sharedRR = mock(ResourceResolver.class);
+ Resource resource = mock(Resource.class);
+ when(resource.getPath()).thenReturn(resourcePath);
+
+ SlingScript script = mock(SlingScript.class);
+
+ Resource wrappedResource = new ResourceWrapper(resource);
+ when(perThreadRR.getResource(resourcePath)).thenReturn(wrappedResource);
+ when(perThreadRR.isLive()).thenReturn(true);
+
+ ScriptResource scriptResource = new ScriptResource(resource, () -> perThreadRR, sharedRR);
+
+ AdapterManager adapterManager = mock(AdapterManager.class);
+ when(adapterManager.getAdapter(eq(scriptResource), eq(SlingScript.class))).thenReturn(script);
+ SlingAdaptable.setAdapterManager(adapterManager);
+
+ SlingScript adaptedScript = scriptResource.adaptTo(SlingScript.class);
+ assertEquals(script, adaptedScript);
+ }
+
+}