[
https://issues.apache.org/jira/browse/DERBY-5631?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Kristian Waagan updated DERBY-5631:
-----------------------------------
Attachment: derby-5631-1b-merge_policy_files.diff
Attaching patch 1b, which replaces patch 1a.
I've fixed the whitespace issues commented on by Dag, and I've also rewritten
the method as suggested by Dag. Made the method static, updated some comments,
and added a debug println:
DEBUG: {SecurityManagerSetup} installed policy
file:/C:/.../derby/test/system/var/AssertFailureTest.policy-MERGED_WITH-derby_tests.policy
DEBUG: {SecurityManagerSetup} installed policy
jar:file:/C:/.../derby/jars/sane/derbyTesting.jar!/org/apache/derbyTesting/unitTests/junit/AssertFailureTest1.policy
Patch ready for review.
I hope to commit patch 1b soon.
> Extend SecurityManagerSetup to add extra privileges to the set of default
> privileges (merge two policy files)
> -------------------------------------------------------------------------------------------------------------
>
> Key: DERBY-5631
> URL: https://issues.apache.org/jira/browse/DERBY-5631
> Project: Derby
> Issue Type: Improvement
> Components: Test
> Affects Versions: 10.9.0.0
> Reporter: Kristian Waagan
> Assignee: Kristian Waagan
> Attachments: derby-5631-1a-merge_policy_files.diff,
> derby-5631-1b-merge_policy_files.diff,
> derby-5631-2a-introduce_NO_POLICY_constant.diff
>
>
> When moving ProtocolTest into suites all (see DERBY-2031), I needed to allow
> the test code to create a socket. There was already a policy file for the old
> harness for doing this, but when using this with the JUnit framework I ended
> up adding privilege after privilege to get the framework itself running.
> Instead of creating a large policy file duplicating all the privileges that
> the framework requires to function properly, I think it would be better to
> make the SecurityManagerSetup capable of merging the test specific policy
> file with the default policy file.
> This mode of operation can be used when you need a few extra privileges to
> execute the test, but there are probably also tests where you want full
> control of the privileges - in which case you use the existing mode of
> operation.
> I'm not 100% sure this approach will always work, but basic testing has shown
> promising results.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
