Skip to content

Commit

Permalink
Make CacheResourcesManager Constructor used for test package protected (
Browse files Browse the repository at this point in the history
#1702)

It looks like the CacheResourcesManager(Cache cache) constructor is oly
used for test, so this is made package protected now.

This has the advantage that there is no public reference to google Cache
API making it an internal implementation detail that can easily be
exchanged.

Signed-off-by: Christoph Läubrich <[email protected]>
  • Loading branch information
laeubi authored Nov 10, 2024
1 parent 2482d4a commit 0aadd16
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ public CacheResourcesManager() {
this(CacheBuilder.newBuilder().maximumSize(100).expireAfterWrite(30, TimeUnit.SECONDS).build());
}

public CacheResourcesManager(Cache<String, CacheResourceDownloadedException> cache) {
CacheResourcesManager(Cache<String, CacheResourceDownloadedException> cache) {
resourcesLoading = new HashMap<>();
protocolsForCache = new HashSet<>();
unavailableURICache = cache;
Expand Down

0 comments on commit 0aadd16

Please sign in to comment.