Skip to content

Commit ce9d305

Browse files
committed
Finalized the caching tests
1 parent 0e0fe4f commit ce9d305

File tree

3 files changed

+13
-379
lines changed

3 files changed

+13
-379
lines changed

core-api/src/main/java/com/optimizely/ab/Optimizely.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ public class Optimizely implements AutoCloseable {
8888

8989
private final ProjectConfigManager projectConfigManager;
9090

91-
private OptimizelyConfigManager optimizelyConfigManager;
91+
private final OptimizelyConfigManager optimizelyConfigManager;
9292

9393
// TODO should be private
9494
public final NotificationCenter notificationCenter;
@@ -110,9 +110,7 @@ private Optimizely(@Nonnull EventHandler eventHandler,
110110
this.decisionService = decisionService;
111111
this.userProfileService = userProfileService;
112112
this.projectConfigManager = projectConfigManager;
113-
if (projectConfigManager instanceof OptimizelyConfigManager) {
114-
this.optimizelyConfigManager = (OptimizelyConfigManager)projectConfigManager;
115-
}
113+
this.optimizelyConfigManager = projectConfigManager instanceof OptimizelyConfigManager ? (OptimizelyConfigManager)projectConfigManager : null;
116114
this.notificationCenter = notificationCenter;
117115
}
118116

core-api/src/main/java/com/optimizely/ab/config/PollingProjectConfigManager.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ void setConfig(ProjectConfig projectConfig) {
103103
logger.info("New datafile set with revision: {}. Old revision: {}", projectConfig.getRevision(), previousRevision);
104104

105105
currentProjectConfig.set(projectConfig);
106-
currentOptimizelyConfig.set(new OptimizelyConfigService(currentProjectConfig.get()).getConfig());
106+
currentOptimizelyConfig.set(new OptimizelyConfigService(projectConfig).getConfig());
107107
countDownLatch.countDown();
108108
notificationCenter.send(SIGNAL);
109109
}

0 commit comments

Comments
 (0)