Skip to content

Commit e2e0deb

Browse files
authored
remove config-ready check from forced-decision apis (#442)
1 parent 25440f1 commit e2e0deb

File tree

2 files changed

+0
-33
lines changed

2 files changed

+0
-33
lines changed

Sources/Optimizely+Decide/OptimizelyUserContext.swift

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -169,11 +169,6 @@ extension OptimizelyUserContext {
169169
/// - decision: A forced decision.
170170
/// - Returns: true if the forced decision has been set successfully.
171171
public func setForcedDecision(context: OptimizelyDecisionContext, decision: OptimizelyForcedDecision) -> Bool {
172-
guard optimizely?.config != nil else {
173-
logger.e(OptimizelyError.sdkNotReady)
174-
return false
175-
}
176-
177172
// create on the first setForcedDecision call
178173

179174
if forcedDecisions == nil {
@@ -189,11 +184,6 @@ extension OptimizelyUserContext {
189184
/// - context: A decision context
190185
/// - Returns: A forced decision or nil if forced decisions are not set for the decision context.
191186
public func getForcedDecision(context: OptimizelyDecisionContext) -> OptimizelyForcedDecision? {
192-
guard optimizely?.config != nil else {
193-
logger.e(OptimizelyError.sdkNotReady)
194-
return nil
195-
}
196-
197187
guard forcedDecisions != nil else { return nil }
198188

199189
return findForcedDecision(context: context)
@@ -204,11 +194,6 @@ extension OptimizelyUserContext {
204194
/// - context: A decision context.
205195
/// - Returns: true if the forced decision has been removed successfully.
206196
public func removeForcedDecision(context: OptimizelyDecisionContext) -> Bool {
207-
guard optimizely?.config != nil else {
208-
logger.e(OptimizelyError.sdkNotReady)
209-
return false
210-
}
211-
212197
guard let fds = forcedDecisions else { return false }
213198

214199
if findForcedDecision(context: context) != nil {
@@ -222,11 +207,6 @@ extension OptimizelyUserContext {
222207
/// Removes all forced decisions bound to this user context.
223208
/// - Returns: true if forced decisions have been removed successfully.
224209
public func removeAllForcedDecisions() -> Bool {
225-
guard optimizely?.config != nil else {
226-
logger.e(OptimizelyError.sdkNotReady)
227-
return false
228-
}
229-
230210
if let fds = forcedDecisions {
231211
fds.removeAll()
232212
}

Tests/OptimizelyTests-Common/OptimizelyUserContextTests_ForcedDecisions.swift

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -46,19 +46,6 @@ class OptimizelyUserContextTests_ForcedDecisions: XCTestCase {
4646
let context = OptimizelyDecisionContext(flagKey: "feature_1")
4747
let d = OptimizelyForcedDecision(variationKey: "3324490562")
4848

49-
try? optimizely.start(datafile: "invalid datafile contents")
50-
51-
status = user.setForcedDecision(context: context, decision: d)
52-
XCTAssertFalse(status)
53-
result = user.getForcedDecision(context: context)
54-
XCTAssertNil(result)
55-
status = user.removeForcedDecision(context: context)
56-
XCTAssertFalse(status)
57-
status = user.removeAllForcedDecisions()
58-
XCTAssertFalse(status)
59-
60-
try? optimizely.start(datafile: datafile)
61-
6249
status = user.setForcedDecision(context: context, decision: d)
6350
XCTAssertTrue(status)
6451
result = user.getForcedDecision(context: context)

0 commit comments

Comments
 (0)