@awoods @acoburn @peichman-umd
This is the cause of the conflict. I can solve it, but I know this is ugly and would appreciate any suggestions.

@whikloj in this case, you can just roll back the change on this line from #47 so you get:

try (final CloseableHttpResponse response = execute(request)) {
    assertEquals(HttpStatus.SC_OK, getStatus(response));
    // the rest of your code from your PR
}

It's fine to use the try block if you actually need to inspect the response. For merely checking the status code it was unnecessary.

@acoburn but is a for loop the best way to do this type of testing? It seems very basic, but perhaps I am just expecting more.

Oh, that code should definitely be changed:

Optional<String> header = Arrays.asList(response.getHeaders("Link")).stream().map(Header::getValue).filter(aclLink::equals).findFirst();
assertTrue("Missing Link header", header.isPresent());

@acoburn++