You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "Yasser Zamani (JIRA)" <ji...@apache.org> on 2018/12/02 06:57:00 UTC
[jira] [Commented] (WW-4988) Upgrade DWR from 1.x to 2.x (for DWR
plugin)
[ https://issues.apache.org/jira/browse/WW-4988?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16706119#comment-16706119 ]
Yasser Zamani commented on WW-4988:
-----------------------------------
Yesterday I tried to add an unit-test but was so complicated that I thought. It was needing mocking servletConfig, servletContext, request, response and etc for DWRServlet and in same time, to be able to execute a validation aware action outside of the core tests.
> Upgrade DWR from 1.x to 2.x (for DWR plugin)
> --------------------------------------------
>
> Key: WW-4988
> URL: https://issues.apache.org/jira/browse/WW-4988
> Project: Struts 2
> Issue Type: Dependency
> Components: Plugin - DWR
> Affects Versions: 2.6
> Environment: All environments
> Reporter: James Chaplin
> Priority: Minor
> Labels: pull-request-available
> Fix For: 2.6
>
>
> Hello Apache Struts Team.
> There appears to be a failure/warning reported in the automated nightly Jenkins builds in relation to the DWR plugin component (reported by org.owasp:dependency-check-maven).
> A PR ([https://github.com/apache/struts/pull/283)] has been opened in hopes of addressing this by _upgrading the Struts 2.6 DWR plugin to the most recent API-compatible version of DWR_ (2.x series).
>
> Since _only manual testing/verification is possible at this time_ (requires modification of the showcase application web.xml, see above PR for details), _there is a risk that upgrading the DWR dependency might introduce issues for the DWR plugin_.
>
> If there is anyone who has _suggestions on creating an automated (preferably self-contained) test for the DWR plugin_ within the regression suite please advise by commenting on this JIRA.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)