On Fri, 09 Nov 2007 19:32:36 +0100, Denis Golovin <dgolovin(a)exadel.com>
wrote:
Hi max,
yes we shiouldn't reuse projects if there were modifications during the
test, but what ther reason to create new project before every test is
executed, it worked this way before:
test.setUp();
test.test1()
test.tearDown()
test.setUp();
test.test2()
test.tearDown()
...
test.setUp();
test.testN()
test.tearDown()
None other than it was the best way to guarantee it was runing from a
correct project and that creation/deletion could be done often.
now it works the same way, but project isn't created if it is
already
there.
there is a problem, that it is not deleted at all, but we have an issue
about deleting projects
http://jira.jboss.org/jira/browse/JBIDE-1012
This issue is windows specific - jars won't be locked under linux.
but we need to figure out who is locking those jars - it is not hibernate
during the tests.
I did it yesterday to looks how it works on hudson, because I always
have problems during tearDown() with deleting projects.
I would say the better way is using TestSetup decorator, that start
setUp() and tearDown() only once for TestCase it should work for this
particular case and tests will be ran faster.
sure - we just need to check that the tests is actually not dependent on
them being deleted.
The tests also create other resources than the projects that needs to be
deleted..currently you disabled that deletion too.
Also I know that we have real problem with mozilla tests, it just
wont
work.
look at error log for JUnit tests below, that happens all the time for
mozilla tests. I will look at it today on linux.
That looks like a NPE that is causing the vm to exit...Seen that many
times in the various VPE jira reports - I hoped that was only under the
old JNI code ...apparently it is not ;(
/max
eclipse-test:
[java] !SESSION 2007-11-08 19:52:02.216
-----------------------------------------------
[java] eclipse.buildId=M20070921-1145
[java] java.version=1.5.0_12
[java] java.vendor=Sun Microsystems Inc.
[java] BootLoader constants: OS=linux, ARCH=x86, WS=gtk, NL=en_US
[java] Framework arguments: -application
org.eclipse.test.uitestapplication
formatter=org.apache.tools.ant.taskdefs.optional.junit.XMLJUnitResultFormatter,org.jboss.tools.jsf.vpe.richfaces.test.RichFacesAllTests.xml
-testPluginName org.jboss.tools.jsf.vpe.richfaces.test -classname
org.jboss.tools.jsf.vpe.richfaces.test.RichFacesAllTests
[java] Command-line arguments: -application
org.eclipse.test.uitestapplication -dev bin -data
/home/hudson/jbosstools-build/200711081857-nightly/tests/workspace
formatter=org.apache.tools.ant.taskdefs.optional.junit.XMLJUnitResultFormatter,org.jboss.tools.jsf.vpe.richfaces.test.RichFacesAllTests.xml
-testPluginName org.jboss.tools.jsf.vpe.richfaces.test -classname
org.jboss.tools.jsf.vpe.richfaces.test.RichFacesAllTests -consolelog
[java] !ENTRY org.eclipse.wst.internet.cache 1 0 2007-11-08
19:52:42.854
[java] !MESSAGE wtp.autotest.noninteractive is set. Licenses
dialogs will not be displayed.
[java] !ENTRY org.eclipse.wst.sse.ui 4 4 2007-11-08 19:52:50.430
[java] !MESSAGE problem with reconciling
[java] !STACK 0
[java] java.lang.NullPointerException
[java] at
org.eclipse.wst.sse.ui.internal.reconcile.validator.ValidatorStrategy.reconcile(ValidatorStrategy.java:205)
[java] at
org.eclipse.wst.sse.ui.internal.reconcile.DocumentRegionProcessor.process(DocumentRegionProcessor.java:183)
[java] at
org.eclipse.wst.sse.ui.internal.reconcile.StructuredRegionProcessor.process(StructuredRegionProcessor.java:252)
[java] at
org.eclipse.wst.sse.ui.internal.reconcile.DirtyRegionProcessor.run(DirtyRegionProcessor.java:633)
[java] at org.eclipse.core.internal.jobs.Worker.run(Worker.java:55)
[java] !ENTRY org.eclipse.core.jobs 4 2 2007-11-08 19:52:50.597
[java] !MESSAGE An internal error occurred during: "Processing
Dirty Regions".
[java] !STACK 0
[java] java.lang.NullPointerException
[java] at
org.eclipse.wst.sse.ui.internal.reconcile.validator.ValidatorStrategy.endProcessing(ValidatorStrategy.java:140)
[java] at
org.eclipse.wst.sse.ui.internal.reconcile.DocumentRegionProcessor.endProcessing(DocumentRegionProcessor.java:71)
[java] at
org.eclipse.wst.sse.ui.internal.reconcile.DirtyRegionProcessor.run(DirtyRegionProcessor.java:647)
[java] at org.eclipse.core.internal.jobs.Worker.run(Worker.java:55)
[java] #
[java] # An unexpected error has been detected by HotSpot Virtual
Machine:
[java] #
[java] # SIGSEGV (0xb) at pc=0xc844aff4, pid=14079, tid=4160620224
[java] #
[java] # Java VM: Java HotSpot(TM) Server VM (1.5.0_12-b04 mixed
mode)
[java] # Problematic frame:
[java] # C [libexpat.so.0+0xeff4]
[java] #
[java] # An error report file with more information is saved as
hs_err_pid14079.log
[java] #
[java] # If you would like to submit a bug report, please visit:
[java] #
http://java.sun.com/webapps/bugreport/crash.jsp
[java] #
[java] Java Result: 134
[echo] Cleaning test workspace...
[delete] Deleting 412 files from
/home/hudson/jbosstools-build/200711081857-nightly/tests/workspace
[delete] Deleted 222 directories from
/home/hudson/jbosstools-build/200711081857-nightly/tests/workspace
[echo] performing "core" test on plugin "org.jboss.tools.tests"
class "org.jboss.tools.tests.ToolsAllTests"
Max Rydahl Andersen wrote:
> hi Denis,
>
> This is a broken fix - we shouldn't reuse the projects since they can
> be different.
>
> Why can't the project be deleted ? That works fine here...
>
> /max
>
>
> ------- Forwarded message -------
> From: jbosstools-commits(a)lists.jboss.org
> To: jbosstools-commits(a)lists.jboss.org
> Cc:
> Subject: [jbosstools-commits] JBoss Tools SVN: r4799 -
>
trunk/seam/tests/org.jboss.tools.seam.core.test/src/org/jboss/tools/seam/core/test/project/facet.
> Date: Thu, 08 Nov 2007 22:02:59 +0100
>
> Author: dgolovin
> Date: 2007-11-08 16:02:59 -0500 (Thu, 08 Nov 2007)
> New Revision: 4799
>
> Modified:
>
>
trunk/seam/tests/org.jboss.tools.seam.core.test/src/org/jboss/tools/seam/core/test/project/facet/AbstractSeamFacetTest.java
>
>
trunk/seam/tests/org.jboss.tools.seam.core.test/src/org/jboss/tools/seam/core/test/project/facet/Seam2FacetInstallDelegateTest.java
> Log:
> to fix exception 1 below test switched to using once created project.
> setUp just check if it is created and use it or create a new one.
>
> delete project is commented out in abstract class, because it always
> shrows exceptions cannot delete project.
>
> Exception 1
> --------------
> Resource '/warprj' already exists.
>
> org.eclipse.core.internal.resources.ResourceException: Resource
> '/warprj' already exists.
> at
> org.eclipse.core.internal.resources.Resource.checkDoesNotExist(Resource.java:289)
> at
> org.eclipse.core.internal.resources.Resource.checkDoesNotExist(Resource.java:276)
> at org.eclipse.core.internal.resources.Project.create(Project.java:225)
> at
>
org.eclipse.wst.common.project.facet.core.internal.FacetedProjectFrameworkImpl.create(FacetedProjectFrameworkImpl.java:717)
> at
>
org.eclipse.wst.common.project.facet.core.ProjectFacetsManager.create(ProjectFacetsManager.java:399)
> at
>
org.jboss.tools.seam.core.test.project.facet.AbstractSeamFacetTest.createSeamProject(AbstractSeamFacetTest.java:142)
> at
>
org.jboss.tools.seam.core.test.project.facet.AbstractSeamFacetTest.createSeamWarProject(AbstractSeamFacetTest.java:157)
> at
>
org.jboss.tools.seam.core.test.project.facet.Seam2FacetInstallDelegateTest.setUp(Seam2FacetInstallDelegateTest.java:62)
> at org.eclipse.test.EclipseTestRunner.run(EclipseTestRunner.java:354)
> at org.eclipse.test.EclipseTestRunner.run(EclipseTestRunner.java:206)
> at
> org.eclipse.test.CoreTestApplication.runTests(CoreTestApplication.java:35)
> at org.eclipse.test.CoreTestApplication.run(CoreTestApplication.java:31)
> at
>
org.eclipse.equinox.internal.app.EclipseAppContainer.callMethod(EclipseAppContainer.java:572)
> at
> org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:171)
> at
>
org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:106)
> at
>
org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:76)
> at
> org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:363)
> at
> org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:176)
> at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:508)
> at org.eclipse.equinox.launcher.Main.basicRun(Main.java:447)
> at org.eclipse.equinox.launcher.Main.run(Main.java:1173)
> at org.eclipse.equinox.launcher.Main.main(Main.java:1148)
> at org.eclipse.core.launcher.Main.main(Main.java:30)
>
> Modified:
>
trunk/seam/tests/org.jboss.tools.seam.core.test/src/org/jboss/tools/seam/core/test/project/facet/AbstractSeamFacetTest.java
> ===================================================================
> ---
>
trunk/seam/tests/org.jboss.tools.seam.core.test/src/org/jboss/tools/seam/core/test/project/facet/AbstractSeamFacetTest.java
> 2007-11-08 20:51:11 UTC (rev 4798)
> +++
>
trunk/seam/tests/org.jboss.tools.seam.core.test/src/org/jboss/tools/seam/core/test/project/facet/AbstractSeamFacetTest.java
> 2007-11-08 21:02:59 UTC (rev 4799)
> @@ -81,7 +81,7 @@
> for (IResource r : this.resourcesToCleanup) {
> try {
> System.out.println("Deleting " + r);
> - r.delete(true, null);
> + //r.delete(true, null);
> } catch(Exception e) {
> System.out.println("Error deleting " + r);
> e.printStackTrace();
>
> Modified:
>
trunk/seam/tests/org.jboss.tools.seam.core.test/src/org/jboss/tools/seam/core/test/project/facet/Seam2FacetInstallDelegateTest.java
> ===================================================================
> ---
>
trunk/seam/tests/org.jboss.tools.seam.core.test/src/org/jboss/tools/seam/core/test/project/facet/Seam2FacetInstallDelegateTest.java
> 2007-11-08 20:51:11 UTC (rev 4798)
> +++
>
trunk/seam/tests/org.jboss.tools.seam.core.test/src/org/jboss/tools/seam/core/test/project/facet/Seam2FacetInstallDelegateTest.java
> 2007-11-08 21:02:59 UTC (rev 4799)
> @@ -21,6 +21,8 @@
> import org.eclipse.core.resources.IResource;
> import org.eclipse.core.resources.IResourceProxy;
> import org.eclipse.core.resources.IResourceProxyVisitor;
> +import org.eclipse.core.resources.IWorkspaceRoot;
> +import org.eclipse.core.resources.ResourcesPlugin;
> import org.eclipse.core.runtime.CoreException;
> import org.eclipse.wst.common.frameworks.datamodel.IDataModel;
> import org.eclipse.wst.common.project.facet.core.IFacetedProject;
> @@ -50,22 +52,29 @@
> @Override
> protected void setUp() throws Exception {
> - - seam2Facet =
> ProjectFacetsManager.getProjectFacet("jst.seam");
> - seam2FacetVersion = seam2Facet.getVersion("2.0");
> + IWorkspaceRoot wsRoot =
> ResourcesPlugin.getWorkspace().getRoot();
> + IProject war = (IProject)wsRoot.findMember("warprj");
> + IProject ear = (IProject)wsRoot.findMember("earprj");
> - File folder = new
> File(System.getProperty("jbosstools.test.seam.2.0.0.home",
> "/home/max/work/products/jboss-seam-2.0.0.CR2"));
> + if(war==null && ear==null) {
> -
> SeamRuntimeManager.getInstance().addRuntime(SEAM_2_0_0,
> folder.getAbsolutePath(), SeamVersion.SEAM_2_0, true);
> - SeamRuntimeManager.getInstance().findRuntimeByName(SEAM_2_0_0);
> - - warProject = createSeamWarProject("warprj");
> - earProject = createSeamEarProject("earprj");
> + seam2Facet =
> ProjectFacetsManager.getProjectFacet("jst.seam");
> + seam2FacetVersion = seam2Facet.getVersion("2.0");
> + + File folder = new
> File(System.getProperty("jbosstools.test.seam.2.0.0.home",
> "c:/java/jboss-seam-2.0.0.CR3"));
> + +
> SeamRuntimeManager.getInstance().addRuntime(SEAM_2_0_0,
> folder.getAbsolutePath(), SeamVersion.SEAM_2_0, true);
> +
> SeamRuntimeManager.getInstance().findRuntimeByName(SEAM_2_0_0);
> -
>
warProject.getProject().getWorkspace().getRoot().refreshLocal(IResource.DEPTH_INFINITE,
> null);
> + warProject = createSeamWarProject("warprj");
> + earProject = createSeamEarProject("earprj");
> + +
>
warProject.getProject().getWorkspace().getRoot().refreshLocal(IResource.DEPTH_INFINITE,
> null);
> + } else {
> + warProject = ProjectFacetsManager.create(war);
> + earProject = ProjectFacetsManager.create(ear);
> + }
> - - super.setUp();
> }
> @@ -87,6 +96,9 @@
> warlibs.add("antlr-runtime.jar");
> warlibs.add("commons-beanutils.jar");
> warlibs.add("commons-digester.jar");
> + warlibs.add("commons-collections.jar");
> + warlibs.add("janino.jar");
> + warlibs.add("antlr.jar");
> warlibs.add("drools-compiler.jar");
> warlibs.add("drools-core.jar");
> warlibs.add("core.jar");
>
> _______________________________________________
> jbosstools-commits mailing list
> jbosstools-commits(a)lists.jboss.org
>
https://lists.jboss.org/mailman/listinfo/jbosstools-commits
>
>
>
--
--
Max Rydahl Andersen
callto://max.rydahl.andersen
Hibernate
max(a)hibernate.org
http://hibernate.org
JBoss a division of Red Hat
max.andersen(a)jboss.com