From ab25ddea2b1a153fa96dcf2faf5765310398db69 Mon Sep 17 00:00:00 2001 From: Marek Gradzki Date: Wed, 23 Aug 2017 14:10:46 +0200 Subject: DumpCacheManager: make dumpParams optional Change-Id: I6ee37f9e39842468f746656dff53b98c2b3796ba Signed-off-by: Marek Gradzki --- .../translate/util/read/cache/DumpCacheManagerTest.java | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) (limited to 'infra/translate-utils/src/test/java/io/fd/honeycomb/translate/util/read/cache/DumpCacheManagerTest.java') diff --git a/infra/translate-utils/src/test/java/io/fd/honeycomb/translate/util/read/cache/DumpCacheManagerTest.java b/infra/translate-utils/src/test/java/io/fd/honeycomb/translate/util/read/cache/DumpCacheManagerTest.java index dfdcd089b..5f9362458 100644 --- a/infra/translate-utils/src/test/java/io/fd/honeycomb/translate/util/read/cache/DumpCacheManagerTest.java +++ b/infra/translate-utils/src/test/java/io/fd/honeycomb/translate/util/read/cache/DumpCacheManagerTest.java @@ -86,7 +86,7 @@ public class DumpCacheManagerTest { // executor cant return null data when(executor.executeDump(identifier, NO_PARAMS)).thenReturn(new IpDetailsReplyDump()); - final Optional stage1Optional = managerNegative.getDump(identifier, cache, NO_PARAMS); + final Optional stage1Optional = managerNegative.getDump(identifier, cache); // this is first call so instance should be from executor // and it should be cached after calling executor @@ -99,7 +99,7 @@ public class DumpCacheManagerTest { IpDetailsReplyDump stage2LoadedDump = new IpDetailsReplyDump(); when(executor.executeDump(identifier, NO_PARAMS)).thenReturn(stage2LoadedDump); - final Optional stage2Optional = managerPositive.getDump(identifier, cache, NO_PARAMS); + final Optional stage2Optional = managerPositive.getDump(identifier, cache); assertEquals(true, stage2Optional.isPresent()); assertEquals(stage2LoadedDump, stage2Optional.get()); @@ -108,7 +108,7 @@ public class DumpCacheManagerTest { IpDetailsReplyDump stage3LoadedDump = new IpDetailsReplyDump(); when(executor.executeDump(identifier, NO_PARAMS)).thenReturn(stage3LoadedDump); - final Optional stage3Optional = managerPositive.getDump(identifier, cache, NO_PARAMS); + final Optional stage3Optional = managerPositive.getDump(identifier, cache); assertEquals(true, stage3Optional.isPresent()); //check if it returns instance cached from previous stage assertEquals(stage2LoadedDump, stage3Optional.get()); @@ -124,7 +124,7 @@ public class DumpCacheManagerTest { when(executor.executeDump(identifier, null)).thenReturn(dump); Optional optionalDump = - managerPositiveWithPostProcessing.getDump(identifier, cache, NO_PARAMS); + managerPositiveWithPostProcessing.getDump(identifier, cache); assertEquals(true, optionalDump.isPresent()); assertEquals(1, optionalDump.get().ipDetails.size()); @@ -143,8 +143,8 @@ public class DumpCacheManagerTest { .acceptOnly(Integer.class) .withExecutor((InstanceIdentifier, Void) -> 3).build(); - final Optional stringDump = stringManager.getDump(identifier, cache, NO_PARAMS); - final Optional integerDump = intManager.getDump(identifier, cache, NO_PARAMS); + final Optional stringDump = stringManager.getDump(identifier, cache); + final Optional integerDump = intManager.getDump(identifier, cache); assertTrue(stringDump.isPresent()); assertTrue(integerDump.isPresent()); -- cgit 1.2.3-korg