[jbpm-issues] [JBoss JIRA] Commented: (JBPM-1201) Resurrect MemLeakTest

Resurrect MemLeakTest --------------------- Key: JBPM-1201 URL: http://jira.jboss.com/jira/browse/JBPM-1201 Project: JBoss jBPM Issue Type: Task Security Level: Public(Everyone can see) Reporter: Thomas Diesler [15:54:19] Thomas Diesler: there seems to be a circular dependency in modules identity and jpdl/jar [15:54:42] ... how should it be? [15:54:52] Tom Baeyens: good question [15:55:20] ... jpdl jar should not need identity [15:55:38] ... but for building the suite, the identity is needed [15:55:40] Thomas Diesler: yes, I found that [15:55:54] Tom Baeyens: building the suite should be in dist [15:56:01] Thomas Diesler: yes, I found that too [15:56:24] ... I am swithcing the build to maven [15:56:32] Tom Baeyens: where did you find the circular dep ? [15:56:52] Thomas Diesler: can't the identity tests live in module identity? [15:57:19] Tom Baeyens: are there identity tests in jpdl/jar ? [15:57:30] Thomas Diesler: yes [15:57:48] Tom Baeyens: probably there is some reason i can't remember now [15:57:59] ... i'll see if i can quickly re-find it [15:58:19] Thomas Diesler: ok, but identity depends on jpdl-core, right? [15:58:32] ... don't worry [15:58:40] ... I'll sort it out [15:59:30] Tom Baeyens: MemLeakTest ? [15:59:50] Thomas Diesler: yes [16:00:17] Tom Baeyens: just delete the test [16:00:19] ... :-) [16:00:46] ... it's a kind of integration test [16:01:00] ... which i guess is not part of the continuous integration [16:01:05] Thomas Diesler: with all the other props and stuff? [16:01:24] Tom Baeyens: other props ? [16:01:46] Thomas Diesler: next to the test are lots of config files [16:01:52] Tom Baeyens: right [16:02:01] Thomas Diesler: delete them too? [16:02:04] Tom Baeyens: maybe it makes sense to put the perf package into a separate module [16:02:28] ... you can delete all of it [16:02:41] ... but in case we ever want to do performance tests [16:02:51] ... then those could provide a good start