You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@struts.apache.org by ya...@apache.org on 2018/09/26 09:19:36 UTC

[struts] branch master updated: fix false positive test pass

This is an automated email from the ASF dual-hosted git repository.

yasserzamani pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/struts.git


The following commit(s) were added to refs/heads/master by this push:
     new 633eec2  fix false positive test pass
633eec2 is described below

commit 633eec29b6a664c2ec84ab76e970c0433d8a9b22
Author: Yasser Zamani <ya...@apache.org>
AuthorDate: Wed Sep 26 12:48:33 2018 +0330

    fix false positive test pass
    
    Co-authored-by: Babak Vahdat <bv...@cyberlogic.ch>
---
 .../src/test/java/com/opensymphony/xwork2/ognl/OgnlUtilStrutsTest.java | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/core/src/test/java/com/opensymphony/xwork2/ognl/OgnlUtilStrutsTest.java b/core/src/test/java/com/opensymphony/xwork2/ognl/OgnlUtilStrutsTest.java
index 1bd07e5..c644bfe 100644
--- a/core/src/test/java/com/opensymphony/xwork2/ognl/OgnlUtilStrutsTest.java
+++ b/core/src/test/java/com/opensymphony/xwork2/ognl/OgnlUtilStrutsTest.java
@@ -40,16 +40,19 @@ public class OgnlUtilStrutsTest extends StrutsInternalTestCase {
 
         try {
             ognlUtil.getExcludedClasses().clear();
+            fail("Missing the expected Exception");
         } catch (Exception ex) {
             assertTrue(ex instanceof UnsupportedOperationException);
         }
         try {
             ognlUtil.getExcludedPackageNames().clear();
+            fail("Missing the expected Exception");
         } catch (Exception ex) {
             assertTrue(ex instanceof UnsupportedOperationException);
         }
         try {
             ognlUtil.getExcludedPackageNamePatterns().clear();
+            fail("Missing the expected Exception");
         } catch (Exception ex) {
             assertTrue(ex instanceof UnsupportedOperationException);
         }