Return-Path: X-Original-To: apmail-struts-commits-archive@minotaur.apache.org Delivered-To: apmail-struts-commits-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 5EB681825E for ; Tue, 13 Oct 2015 11:28:23 +0000 (UTC) Received: (qmail 53826 invoked by uid 500); 13 Oct 2015 11:28:23 -0000 Delivered-To: apmail-struts-commits-archive@struts.apache.org Received: (qmail 53788 invoked by uid 500); 13 Oct 2015 11:28:23 -0000 Mailing-List: contact commits-help@struts.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@struts.apache.org Delivered-To: mailing list commits@struts.apache.org Received: (qmail 53779 invoked by uid 99); 13 Oct 2015 11:28:23 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 13 Oct 2015 11:28:23 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 11836DF9AD; Tue, 13 Oct 2015 11:28:23 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: cnenning@apache.org To: commits@struts.apache.org Date: Tue, 13 Oct 2015 11:28:23 -0000 Message-Id: <0cba4e271da54e44b0b2ab6f9f0b0f32@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: [1/3] struts git commit: Revert "Cleaned up project to be buildable with jdk8. Moved StrutsPortletTestCase to src/test where it belongs! Declared dependencies -junit-plugin and junit as scope test, as it should be." Repository: struts Updated Branches: refs/heads/master 4bff5a874 -> ed1b8cc7b Revert "Cleaned up project to be buildable with jdk8. Moved StrutsPortletTestCase to src/test where it belongs! Declared dependencies -junit-plugin and junit as scope test, as it should be." This reverts commit 4bff5a87486b67a6da0f7703af574f2eff65e036. Project: http://git-wip-us.apache.org/repos/asf/struts/repo Commit: http://git-wip-us.apache.org/repos/asf/struts/commit/9d5b9481 Tree: http://git-wip-us.apache.org/repos/asf/struts/tree/9d5b9481 Diff: http://git-wip-us.apache.org/repos/asf/struts/diff/9d5b9481 Branch: refs/heads/master Commit: 9d5b948190480aa6916acf639f84529f534b7041 Parents: 4bff5a8 Author: cnenning Authored: Tue Oct 13 12:52:54 2015 +0200 Committer: cnenning Committed: Tue Oct 13 12:52:54 2015 +0200 ---------------------------------------------------------------------- plugins/portlet/pom.xml | 27 +++--- .../apache/struts2/StrutsPortletTestCase.java | 90 +++++++++++++++++++ .../struts2/portlet/StrutsPortletTestCase.java | 94 -------------------- 3 files changed, 102 insertions(+), 109 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/struts/blob/9d5b9481/plugins/portlet/pom.xml ---------------------------------------------------------------------- diff --git a/plugins/portlet/pom.xml b/plugins/portlet/pom.xml index f3f71a2..da5a8ca 100644 --- a/plugins/portlet/pom.xml +++ b/plugins/portlet/pom.xml @@ -12,6 +12,18 @@ + org.apache.struts + struts2-junit-plugin + true + + + + junit + junit + true + + + javax.servlet jsp-api provided @@ -42,21 +54,6 @@ provided - - - org.apache.struts - struts2-junit-plugin - true - test - - - - junit - junit - true - test - - mockobjects mockobjects-jdk1.3-j2ee1.3 http://git-wip-us.apache.org/repos/asf/struts/blob/9d5b9481/plugins/portlet/src/main/java/org/apache/struts2/StrutsPortletTestCase.java ---------------------------------------------------------------------- diff --git a/plugins/portlet/src/main/java/org/apache/struts2/StrutsPortletTestCase.java b/plugins/portlet/src/main/java/org/apache/struts2/StrutsPortletTestCase.java new file mode 100644 index 0000000..ad89df1 --- /dev/null +++ b/plugins/portlet/src/main/java/org/apache/struts2/StrutsPortletTestCase.java @@ -0,0 +1,90 @@ +package org.apache.struts2; + +import com.opensymphony.xwork2.ActionContext; +import org.apache.logging.log4j.Logger; +import org.apache.logging.log4j.LogManager; +import org.apache.struts2.portlet.PortletConstants; +import org.apache.struts2.portlet.PortletPhase; +import org.springframework.mock.web.portlet.MockPortletContext; +import org.springframework.mock.web.portlet.MockPortletRequest; +import org.springframework.mock.web.portlet.MockPortletResponse; +import org.springframework.mock.web.portlet.MockPortletSession; +import org.springframework.mock.web.portlet.MockStateAwareResponse; + +import javax.portlet.PortletMode; +import java.util.HashMap; +import java.util.Map; + +/** + * Base class used to test action in portlet environment + */ +public abstract class StrutsPortletTestCase extends StrutsTestCase { + + private static final Logger LOG = LogManager.getLogger(StrutsPortletTestCase.class); + + protected MockPortletSession portletSession; + protected MockPortletRequest portletRequest; + protected MockPortletResponse portletResponse; + protected MockContext portletContext; + + @Override + protected void initActionContext(ActionContext actionContext) { + super.initActionContext(actionContext); + initPortletContext(actionContext); + } + + protected void initPortletContext(ActionContext actionContext) { + LOG.debug("Initializing mock portlet environment"); + portletContext = new MockContext(); + portletContext.setMajorVersion(getMajorVersion()); + actionContext.put(StrutsStatics.STRUTS_PORTLET_CONTEXT, portletContext); + + portletRequest = new MockPortletRequest(portletContext); + portletResponse = new MockStateAwareResponse(); + portletSession = new MockPortletSession(); + portletRequest.setSession(portletSession); + actionContext.setSession(createSession()); + actionContext.put(PortletConstants.REQUEST, portletRequest); + actionContext.put(PortletConstants.RESPONSE, portletResponse); + actionContext.put(PortletConstants.MODE_NAMESPACE_MAP, new HashMap()); + actionContext.put(PortletConstants.PHASE, PortletPhase.EVENT_PHASE); + } + + /** + * Override to define version of your portlet environment + * + * @return portlet version + */ + protected int getMajorVersion() { + return 2; + } + + /** + * Override to create your own session + * + * @return Map with session parameters + */ + private Map createSession() { + return new HashMap(portletRequest.getPortletSession().getAttributeMap()); + } + +} + +/** + * Simple workaround to define Portlet version + */ +class MockContext extends MockPortletContext { + + private int majorVersion; + + @Override + public int getMajorVersion() { + return majorVersion; + } + + public void setMajorVersion(int majorVersion) { + this.majorVersion = majorVersion; + } + +} + http://git-wip-us.apache.org/repos/asf/struts/blob/9d5b9481/plugins/portlet/src/test/java/org/apache/struts2/portlet/StrutsPortletTestCase.java ---------------------------------------------------------------------- diff --git a/plugins/portlet/src/test/java/org/apache/struts2/portlet/StrutsPortletTestCase.java b/plugins/portlet/src/test/java/org/apache/struts2/portlet/StrutsPortletTestCase.java deleted file mode 100644 index a4bb71c..0000000 --- a/plugins/portlet/src/test/java/org/apache/struts2/portlet/StrutsPortletTestCase.java +++ /dev/null @@ -1,94 +0,0 @@ -package org.apache.struts2.portlet; - -import com.opensymphony.xwork2.ActionContext; - -import org.apache.logging.log4j.Logger; -import org.apache.logging.log4j.LogManager; -import org.apache.struts2.StrutsStatics; -import org.apache.struts2.StrutsTestCase; -import org.apache.struts2.portlet.PortletConstants; -import org.apache.struts2.portlet.PortletPhase; -import org.springframework.mock.web.portlet.MockPortletContext; -import org.springframework.mock.web.portlet.MockPortletRequest; -import org.springframework.mock.web.portlet.MockPortletResponse; -import org.springframework.mock.web.portlet.MockPortletSession; -import org.springframework.mock.web.portlet.MockStateAwareResponse; - -import javax.portlet.PortletMode; - -import java.util.HashMap; -import java.util.Map; - -/** - * Base class used to test action in portlet environment - */ -public abstract class StrutsPortletTestCase extends StrutsTestCase { - - private static final Logger LOG = LogManager.getLogger(StrutsPortletTestCase.class); - - protected MockPortletSession portletSession; - protected MockPortletRequest portletRequest; - protected MockPortletResponse portletResponse; - protected MockContext portletContext; - - @Override - protected void initActionContext(ActionContext actionContext) { - super.initActionContext(actionContext); - initPortletContext(actionContext); - } - - protected void initPortletContext(ActionContext actionContext) { - LOG.debug("Initializing mock portlet environment"); - portletContext = new MockContext(); - portletContext.setMajorVersion(getMajorVersion()); - actionContext.put(StrutsStatics.STRUTS_PORTLET_CONTEXT, portletContext); - - portletRequest = new MockPortletRequest(portletContext); - portletResponse = new MockStateAwareResponse(); - portletSession = new MockPortletSession(); - portletRequest.setSession(portletSession); - actionContext.setSession(createSession()); - actionContext.put(PortletConstants.REQUEST, portletRequest); - actionContext.put(PortletConstants.RESPONSE, portletResponse); - actionContext.put(PortletConstants.MODE_NAMESPACE_MAP, new HashMap()); - actionContext.put(PortletConstants.PHASE, PortletPhase.EVENT_PHASE); - } - - /** - * Override to define version of your portlet environment - * - * @return portlet version - */ - protected int getMajorVersion() { - return 2; - } - - /** - * Override to create your own session - * - * @return Map with session parameters - */ - private Map createSession() { - return new HashMap(portletRequest.getPortletSession().getAttributeMap()); - } - -} - -/** - * Simple workaround to define Portlet version - */ -class MockContext extends MockPortletContext { - - private int majorVersion; - - @Override - public int getMajorVersion() { - return majorVersion; - } - - public void setMajorVersion(int majorVersion) { - this.majorVersion = majorVersion; - } - -} -