Skip to content

Commit 44eec3b

Browse files
committed
#11710 add test for without perms
1 parent b63555f commit 44eec3b

2 files changed

Lines changed: 9 additions & 2 deletions

File tree

src/main/java/edu/harvard/iq/dataverse/engine/command/impl/GetLinkingDataverseListCommand.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ public List<Dataverse> execute(CommandContext ctxt) throws CommandException {
7777
} else {
7878
permToCheck = Permission.LinkDataverse;
7979
}
80-
//dependin on the already linked boolean the command will return a list of Dataverses available for linking
80+
//depending on the already linked boolean the command will return a list of Dataverses available for linking
8181
// or a list of dataverses to which the object has already been linked - for the unlink function
8282
if (!alreadyLinked) {
8383
dataversesForLinking = ctxt.permissions().findPermittedCollections(getRequest(), authUser, permToCheck, searchParam);
@@ -98,7 +98,6 @@ public List<Dataverse> execute(CommandContext ctxt) throws CommandException {
9898
}
9999
return dataversesAlreadyLinkedCanUnlink;
100100
}
101-
//this.permissionService.requestOn(req, res).has(Permission.LinkDataverse)
102101
} else {
103102
dataversesAlreadyLinked = ctxt.dvLinking().findLinkingDataverses(dvObject.getId(), searchParam);
104103
for (Dataverse dv : dataversesAlreadyLinked) {

src/test/java/edu/harvard/iq/dataverse/api/DataversesIT.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -844,6 +844,14 @@ public void testGetLinkableDataverses(){
844844
.statusCode(OK.getStatusCode())
845845
.body("data.size()", equalTo(1));
846846

847+
//if you ask for already linked you should get one unless you don't have perms
848+
getLinkableDataverses = UtilIT.getLinkableDataverses("dataset", datasetPersistentId, apiTokenThree, dataverseAliasForLinking, true);
849+
850+
getLinkableDataverses.prettyPrint();
851+
getLinkableDataverses.then().assertThat()
852+
.statusCode(OK.getStatusCode())
853+
.body("data.size()", equalTo(0));
854+
847855
//if you ask for already linked you should get one unless there's a bad search terms
848856
getLinkableDataverses = UtilIT.getLinkableDataverses("dataset", datasetPersistentId, apiToken, "QQQBatmanSymbol", true);
849857

0 commit comments

Comments
 (0)