« Return to Thread: [RFC][icedtea-web] Test for regression of ServiceManager not setup correctly
[RFC][icedtea-web] Test for regression of ServiceManager not setup correctly
by Danesh Dadachanji
::
Rate this Message:
Hi,
Here's a test that checks whether or not ServiceManager is setup
correctly by the time an applet constructor is called. The constructor
uses BasicService and will throw a NPE if it is not setup.
The test is for checking regression against this changeset:
http://icedtea.classpath.org/hg/icedtea-web/rev/221174bcd4ec
Thank you very much to Jiri for helping me get this through the door!
+2012-02-17 Danesh Dadachanji <ddadacha@...>
+
+ Adding test for regression of JNLP API accessibility in constructor
+ methods of applets.
+ * Makefile.am: Added classes.jar to classpath when compiling jnlp_tests.
+ * tests/jnlp_tests/simple/CheckServices/: Tests ServiceManager is setup
+ correctly when called from applet constructors.
+
Cheers,
Danesh
[test-applet-jnlp_href.patch]
diff --git a/Makefile.am b/Makefile.am
--- a/Makefile.am
+++ b/Makefile.am
@@ -485,7 +485,7 @@ stamps/netx-dist-tests-prepare-reproduce
cd "$(JNLP_TESTS_SRCDIR)/$$which/$$dir/srcs/" ; \
srcFiles=`find . -mindepth 1 -type f -name "*.java" | sed "s/.\/*//"` ; \
notSrcFiles=`find . -mindepth 1 -type f \! -name "*.java" | sed "s/.\/*//"` ; \
- $(BOOT_DIR)/bin/javac -d "$(JNLP_TESTS_DIR)/$$dir/" $$srcFiles ; \
+ $(BOOT_DIR)/bin/javac -cp $(NETX_DIR)/lib/classes.jar -d "$(JNLP_TESTS_DIR)/$$dir/" $$srcFiles ; \
if [ -n "$$notSrcFiles" ] ; then \
cp -R --parents "$$notSrcFiles" "$(JNLP_TESTS_DIR)/$$dir/" ; \
fi ; \
diff --git a/tests/jnlp_tests/simple/CheckServices/resources/CheckServices.jnlp b/tests/jnlp_tests/simple/CheckServices/resources/CheckServices.jnlp
new file mode 100644
--- /dev/null
+++ b/tests/jnlp_tests/simple/CheckServices/resources/CheckServices.jnlp
@@ -0,0 +1,57 @@
+<!--
+
+This file is part of IcedTea.
+
+IcedTea is free software; you can redistribute it and/or modify
+it under the terms of the GNU General Public License as published by
+the Free Software Foundation; either version 2, or (at your option)
+any later version.
+
+IcedTea is distributed in the hope that it will be useful, but
+WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with IcedTea; see the file COPYING. If not, write to the
+Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+02110-1301 USA.
+
+Linking this library statically or dynamically with other modules is
+making a combined work based on this library. Thus, the terms and
+conditions of the GNU General Public License cover the whole
+combination.
+
+As a special exception, the copyright holders of this library give you
+permission to link this library with independent modules to produce an
+executable, regardless of the license terms of these independent
+modules, and to copy and distribute the resulting executable under
+terms of your choice, provided that you also meet, for each linked
+independent module, the terms and conditions of the license of that
+module. An independent module is a module which is not derived from
+or based on this library. If you modify this library, you may extend
+this exception to your version of the library, but you are not
+obligated to do so. If you do not wish to do so, delete this
+exception statement from your version.
+
+ -->
+<?xml version="1.0" encoding="utf-8"?>
+<jnlp spec="1.0" href="CheckServices.jnlp" codebase=".">
+ <information>
+ <title>CheckServices</title>
+ <vendor>NetX</vendor>
+ <homepage href="http://jnlp.sourceforge.net/netx/"/>
+ <description>CheckServices</description>
+ <offline/>
+ </information>
+ <resources>
+ <j2se version="1.4+"/>
+ <jar href="CheckServices.jar"/>
+ </resources>
+ <applet-desc
+ documentBase="."
+ name="CheckServices"
+ main-class="CheckServices"
+ width="100"
+ height="100" />
+</jnlp>
diff --git a/tests/jnlp_tests/simple/CheckServices/srcs/CheckServices.java b/tests/jnlp_tests/simple/CheckServices/srcs/CheckServices.java
new file mode 100644
--- /dev/null
+++ b/tests/jnlp_tests/simple/CheckServices/srcs/CheckServices.java
@@ -0,0 +1,81 @@
+/* CheckServices.java
+Copyright (C) 2011 Red Hat, Inc.
+
+This file is part of IcedTea.
+
+IcedTea is free software; you can redistribute it and/or
+modify it under the terms of the GNU General Public License as published by
+the Free Software Foundation, version 2.
+
+IcedTea is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with IcedTea; see the file COPYING. If not, write to
+the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+02110-1301 USA.
+
+Linking this library statically or dynamically with other modules is
+making a combined work based on this library. Thus, the terms and
+conditions of the GNU General Public License cover the whole
+combination.
+
+As a special exception, the copyright holders of this library give you
+permission to link this library with independent modules to produce an
+executable, regardless of the license terms of these independent
+modules, and to copy and distribute the resulting executable under
+terms of your choice, provided that you also meet, for each linked
+independent module, the terms and conditions of the license of that
+module. An independent module is a module which is not derived from
+or based on this library. If you modify this library, you may extend
+this exception to your version of the library, but you are not
+obligated to do so. If you do not wish to do so, delete this
+exception statement from your version.
+ */
+
+import javax.jnlp.ServiceManager;
+import javax.jnlp.BasicService;
+import java.applet.Applet;
+
+
+public class CheckServices extends Applet {
+
+ public CheckServices() {
+
+ try {
+ BasicService basicService =
+ (BasicService)ServiceManager.lookup("javax.jnlp.BasicService");
+ // getCodeBase() will return null if ServiceManager does not
+ // have access to ApplicationInstance.
+ String codebase = basicService.getCodeBase().toString();
+ System.out.println("Codebase for applet was found: " + codebase);
+ } catch (NullPointerException npe) {
+ System.err.println("Exception occurred with null codebase.");
+ npe.printStackTrace();
+ } catch (Exception ex) {
+ System.err.println("Exception occurred (probably with ServiceManager).");
+ ex.printStackTrace();
+ }
+ }
+
+ @Override
+ public void init() {
+ System.out.println("Applet was initialized");
+ Thread killer = new Thread() {
+ public int n = 2000;
+
+ @Override
+ public void run() {
+ try {
+ Thread.sleep(n);
+ System.out.println("Applet killing itself after " + n + " ms of life");
+ System.exit(0);
+ } catch (Exception ex) {
+ }
+ }
+ };
+ killer.start();
+ }
+}
diff --git a/tests/jnlp_tests/simple/CheckServices/testcases/CheckServicesTests.java b/tests/jnlp_tests/simple/CheckServices/testcases/CheckServicesTests.java
new file mode 100644
--- /dev/null
+++ b/tests/jnlp_tests/simple/CheckServices/testcases/CheckServicesTests.java
@@ -0,0 +1,60 @@
+/* CheckServicesTests.java
+Copyright (C) 2011 Red Hat, Inc.
+
+This file is part of IcedTea.
+
+IcedTea is free software; you can redistribute it and/or
+modify it under the terms of the GNU General Public License as published by
+the Free Software Foundation, version 2.
+
+IcedTea is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with IcedTea; see the file COPYING. If not, write to
+the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+02110-1301 USA.
+
+Linking this library statically or dynamically with other modules is
+making a combined work based on this library. Thus, the terms and
+conditions of the GNU General Public License cover the whole
+combination.
+
+As a special exception, the copyright holders of this library give you
+permission to link this library with independent modules to produce an
+executable, regardless of the license terms of these independent
+modules, and to copy and distribute the resulting executable under
+terms of your choice, provided that you also meet, for each linked
+independent module, the terms and conditions of the license of that
+module. An independent module is a module which is not derived from
+or based on this library. If you modify this library, you may extend
+this exception to your version of the library, but you are not
+obligated to do so. If you do not wish to do so, delete this
+exception statement from your version.
+ */
+
+import net.sourceforge.jnlp.ServerAccess;
+import org.junit.Assert;
+import org.junit.Test;
+
+public class CheckServicesTests {
+
+ private static ServerAccess server = new ServerAccess();
+ @Test
+ public void CheckServices() throws Exception {
+ System.out.println("connecting CheckServices request");
+ System.err.println("connecting CheckServices request");
+ ServerAccess.ProcessResult pr = server.executeJavawsHeadless(null, "/CheckServices.jnlp");
+ System.out.println(pr.stdout);
+ System.err.println(pr.stderr);
+ String s0 = "Codebase for applet was found";
+ Assert.assertTrue("CheckServices stdout should contain `" + s0 + "' bud didn't.", pr.stdout.contains(s0));
+ String s1 = "Exception occurred with null codebase.";
+ Assert.assertFalse("CheckServices stderr should not contain `" + s1 + "' bud did.", pr.stdout.contains(s1));
+ String s2 = "Applet killing itself after 2000 ms of life";
+ Assert.assertTrue("CheckServices stdout should contain `" + s2 + "' bud didn't.", pr.stdout.contains(s2));
+ Assert.assertEquals((Integer)0, pr.returnValue);
+ }
+}
Here's a test that checks whether or not ServiceManager is setup
correctly by the time an applet constructor is called. The constructor
uses BasicService and will throw a NPE if it is not setup.
The test is for checking regression against this changeset:
http://icedtea.classpath.org/hg/icedtea-web/rev/221174bcd4ec
Thank you very much to Jiri for helping me get this through the door!
+2012-02-17 Danesh Dadachanji <ddadacha@...>
+
+ Adding test for regression of JNLP API accessibility in constructor
+ methods of applets.
+ * Makefile.am: Added classes.jar to classpath when compiling jnlp_tests.
+ * tests/jnlp_tests/simple/CheckServices/: Tests ServiceManager is setup
+ correctly when called from applet constructors.
+
Cheers,
Danesh
[test-applet-jnlp_href.patch]
diff --git a/Makefile.am b/Makefile.am
--- a/Makefile.am
+++ b/Makefile.am
@@ -485,7 +485,7 @@ stamps/netx-dist-tests-prepare-reproduce
cd "$(JNLP_TESTS_SRCDIR)/$$which/$$dir/srcs/" ; \
srcFiles=`find . -mindepth 1 -type f -name "*.java" | sed "s/.\/*//"` ; \
notSrcFiles=`find . -mindepth 1 -type f \! -name "*.java" | sed "s/.\/*//"` ; \
- $(BOOT_DIR)/bin/javac -d "$(JNLP_TESTS_DIR)/$$dir/" $$srcFiles ; \
+ $(BOOT_DIR)/bin/javac -cp $(NETX_DIR)/lib/classes.jar -d "$(JNLP_TESTS_DIR)/$$dir/" $$srcFiles ; \
if [ -n "$$notSrcFiles" ] ; then \
cp -R --parents "$$notSrcFiles" "$(JNLP_TESTS_DIR)/$$dir/" ; \
fi ; \
diff --git a/tests/jnlp_tests/simple/CheckServices/resources/CheckServices.jnlp b/tests/jnlp_tests/simple/CheckServices/resources/CheckServices.jnlp
new file mode 100644
--- /dev/null
+++ b/tests/jnlp_tests/simple/CheckServices/resources/CheckServices.jnlp
@@ -0,0 +1,57 @@
+<!--
+
+This file is part of IcedTea.
+
+IcedTea is free software; you can redistribute it and/or modify
+it under the terms of the GNU General Public License as published by
+the Free Software Foundation; either version 2, or (at your option)
+any later version.
+
+IcedTea is distributed in the hope that it will be useful, but
+WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with IcedTea; see the file COPYING. If not, write to the
+Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+02110-1301 USA.
+
+Linking this library statically or dynamically with other modules is
+making a combined work based on this library. Thus, the terms and
+conditions of the GNU General Public License cover the whole
+combination.
+
+As a special exception, the copyright holders of this library give you
+permission to link this library with independent modules to produce an
+executable, regardless of the license terms of these independent
+modules, and to copy and distribute the resulting executable under
+terms of your choice, provided that you also meet, for each linked
+independent module, the terms and conditions of the license of that
+module. An independent module is a module which is not derived from
+or based on this library. If you modify this library, you may extend
+this exception to your version of the library, but you are not
+obligated to do so. If you do not wish to do so, delete this
+exception statement from your version.
+
+ -->
+<?xml version="1.0" encoding="utf-8"?>
+<jnlp spec="1.0" href="CheckServices.jnlp" codebase=".">
+ <information>
+ <title>CheckServices</title>
+ <vendor>NetX</vendor>
+ <homepage href="http://jnlp.sourceforge.net/netx/"/>
+ <description>CheckServices</description>
+ <offline/>
+ </information>
+ <resources>
+ <j2se version="1.4+"/>
+ <jar href="CheckServices.jar"/>
+ </resources>
+ <applet-desc
+ documentBase="."
+ name="CheckServices"
+ main-class="CheckServices"
+ width="100"
+ height="100" />
+</jnlp>
diff --git a/tests/jnlp_tests/simple/CheckServices/srcs/CheckServices.java b/tests/jnlp_tests/simple/CheckServices/srcs/CheckServices.java
new file mode 100644
--- /dev/null
+++ b/tests/jnlp_tests/simple/CheckServices/srcs/CheckServices.java
@@ -0,0 +1,81 @@
+/* CheckServices.java
+Copyright (C) 2011 Red Hat, Inc.
+
+This file is part of IcedTea.
+
+IcedTea is free software; you can redistribute it and/or
+modify it under the terms of the GNU General Public License as published by
+the Free Software Foundation, version 2.
+
+IcedTea is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with IcedTea; see the file COPYING. If not, write to
+the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+02110-1301 USA.
+
+Linking this library statically or dynamically with other modules is
+making a combined work based on this library. Thus, the terms and
+conditions of the GNU General Public License cover the whole
+combination.
+
+As a special exception, the copyright holders of this library give you
+permission to link this library with independent modules to produce an
+executable, regardless of the license terms of these independent
+modules, and to copy and distribute the resulting executable under
+terms of your choice, provided that you also meet, for each linked
+independent module, the terms and conditions of the license of that
+module. An independent module is a module which is not derived from
+or based on this library. If you modify this library, you may extend
+this exception to your version of the library, but you are not
+obligated to do so. If you do not wish to do so, delete this
+exception statement from your version.
+ */
+
+import javax.jnlp.ServiceManager;
+import javax.jnlp.BasicService;
+import java.applet.Applet;
+
+
+public class CheckServices extends Applet {
+
+ public CheckServices() {
+
+ try {
+ BasicService basicService =
+ (BasicService)ServiceManager.lookup("javax.jnlp.BasicService");
+ // getCodeBase() will return null if ServiceManager does not
+ // have access to ApplicationInstance.
+ String codebase = basicService.getCodeBase().toString();
+ System.out.println("Codebase for applet was found: " + codebase);
+ } catch (NullPointerException npe) {
+ System.err.println("Exception occurred with null codebase.");
+ npe.printStackTrace();
+ } catch (Exception ex) {
+ System.err.println("Exception occurred (probably with ServiceManager).");
+ ex.printStackTrace();
+ }
+ }
+
+ @Override
+ public void init() {
+ System.out.println("Applet was initialized");
+ Thread killer = new Thread() {
+ public int n = 2000;
+
+ @Override
+ public void run() {
+ try {
+ Thread.sleep(n);
+ System.out.println("Applet killing itself after " + n + " ms of life");
+ System.exit(0);
+ } catch (Exception ex) {
+ }
+ }
+ };
+ killer.start();
+ }
+}
diff --git a/tests/jnlp_tests/simple/CheckServices/testcases/CheckServicesTests.java b/tests/jnlp_tests/simple/CheckServices/testcases/CheckServicesTests.java
new file mode 100644
--- /dev/null
+++ b/tests/jnlp_tests/simple/CheckServices/testcases/CheckServicesTests.java
@@ -0,0 +1,60 @@
+/* CheckServicesTests.java
+Copyright (C) 2011 Red Hat, Inc.
+
+This file is part of IcedTea.
+
+IcedTea is free software; you can redistribute it and/or
+modify it under the terms of the GNU General Public License as published by
+the Free Software Foundation, version 2.
+
+IcedTea is distributed in the hope that it will be useful,
+but WITHOUT ANY WARRANTY; without even the implied warranty of
+MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+General Public License for more details.
+
+You should have received a copy of the GNU General Public License
+along with IcedTea; see the file COPYING. If not, write to
+the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+02110-1301 USA.
+
+Linking this library statically or dynamically with other modules is
+making a combined work based on this library. Thus, the terms and
+conditions of the GNU General Public License cover the whole
+combination.
+
+As a special exception, the copyright holders of this library give you
+permission to link this library with independent modules to produce an
+executable, regardless of the license terms of these independent
+modules, and to copy and distribute the resulting executable under
+terms of your choice, provided that you also meet, for each linked
+independent module, the terms and conditions of the license of that
+module. An independent module is a module which is not derived from
+or based on this library. If you modify this library, you may extend
+this exception to your version of the library, but you are not
+obligated to do so. If you do not wish to do so, delete this
+exception statement from your version.
+ */
+
+import net.sourceforge.jnlp.ServerAccess;
+import org.junit.Assert;
+import org.junit.Test;
+
+public class CheckServicesTests {
+
+ private static ServerAccess server = new ServerAccess();
+ @Test
+ public void CheckServices() throws Exception {
+ System.out.println("connecting CheckServices request");
+ System.err.println("connecting CheckServices request");
+ ServerAccess.ProcessResult pr = server.executeJavawsHeadless(null, "/CheckServices.jnlp");
+ System.out.println(pr.stdout);
+ System.err.println(pr.stderr);
+ String s0 = "Codebase for applet was found";
+ Assert.assertTrue("CheckServices stdout should contain `" + s0 + "' bud didn't.", pr.stdout.contains(s0));
+ String s1 = "Exception occurred with null codebase.";
+ Assert.assertFalse("CheckServices stderr should not contain `" + s1 + "' bud did.", pr.stdout.contains(s1));
+ String s2 = "Applet killing itself after 2000 ms of life";
+ Assert.assertTrue("CheckServices stdout should contain `" + s2 + "' bud didn't.", pr.stdout.contains(s2));
+ Assert.assertEquals((Integer)0, pr.returnValue);
+ }
+}
« Return to Thread: [RFC][icedtea-web] Test for regression of ServiceManager not setup correctly
| Free embeddable forum powered by Nabble | Forum Help |
