-
1. Re: jboss/test-1.0.4GA broke tests
starksm64 Dec 21, 2007 12:29 PM (in response to flavia.rainone)You have to be running with a security manager to see permission failures. Create a bug report with a test that fails before/after as there should not be any change in behavior as far as I know.
-
2. Re: jboss/test-1.0.4GA broke tests
anil.saldhana Dec 21, 2007 12:33 PM (in response to flavia.rainone)It needs to be ensured that by default the following call:
System.setSecurityManager(SecurityManager sm)
does not exist in the test infrastructure anywhere (aop or jbosstest). -
3. Re: jboss/test-1.0.4GA broke tests
anil.saldhana Dec 21, 2007 12:51 PM (in response to flavia.rainone)JBossTest 1.0.3 was released 9 months ago:
http://viewvc.jboss.org/cgi-bin/viewvc.cgi/jbossas/projects/test/tags/
The test code has not had major changes.
The only change I see that Adrian made was to make the SecurityManager setup methods static.
http://viewvc.jboss.org/cgi-bin/viewvc.cgi/jbossas/projects/test/trunk/src/main/java/org/jboss/test/AbstractTestCaseWithSetup.java?r1=62401&r2=62649
So if you say that an upgrade of JBossTest from 1.0.3 to 1.0.4 introduces the security manager, then you may be affected by this change that Adrian has:
http://viewvc.jboss.org/cgi-bin/viewvc.cgi/jbossas/projects/test/trunk/src/test/java/org/jboss/test/security/test/NothingSecurityUnitTestCase.java?view=log
The right person to answer is Adrian. :) -
4. Re: jboss/test-1.0.4GA broke tests
flavia.rainone Dec 21, 2007 1:10 PM (in response to flavia.rainone)Thank's for the replies :)
I'm doing a release of JBoss AOP and basically, I have to update the library versions to the latest ones before closing the release.
Now, after taking a look at the release date of jboss-test 1.0.4.GA, I see that there is something wrong. Since it has been released on September, this version should already be being used by JBoss AOP previous release...
If it is not updated, it is because there is really something wrong. So now we need an answer from Adrian and from Kabir to know what is going on :)
Taking all this into consideration, I'll do the release with the 1.0.3GA version of the library. -
5. Re: jboss/test-1.0.4GA broke tests
anil.saldhana Dec 21, 2007 1:13 PM (in response to flavia.rainone)From Adrian's commit message, it seems that the old security policy (maybe test 1.0.3) was too lenient and he made it more stringent. So even with JBossTest 1.0.3, you may already be running under a security manager for your AOP tests with a relaxed policy.
-
6. Re: jboss/test-1.0.4GA broke tests
flavia.rainone Dec 21, 2007 2:27 PM (in response to flavia.rainone)Unfortunately, I can't see any security policy being enabled in the AOP tests.
So, I'm assuming that there is no security manager enabled. -
7. Re: jboss/test-1.0.4GA broke tests
flavia.rainone Dec 21, 2007 3:07 PM (in response to flavia.rainone)I have found out why Kabir hasn't used jboss-test 1.0.4GA on the previous relase.
This is the comment of a commit he did a while ago:Revert back to test 1.0.3.GA since 1.0.4.GA is not in the repository
-
8. Re: jboss/test-1.0.4GA broke tests
adrian.brock Jan 7, 2008 8:43 AM (in response to flavia.rainone)"anil.saldhana@jboss.com" wrote:
The right person to answer is Adrian. :)
I can't answer because there is nothiing specific in the question.
After updating the version of jboss-test.jar from 1.0.3.GA to 1.0.4.GA, I started getting a bunch of security permission failures.
Does anybody know if this is expected?
P.S. I didn't change the way this works by default. I just added a better way
to configure alternative security poliicies.
Most likely it is the chage to the codesource checking that has caught you out.
i.e. previously you weren't testing with the proper privileges for the tests.
This would indicate you are missing some privileged blocks.