NIHVIVO-3404 Adjust the clients to use the most convenient constructor of EditDataPropertyStatement
This commit is contained in:
parent
b0c7241ede
commit
ed2f3ddc12
2 changed files with 7 additions and 37 deletions
|
@ -36,8 +36,6 @@ import edu.cornell.mannlib.vitro.webapp.auth.requestedAction.propstmt.AddObjectP
|
||||||
import edu.cornell.mannlib.vitro.webapp.auth.requestedAction.propstmt.DropObjectPropertyStatement;
|
import edu.cornell.mannlib.vitro.webapp.auth.requestedAction.propstmt.DropObjectPropertyStatement;
|
||||||
import edu.cornell.mannlib.vitro.webapp.auth.requestedAction.propstmt.EditDataPropertyStatement;
|
import edu.cornell.mannlib.vitro.webapp.auth.requestedAction.propstmt.EditDataPropertyStatement;
|
||||||
import edu.cornell.mannlib.vitro.webapp.auth.requestedAction.propstmt.EditObjectPropertyStatement;
|
import edu.cornell.mannlib.vitro.webapp.auth.requestedAction.propstmt.EditObjectPropertyStatement;
|
||||||
import edu.cornell.mannlib.vitro.webapp.beans.DataPropertyStatement;
|
|
||||||
import edu.cornell.mannlib.vitro.webapp.beans.DataPropertyStatementImpl;
|
|
||||||
import edu.cornell.mannlib.vitro.webapp.beans.IndividualImpl;
|
import edu.cornell.mannlib.vitro.webapp.beans.IndividualImpl;
|
||||||
import edu.cornell.mannlib.vitro.webapp.dao.VitroVocabulary;
|
import edu.cornell.mannlib.vitro.webapp.dao.VitroVocabulary;
|
||||||
|
|
||||||
|
@ -212,31 +210,16 @@ public class SelfEditingPolicyTest extends AbstractTestClass {
|
||||||
//
|
//
|
||||||
@Test
|
@Test
|
||||||
public void testVisitIdentifierBundleEditDataPropStmt() {
|
public void testVisitIdentifierBundleEditDataPropStmt() {
|
||||||
|
whatToAuth = new EditDataPropertyStatement(SELFEDITOR_URI,SAFE_PREDICATE);
|
||||||
DataPropertyStatement dps = new DataPropertyStatementImpl();
|
|
||||||
dps.setIndividualURI(SELFEDITOR_URI);
|
|
||||||
dps.setDatapropURI(SAFE_PREDICATE);
|
|
||||||
dps.setData("junk");
|
|
||||||
whatToAuth = new EditDataPropertyStatement(dps);
|
|
||||||
assertDecision(AUTHORIZED, policy.isAuthorized(ids, whatToAuth));
|
assertDecision(AUTHORIZED, policy.isAuthorized(ids, whatToAuth));
|
||||||
|
|
||||||
dps = new DataPropertyStatementImpl();
|
whatToAuth = new EditDataPropertyStatement(SELFEDITOR_URI, UNSAFE_PREDICATE);
|
||||||
dps.setIndividualURI(SELFEDITOR_URI);
|
|
||||||
dps.setDatapropURI(UNSAFE_PREDICATE);
|
|
||||||
dps.setData("junk");
|
|
||||||
whatToAuth = new EditDataPropertyStatement(dps);
|
|
||||||
assertDecision(INCONCLUSIVE, policy.isAuthorized(ids, whatToAuth));
|
assertDecision(INCONCLUSIVE, policy.isAuthorized(ids, whatToAuth));
|
||||||
|
|
||||||
dps = new DataPropertyStatementImpl();
|
whatToAuth = new EditDataPropertyStatement(UNSAFE_RESOURCE, SAFE_PREDICATE);
|
||||||
dps.setIndividualURI(UNSAFE_RESOURCE);
|
|
||||||
dps.setDatapropURI(SAFE_PREDICATE);
|
|
||||||
whatToAuth = new EditDataPropertyStatement(dps);
|
|
||||||
assertDecision(INCONCLUSIVE, policy.isAuthorized(ids, whatToAuth));
|
assertDecision(INCONCLUSIVE, policy.isAuthorized(ids, whatToAuth));
|
||||||
|
|
||||||
dps = new DataPropertyStatementImpl();
|
whatToAuth = new EditDataPropertyStatement(SAFE_RESOURCE, SAFE_PREDICATE);
|
||||||
dps.setIndividualURI(SAFE_RESOURCE);
|
|
||||||
dps.setDatapropURI(SAFE_PREDICATE);
|
|
||||||
whatToAuth = new EditDataPropertyStatement(dps);
|
|
||||||
assertDecision(INCONCLUSIVE, policy.isAuthorized(ids, whatToAuth));
|
assertDecision(INCONCLUSIVE, policy.isAuthorized(ids, whatToAuth));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -296,11 +279,7 @@ public class SelfEditingPolicyTest extends AbstractTestClass {
|
||||||
public void twoSEIsFindDataPropertySubject() {
|
public void twoSEIsFindDataPropertySubject() {
|
||||||
setUpTwoSEIs();
|
setUpTwoSEIs();
|
||||||
|
|
||||||
DataPropertyStatement dps = new DataPropertyStatementImpl();
|
whatToAuth = new EditDataPropertyStatement(SELFEDITOR_URI, SAFE_PREDICATE);
|
||||||
dps.setIndividualURI(SELFEDITOR_URI);
|
|
||||||
dps.setDatapropURI(SAFE_PREDICATE);
|
|
||||||
dps.setData("junk");
|
|
||||||
whatToAuth = new EditDataPropertyStatement(dps);
|
|
||||||
assertDecision(AUTHORIZED, policy.isAuthorized(ids, whatToAuth));
|
assertDecision(AUTHORIZED, policy.isAuthorized(ids, whatToAuth));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -308,11 +287,7 @@ public class SelfEditingPolicyTest extends AbstractTestClass {
|
||||||
public void twoSEIsDontFindInDataProperty() {
|
public void twoSEIsDontFindInDataProperty() {
|
||||||
setUpTwoSEIs();
|
setUpTwoSEIs();
|
||||||
|
|
||||||
DataPropertyStatement dps = new DataPropertyStatementImpl();
|
whatToAuth = new EditDataPropertyStatement(SAFE_RESOURCE, SAFE_PREDICATE);
|
||||||
dps.setIndividualURI(SAFE_RESOURCE);
|
|
||||||
dps.setDatapropURI(SAFE_PREDICATE);
|
|
||||||
dps.setData("junk");
|
|
||||||
whatToAuth = new EditDataPropertyStatement(dps);
|
|
||||||
assertDecision(INCONCLUSIVE, policy.isAuthorized(ids, whatToAuth));
|
assertDecision(INCONCLUSIVE, policy.isAuthorized(ids, whatToAuth));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -294,13 +294,8 @@ public class SelfEditingPolicy_2_Test extends AbstractTestClass {
|
||||||
*/
|
*/
|
||||||
private void assertEditDataPropStmt(String individualURI,
|
private void assertEditDataPropStmt(String individualURI,
|
||||||
String datapropURI, String data, Authorization expectedAuthorization) {
|
String datapropURI, String data, Authorization expectedAuthorization) {
|
||||||
DataPropertyStatement dps = new DataPropertyStatementImpl();
|
|
||||||
dps.setIndividualURI(individualURI);
|
|
||||||
dps.setDatapropURI(datapropURI);
|
|
||||||
dps.setData(data);
|
|
||||||
|
|
||||||
EditDataPropertyStatement whatToAuth = new EditDataPropertyStatement(
|
EditDataPropertyStatement whatToAuth = new EditDataPropertyStatement(
|
||||||
dps);
|
individualURI, datapropURI);
|
||||||
PolicyDecision dec = policy.isAuthorized(ids, whatToAuth);
|
PolicyDecision dec = policy.isAuthorized(ids, whatToAuth);
|
||||||
log.debug(dec);
|
log.debug(dec);
|
||||||
Assert.assertNotNull(dec);
|
Assert.assertNotNull(dec);
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue