Skip to content

Commit

Permalink
fix quality violations
Browse files Browse the repository at this point in the history
  • Loading branch information
uchitsa committed Nov 28, 2024
1 parent faeb2dd commit e9eb17a
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -201,6 +201,7 @@ void doesNotCacheGetRequestIfTheLastModifiedHeaderIsMissing()
/**
* LastModifiedCachingWire can resist cache eviction in the event of a non
* OK response without a last modified header.
* @throws Exception If fails
* @todo #120:30min Confirm cache clearing behaviour in all non-OK responses
* Non-OK behaviour was not specified in #120, so for example, if the
* response is 404 as below, does it make any sense to keep the item in
Expand All @@ -210,7 +211,6 @@ void doesNotCacheGetRequestIfTheLastModifiedHeaderIsMissing()
* or 410 Gone etc. Or, personally I think all non-OK and OK responses
* should behave the same WRT to clearing the cache as the cache value is
* so unlikely to be returned in future.
* @throws Exception If fails
*/
@Test
void doesNotEvictCacheOnNonOk()
Expand Down

0 comments on commit e9eb17a

Please sign in to comment.