/hg/icedtea-web: Removed most of checks to no output from reprod...

jvanek at icedtea.classpath.org jvanek at icedtea.classpath.org
Mon Nov 25 07:47:47 PST 2013


changeset 349c86e05863 in /hg/icedtea-web
details: http://icedtea.classpath.org/hg/icedtea-web?cmd=changeset;node=349c86e05863
author: Jiri Vanek <jvanek at redhat.com>
date: Mon Nov 25 16:54:23 2013 +0100

	Removed most of checks to no output from reproducers


diffstat:

 ChangeLog                                                                                         |  22 ++++
 netx/net/sourceforge/jnlp/JNLPFile.java                                                           |   4 +-
 netx/net/sourceforge/jnlp/runtime/JNLPSecurityManager.java                                        |   2 +-
 tests/reproducers/custom/remote/testcases/RemoteApplicationSettings.java                          |  50 ++++++---
 tests/reproducers/signed/ReadPropertiesBySignedHack/testcases/ReadPropertiesBySignedHackTest.java |   1 -
 tests/reproducers/signed/ReadPropertiesSigned/testcases/ReadPropertiesSignedTest.java             |   3 -
 tests/reproducers/simple/AddShutdownHook/testcases/AddShutdownHookTest.java                       |   1 -
 tests/reproducers/simple/AllStackTraces/testcases/AllStackTracesTest.java                         |   1 -
 tests/reproducers/simple/CreateClassLoader/testcases/CreateClassLoaderTest.java                   |   1 -
 tests/reproducers/simple/ReadEnvironment/testcases/ReadEnvironmentTest.java                       |   1 -
 tests/reproducers/simple/ReadProperties/testcases/ReadPropertiesTest.java                         |   2 -
 tests/reproducers/simple/RedirectStreams/testcases/RedirectStreamsTest.java                       |   1 -
 tests/reproducers/simple/ReplaceSecurityManager/testcases/ReplaceSecurityManagerTest.java         |   1 -
 tests/reproducers/simple/SetContextClassLoader/testcases/SetContextClassLoaderTest.java           |   1 -
 tests/reproducers/simple/simpletest2/testcases/SimpleTest2Test.java                               |   1 -
 15 files changed, 59 insertions(+), 33 deletions(-)

diffs (347 lines):

diff -r bf80f38923af -r 349c86e05863 ChangeLog
--- a/ChangeLog	Mon Nov 25 10:16:42 2013 +0100
+++ b/ChangeLog	Mon Nov 25 16:54:23 2013 +0100
@@ -1,3 +1,25 @@
+2013-11-25  Jiri Vanek  <jvanek at redhat.com>
+
+	* netx/net/sourceforge/jnlp/JNLPFile.java: (TITLE_NOT_FOUND) new constant
+	holding the no title found string to be reused. (getTitleFromManifest)
+	Now using that constant.
+	* netx/net/sourceforge/jnlp/runtime/JNLPSecurityManager.java: adding window
+	erro message moved to be debug only.
+	* ests/reproducers/custom/remote/testcases/RemoteApplicationSettings.java: 
+	Added (clean) mechanism to filter out TITLE_NOT_FOUND
+	* tests/reproducers/signed/ReadPropertiesBySignedHack/testcases/ReadPropertiesBySignedHackTest.java:
+	* tests/reproducers/signed/ReadPropertiesSigned/testcases/ReadPropertiesSignedTest.java:
+	* tests/reproducers/simple/AddShutdownHook/testcases/AddShutdownHookTest.java:
+	* tests/reproducers/simple/AllStackTraces/testcases/AllStackTracesTest.java	
+	* tests/reproducers/simple/CreateClassLoader/testcases/CreateClassLoaderTest.java
+	* tests/reproducers/simple/ReadEnvironment/testcases/ReadEnvironmentTest.java
+	* tests/reproducers/simple/ReadProperties/testcases/ReadPropertiesTest.java
+	* tests/reproducers/simple/RedirectStreams/testcases/RedirectStreamsTest.java
+	* tests/reproducers/simple/ReplaceSecurityManager/testcases/ReplaceSecurityManagerTest.java
+	* tests/reproducers/simple/SetContextClassLoader/testcases/SetContextClassLoaderTest.java
+	* tests/reproducers/simple/simpletest2/testcases/SimpleTest2Test.java
+	Removed checks for emty outputs
+
 2013-11-25  Jiri Vanek  <jvanek at redhat.com>
 
 	* netx/net/sourceforge/jnlp/ResourcesDesc.java: (getMainJAR) throw an
diff -r bf80f38923af -r 349c86e05863 netx/net/sourceforge/jnlp/JNLPFile.java
--- a/netx/net/sourceforge/jnlp/JNLPFile.java	Mon Nov 25 10:16:42 2013 +0100
+++ b/netx/net/sourceforge/jnlp/JNLPFile.java	Mon Nov 25 16:54:23 2013 +0100
@@ -134,6 +134,8 @@
     /** important manifests' attributes */
     private final ManifestsAttributes manifestsAttributes = new ManifestsAttributes();
 
+    public static final String TITLE_NOT_FOUND = "Application title was not found in manifest. Check with application vendor";
+
 
     { // initialize defaults if security allows
         try {
@@ -340,7 +342,7 @@
     public String getTitleFromManifest() {
         String inManifestTitle = getManifestsAttributes().getApplicationName();
         if (inManifestTitle == null && getManifestsAttributes().isLoader()){
-            OutputController.getLogger().log(OutputController.Level.WARNING_ALL,"Application title was not found in manifest. Check with application vendor");
+            OutputController.getLogger().log(OutputController.Level.WARNING_ALL, TITLE_NOT_FOUND);
         }
         return inManifestTitle;
     }
diff -r bf80f38923af -r 349c86e05863 netx/net/sourceforge/jnlp/runtime/JNLPSecurityManager.java
--- a/netx/net/sourceforge/jnlp/runtime/JNLPSecurityManager.java	Mon Nov 25 10:16:42 2013 +0100
+++ b/netx/net/sourceforge/jnlp/runtime/JNLPSecurityManager.java	Mon Nov 25 16:54:23 2013 +0100
@@ -344,7 +344,7 @@
         if (app != null && window instanceof Window) {
             Window w = (Window) window;
 
-            OutputController.getLogger().log(OutputController.Level.ERROR_ALL, "SM: app: " + app.getTitle() + " is adding a window: " + window + " with appContext " + AppContext.getAppContext());
+            OutputController.getLogger().log(OutputController.Level.ERROR_DEBUG, "SM: app: " + app.getTitle() + " is adding a window: " + window + " with appContext " + AppContext.getAppContext());
 
             weakWindows.add(w); // for mapping window -> app
             weakApplications.add(app);
diff -r bf80f38923af -r 349c86e05863 tests/reproducers/custom/remote/testcases/RemoteApplicationSettings.java
--- a/tests/reproducers/custom/remote/testcases/RemoteApplicationSettings.java	Mon Nov 25 10:16:42 2013 +0100
+++ b/tests/reproducers/custom/remote/testcases/RemoteApplicationSettings.java	Mon Nov 25 16:54:23 2013 +0100
@@ -37,6 +37,7 @@
 
 import java.net.MalformedURLException;
 import java.net.URL;
+import net.sourceforge.jnlp.JNLPFile;
 import net.sourceforge.jnlp.ProcessResult;
 import org.junit.Assert;
 import org.junit.Test;
@@ -70,6 +71,9 @@
 
         URL u;
 
+        public String clean(String s){
+            return s.replaceAll("\\s*" + JNLPFile.TITLE_NOT_FOUND + "\\s*", "").trim();
+        }
         @Override
         public URL getUrl() {
             return u;
@@ -88,8 +92,8 @@
 
         @Override
         public void evaluate(ProcessResult pr) {
-            Assert.assertTrue(stdoutEmpty, pr.stdout.length() == 0);
-            Assert.assertTrue(pr.stderr.length() == 0 || pr.stderr.contains(IllegalStateException.class.getName()));
+            Assert.assertTrue(stdoutEmpty, clean(pr.stdout).length() == 0);
+            Assert.assertTrue(clean(pr.stderr).length() == 0 || pr.stderr.contains(IllegalStateException.class.getName()));
 
         }
     }
@@ -102,8 +106,8 @@
 
         @Override
         public void evaluate(ProcessResult pr) {
-            Assert.assertTrue(stdoutEmpty, pr.stdout.length() == 0);
-            Assert.assertTrue(pr.stderr.length() == 0 || pr.stderr.contains("Cannot grant permissions to unsigned jars. Application requested security permissions, but jars are not signed"));
+            Assert.assertTrue(stdoutEmpty, clean(pr.stdout).length() == 0);
+            Assert.assertTrue(clean(pr.stderr).length() == 0 || pr.stderr.contains("Cannot grant permissions to unsigned jars. Application requested security permissions, but jars are not signed"));
 
         }
     }
@@ -116,7 +120,7 @@
 
         @Override
         public void evaluate(ProcessResult pr) {
-            Assert.assertTrue(stdout, pr.stdout.length() == 0);
+            Assert.assertTrue(stdout, clean(pr.stdout).length() == 0);
             Assert.assertTrue(pr.stderr.contains("Splash closed"));
             Assert.assertFalse(pr.stderr.contains("Exception"));
 
@@ -131,10 +135,8 @@
 
         @Override
         public void evaluate(ProcessResult pr) {
-            Assert.assertTrue(pr.stdout.length() > 0);
-            Assert.assertTrue(pr.stderr.length() > 0);
-            Assert.assertFalse(pr.stderr.contains("Exception"));
-            Assert.assertFalse(pr.stdout.contains("Exception"));
+            Assert.assertTrue(pr.stdout.length() == 0);
+            Assert.assertTrue(pr.stderr.length() == 0);
 
         }
     }
@@ -153,35 +155,49 @@
         }
     }
 
-    public static class Arbores extends NoOutputs {
+     public abstract static class NearlyNoOutputs extends StringBasedURL {
+
+        public NearlyNoOutputs(String r) {
+            super(r);
+        }
+
+        @Override
+        public void evaluate(ProcessResult pr) {
+            Assert.assertTrue(stdoutEmpty, clean(pr.stdout).length() == 0);
+            Assert.assertTrue(stderrEmpty, clean(pr.stderr).length() == 0);
+
+        }
+    }
+
+    public static class Arbores extends NearlyNoOutputs {
 
         public Arbores() {
             super("http://www.arbores.ca/AnnuityCalc.jnlp");
         }
     }
 
-    public static class PhetSims extends NoOutputs {
+    public static class PhetSims extends NearlyNoOutputs {
 
         public PhetSims() {
             super("http://phetsims.colorado.edu/sims/circuit-construction-kit/circuit-construction-kit-dc_en.jnlp");
         }
     }
 
-    public static class TopCoder extends NoOutputs {
+    public static class TopCoder extends NearlyNoOutputs {
 
         public TopCoder() {
             super("http://www.topcoder.com/contest/arena/ContestAppletProd.jnlp");
         }
     }
 
-    public static class SunSwingDemo extends NoOutputs {
+    public static class SunSwingDemo extends NearlyNoOutputs {
 
         public SunSwingDemo() throws MalformedURLException {
             super("http://java.sun.com/docs/books/tutorialJWS/uiswing/events/ex6/ComponentEventDemo.jnlp");
         }
     }
 
-    public static class ArboresDeposit extends NoOutputs {
+    public static class ArboresDeposit extends NearlyNoOutputs {
 
         public ArboresDeposit() throws MalformedURLException {
             super("http://www.arbores.ca/Deposit.jnlp");
@@ -192,8 +208,8 @@
 
         @Override
         public void evaluate(ProcessResult pr) {
-            Assert.assertTrue(stdoutEmpty, pr.stdout.length() == 0);
-            Assert.assertTrue(pr.stderr.length() == 0 || (pr.stderr.contains("Cannot read File Manager history data file,")
+            Assert.assertTrue(stdoutEmpty, clean(pr.stdout).length() == 0);
+            Assert.assertTrue(clean(pr.stderr).length() == 0 || (clean(pr.stderr).contains("Cannot read File Manager history data file,")
                     && pr.stderr.contains("FileMgr will be initialized with default options")));
 
         }
@@ -203,7 +219,7 @@
         }
     }
 
-    public static class FuseSwing extends NoOutputs {
+    public static class FuseSwing extends NearlyNoOutputs {
 
         public FuseSwing() {
             super("http://www.progx.org/users/Gfx/apps/fuse-swing-demo.jnlp");
diff -r bf80f38923af -r 349c86e05863 tests/reproducers/signed/ReadPropertiesBySignedHack/testcases/ReadPropertiesBySignedHackTest.java
--- a/tests/reproducers/signed/ReadPropertiesBySignedHack/testcases/ReadPropertiesBySignedHackTest.java	Mon Nov 25 10:16:42 2013 +0100
+++ b/tests/reproducers/signed/ReadPropertiesBySignedHack/testcases/ReadPropertiesBySignedHackTest.java	Mon Nov 25 16:54:23 2013 +0100
@@ -58,7 +58,6 @@
         Assert.assertTrue("Stderr should contains "+s+" but did not",pr.stderr.contains(s));
         String ss="ClassNotFoundException";
         Assert.assertFalse("Stderr should not contains "+ss+" but did",pr.stderr.contains(ss));
-        Assert.assertTrue("stdout lenght should be <2 but was "+pr.stdout.length(),pr.stdout.length()<2); // /home/user or /root or eanything else :(
         Assert.assertFalse("should not be terminated but was",pr.wasTerminated);
         Assert.assertEquals((Integer)0, pr.returnValue);
    }
diff -r bf80f38923af -r 349c86e05863 tests/reproducers/signed/ReadPropertiesSigned/testcases/ReadPropertiesSignedTest.java
--- a/tests/reproducers/signed/ReadPropertiesSigned/testcases/ReadPropertiesSignedTest.java	Mon Nov 25 10:16:42 2013 +0100
+++ b/tests/reproducers/signed/ReadPropertiesSigned/testcases/ReadPropertiesSignedTest.java	Mon Nov 25 16:54:23 2013 +0100
@@ -58,7 +58,6 @@
         Assert.assertTrue("Stderr should match "+accessMatcher+" but did not",pr.stderr.matches(accessMatcher));
         String ss="ClassNotFoundException";
         Assert.assertFalse("Stderr should not contains "+ss+" but did",pr.stderr.contains(ss));
-        Assert.assertTrue("stdout lenght should be <2 but was "+pr.stdout.length(),pr.stdout.length()<2); // /home/user or /root or eanything else :(
         Assert.assertFalse("should not be terminated but was",pr.wasTerminated);
         Assert.assertEquals((Integer)0, pr.returnValue);
     }
@@ -70,7 +69,6 @@
         Assert.assertFalse("Stderr should NOT match "+accessMatcher+" but did",pr.stderr.matches(accessMatcher));
         String ss="ClassNotFoundException";
         Assert.assertFalse("Stderr should not contains "+ss+" but did",pr.stderr.contains(ss));
-        Assert.assertTrue("stdout lenght should be >= but was "+pr.stdout.length(),pr.stdout.length()>=4); // /home/user or /root or eanything else :(
         Assert.assertFalse("should not be terminated but was",pr.wasTerminated);
         Assert.assertEquals((Integer)0, pr.returnValue);
     }
@@ -81,7 +79,6 @@
         Assert.assertTrue("Stderr should match "+accessMatcher+" but did not",pr.stderr.matches(accessMatcher));
         String ss="ClassNotFoundException";
         Assert.assertFalse("Stderr should not contains "+ss+" but did",pr.stderr.contains(ss));
-        Assert.assertFalse("stdout lenght should not be  >2 but was "+pr.stdout.length(),pr.stdout.length()>2);
         Assert.assertFalse("should not be terminated but was",pr.wasTerminated);
         Assert.assertEquals((Integer)0, pr.returnValue);
         ProcessResult pr2=server.executeJavawsHeadless(null,"/ReadProperties1.jnlp");
diff -r bf80f38923af -r 349c86e05863 tests/reproducers/simple/AddShutdownHook/testcases/AddShutdownHookTest.java
--- a/tests/reproducers/simple/AddShutdownHook/testcases/AddShutdownHookTest.java	Mon Nov 25 10:16:42 2013 +0100
+++ b/tests/reproducers/simple/AddShutdownHook/testcases/AddShutdownHookTest.java	Mon Nov 25 16:54:23 2013 +0100
@@ -61,7 +61,6 @@
         ProcessResult pr = server.executeJavawsHeadless(null, "/AddShutdownHook.jnlp");
         Assert.assertTrue("stderr " + mr.toPassingString(), mr.evaluate(pr.stderr));
         Assert.assertFalse("stderr " + cnf.toFailingString(), cnf.evaluate(pr.stderr));
-        Assert.assertFalse("stdout length should be <=2, but was " + pr.stdout.length(), pr.stdout.length() > 2);
         Assert.assertFalse("AddShutdownHookTestLunch1 should not be terminated, but was", pr.wasTerminated);
         Assert.assertFalse("stderr " + cf.toFailingString(), cf.evaluate(pr.stderr));
         Assert.assertEquals((Integer) 0, pr.returnValue);
diff -r bf80f38923af -r 349c86e05863 tests/reproducers/simple/AllStackTraces/testcases/AllStackTracesTest.java
--- a/tests/reproducers/simple/AllStackTraces/testcases/AllStackTracesTest.java	Mon Nov 25 10:16:42 2013 +0100
+++ b/tests/reproducers/simple/AllStackTraces/testcases/AllStackTracesTest.java	Mon Nov 25 16:54:23 2013 +0100
@@ -54,7 +54,6 @@
         Assert.assertTrue("stderr should match `"+c+"`, but didn't ",pr.stderr.matches(c));
         String cc="ClassNotFoundException";
         Assert.assertFalse("stderr should NOT contains `"+cc+"`, but did ",pr.stderr.contains(cc));
-        Assert.assertFalse("stdout length should be <=2, but was "+pr.stdout.length(),pr.stdout.length()>2);
         Assert.assertFalse("AllStackTracesTest1 should not be terminated, but was",pr.wasTerminated);
         Assert.assertEquals((Integer)0, pr.returnValue);
     }
diff -r bf80f38923af -r 349c86e05863 tests/reproducers/simple/CreateClassLoader/testcases/CreateClassLoaderTest.java
--- a/tests/reproducers/simple/CreateClassLoader/testcases/CreateClassLoaderTest.java	Mon Nov 25 10:16:42 2013 +0100
+++ b/tests/reproducers/simple/CreateClassLoader/testcases/CreateClassLoaderTest.java	Mon Nov 25 16:54:23 2013 +0100
@@ -52,7 +52,6 @@
         Assert.assertTrue("Stderr should match "+s+" but didn't",pr.stderr.matches(s));
         String cc="ClassNotFoundException";
         Assert.assertFalse("stderr should NOT contains `"+cc+"`, but did",pr.stderr.contains(cc));
-        Assert.assertFalse("stdout length should be <=2, but was "+pr.stdout.length(),pr.stdout.length()>2);
         Assert.assertFalse("CreateClassLoaderLunch1 should not be terminated, but was",pr.wasTerminated);
         Assert.assertEquals((Integer) 0, pr.returnValue);
     }
diff -r bf80f38923af -r 349c86e05863 tests/reproducers/simple/ReadEnvironment/testcases/ReadEnvironmentTest.java
--- a/tests/reproducers/simple/ReadEnvironment/testcases/ReadEnvironmentTest.java	Mon Nov 25 10:16:42 2013 +0100
+++ b/tests/reproducers/simple/ReadEnvironment/testcases/ReadEnvironmentTest.java	Mon Nov 25 16:54:23 2013 +0100
@@ -51,7 +51,6 @@
         Assert.assertTrue("stderr should match"+s+"but didn't",pr.stderr.matches(s));
         String cc="ClassNotFoundException";
         Assert.assertFalse("stderr should NOT contains `"+cc+"`, but did",pr.stderr.contains(cc));
-        Assert.assertFalse("stdout length should be <=2, but was "+pr.stdout.length(),pr.stdout.length()>2);
         Assert.assertFalse("ReadEnvironmentLunch1 should not be terminated, but was",pr.wasTerminated);
         Assert.assertEquals((Integer) 0, pr.returnValue);
     }
diff -r bf80f38923af -r 349c86e05863 tests/reproducers/simple/ReadProperties/testcases/ReadPropertiesTest.java
--- a/tests/reproducers/simple/ReadProperties/testcases/ReadPropertiesTest.java	Mon Nov 25 10:16:42 2013 +0100
+++ b/tests/reproducers/simple/ReadProperties/testcases/ReadPropertiesTest.java	Mon Nov 25 16:54:23 2013 +0100
@@ -52,7 +52,6 @@
         Assert.assertTrue("stderr should match "+s+" but didn't",pr.stderr.matches(s));
         String cc="ClassNotFoundException";
         Assert.assertFalse("stderr should NOT contains `"+cc+"`, but did",pr.stderr.contains(cc));
-        Assert.assertFalse("stdout length should be <=2, but was "+pr.stdout.length(),pr.stdout.length()>2);
         Assert.assertFalse("ReadPropertiesLunch1 should not be terminated, but was",pr.wasTerminated);
         Assert.assertEquals((Integer)0, pr.returnValue);
     }
@@ -64,7 +63,6 @@
         Assert.assertTrue("stderr should match "+s+" but didn't",pr.stderr.matches(s));
         String cc="ClassNotFoundException";
         Assert.assertFalse("stderr should NOT contains `"+cc+"`, but did",pr.stderr.contains(cc));
-        Assert.assertFalse("stdout length should be <=2, but was "+pr.stdout.length(),pr.stdout.length()>2);
         Assert.assertFalse("ReadPropertiesLunch2 should not be terminated, but was",pr.wasTerminated);
         Assert.assertEquals((Integer)0, pr.returnValue);
     }
diff -r bf80f38923af -r 349c86e05863 tests/reproducers/simple/RedirectStreams/testcases/RedirectStreamsTest.java
--- a/tests/reproducers/simple/RedirectStreams/testcases/RedirectStreamsTest.java	Mon Nov 25 10:16:42 2013 +0100
+++ b/tests/reproducers/simple/RedirectStreams/testcases/RedirectStreamsTest.java	Mon Nov 25 16:54:23 2013 +0100
@@ -51,7 +51,6 @@
         Assert.assertTrue("Stderr should match "+s+" but didn't",pr.stderr.matches(s));
         String cc="ClassNotFoundException";
         Assert.assertFalse("stderr should NOT contains `"+cc+"`, but did",pr.stderr.contains(cc));
-        Assert.assertFalse("stdout length should be <=2, but was "+pr.stdout.length(),pr.stdout.length()>2);
         Assert.assertFalse("RedirectStreams should not be terminated, but was",pr.wasTerminated);
         Assert.assertEquals((Integer) 0, pr.returnValue);
     }
diff -r bf80f38923af -r 349c86e05863 tests/reproducers/simple/ReplaceSecurityManager/testcases/ReplaceSecurityManagerTest.java
--- a/tests/reproducers/simple/ReplaceSecurityManager/testcases/ReplaceSecurityManagerTest.java	Mon Nov 25 10:16:42 2013 +0100
+++ b/tests/reproducers/simple/ReplaceSecurityManager/testcases/ReplaceSecurityManagerTest.java	Mon Nov 25 16:54:23 2013 +0100
@@ -51,7 +51,6 @@
         Assert.assertTrue("stderr should match "+s+" but didn't",pr.stderr.matches(s));
         String cc="ClassNotFoundException";
         Assert.assertFalse("stderr should NOT contains `"+cc+"`, but did",pr.stderr.contains(cc));
-        Assert.assertFalse("stdout length should be <=2, but was "+pr.stdout.length(),pr.stdout.length()>2);
         Assert.assertFalse("ReplaceSecurityManagerLunch1 should not be terminated, but was",pr.wasTerminated);
         Assert.assertEquals((Integer) 0, pr.returnValue);
     }
diff -r bf80f38923af -r 349c86e05863 tests/reproducers/simple/SetContextClassLoader/testcases/SetContextClassLoaderTest.java
--- a/tests/reproducers/simple/SetContextClassLoader/testcases/SetContextClassLoaderTest.java	Mon Nov 25 10:16:42 2013 +0100
+++ b/tests/reproducers/simple/SetContextClassLoader/testcases/SetContextClassLoaderTest.java	Mon Nov 25 16:54:23 2013 +0100
@@ -51,7 +51,6 @@
         Assert.assertTrue("stderr should match "+s+" but didn't",pr.stderr.matches(s));
         String cc="ClassNotFoundException";
         Assert.assertFalse("stderr should NOT contains `"+cc+"`, but did",pr.stderr.contains(cc));
-        Assert.assertFalse("stdout length should be <=2, but was "+pr.stdout.length(),pr.stdout.length()>2);
         Assert.assertFalse("SetContextClassLoader1 should not be terminated, but was",pr.wasTerminated);
         Assert.assertEquals((Integer) 0, pr.returnValue);
     }
diff -r bf80f38923af -r 349c86e05863 tests/reproducers/simple/simpletest2/testcases/SimpleTest2Test.java
--- a/tests/reproducers/simple/simpletest2/testcases/SimpleTest2Test.java	Mon Nov 25 10:16:42 2013 +0100
+++ b/tests/reproducers/simple/simpletest2/testcases/SimpleTest2Test.java	Mon Nov 25 16:54:23 2013 +0100
@@ -49,7 +49,6 @@
     @Test
     public void testSimpletest2lunchException() throws Exception {
         ProcessResult pr=server.executeJavawsHeadless(null,"/simpletest2.jnlp");
-        Assert.assertTrue("stdout should be < 1 , but was "+pr.stdout.trim().length(),pr.stdout.trim().length() < 1);
         String s="Correct exception";
         Assert.assertTrue("stderr should contains "+s+" but didn't",pr.stderr.contains(s));
         String ss="Exception";


More information about the distro-pkg-dev mailing list