Skip to content

Commit

Permalink
Merge pull request DSpace#9479 from 4Science/coar-notify-NotifyServic…
Browse files Browse the repository at this point in the history
…eRestRepositoryIT-fix

COAR NotifyService IT failures
  • Loading branch information
tdonohue authored Apr 22, 2024
2 parents b6c5fbb + 2a46888 commit 3ea0022
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,6 @@ public boolean supports(Object objectToMatch, Operation operation) {
String path = operation.getPath().trim();
return (objectToMatch instanceof NotifyServiceEntity &&
operation.getOp().trim().equalsIgnoreCase(OPERATION_REMOVE) &&
path.startsWith(OPERATION_PATH));
path.startsWith(OPERATION_PATH) && path.endsWith(OPERATION_PATH));
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -1266,8 +1266,7 @@ public void NotifyServiceInboundPatternConstraintReplaceOperationBadRequestTest(
.andExpect(status().isBadRequest());
}

// Temporarily disable. See: https://github.com/DSpace/DSpace/issues/9457
@Ignore

@Test
public void NotifyServiceInboundPatternConstraintRemoveOperationTest() throws Exception {

Expand Down Expand Up @@ -1332,8 +1331,6 @@ public void NotifyServiceInboundPatternConstraintRemoveOperationTest() throws Ex
)));
}

// Temporarily disable. See: https://github.com/DSpace/DSpace/issues/9457
@Ignore
@Test
public void NotifyServiceInboundPatternConstraintRemoveOperationBadRequestTest() throws Exception {

Expand Down

0 comments on commit 3ea0022

Please sign in to comment.