Author: sohil.shah(a)jboss.com Date: 2009-07-11 14:09:49 -0400 (Sat, 11 Jul 2009) New Revision: 13544 Modified: modules/authorization/trunk/agent/src/main/java/org/jboss/security/authz/agent/services/PolicyComposer.java modules/authorization/trunk/http-profile/src/test/java/org/jboss/security/authz/http/components/TestParameterMatching.java Log: establishing baseline before fixing the testsuite with the new framework Modified: modules/authorization/trunk/agent/src/main/java/org/jboss/security/authz/agent/services/PolicyComposer.java =================================================================== --- modules/authorization/trunk/agent/src/main/java/org/jboss/security/authz/agent/services/PolicyComposer.java 2009-07-11 18:00:13 UTC (rev 13543) +++ modules/authorization/trunk/agent/src/main/java/org/jboss/security/authz/agent/services/PolicyComposer.java 2009-07-11 18:09:49 UTC (rev 13544) @@ -54,10 +54,10 @@ throw new IllegalStateException("Policy Target is missing!!"); } - /*if(policyRules == null || policyRules.isEmpty()) + if(policyRules == null || policyRules.isEmpty()) { throw new IllegalStateException("Policy Rule(s) are missing!!"); - }*/ + } Target target = policyTarget.compose(); Set<Rule> rules = new HashSet<Rule>(); Modified: modules/authorization/trunk/http-profile/src/test/java/org/jboss/security/authz/http/components/TestParameterMatching.java =================================================================== --- modules/authorization/trunk/http-profile/src/test/java/org/jboss/security/authz/http/components/TestParameterMatching.java 2009-07-11 18:00:13 UTC (rev 13543) +++ modules/authorization/trunk/http-profile/src/test/java/org/jboss/security/authz/http/components/TestParameterMatching.java 2009-07-11 18:09:49 UTC (rev 13544) @@ -77,7 +77,7 @@ //Setup the Context for the Composition with these components CompositionContext context = new CompositionContext(); context.setPolicyTarget(policyResource); - //context.addPolicyRule(Effect.PERMIT, new Get(), allowedRoles, "allowExpression"); + context.addPolicyRule(Effect.PERMIT, new Get(), allowedRoles, "allowExpression"); //Store the policy into the Policy Server PolicyMetaData policyMetaData = this.policyComposer.compose(context);