Skip to content

Commit 6d95172

Browse files
fix: fix typo in log message
- Update log message to remove unnecessary punctuation - Correct typo in log message for failed CMAB data fetch in experiment
1 parent d772ee2 commit 6d95172

File tree

4 files changed

+16
-2
lines changed

4 files changed

+16
-2
lines changed

Sources/Implementation/DefaultDecisionService.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ class DefaultDecisionService: OPTDecisionService {
125125
case .failure:
126126
let info = LogMessage.cmabFetchFailed(experiment.key)
127127
self.logger.e(info)
128-
reasons.addInfo(info)
128+
reasons.addError(info)
129129
let nilVariation = VariationDecision(variation: nil, cmabError: true, cmabUUID: nil)
130130
return DecisionResponse(result: nilVariation, reasons: reasons)
131131
}

Sources/Optimizely+Decide/OptimizelyClient+Decide.swift

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -297,6 +297,10 @@ extension OptimizelyClient {
297297
return OptimizelyDecision.errorDecision(key: flagKey, user: user, error: .featureKeyInvalid(flagKey))
298298
}
299299

300+
guard flagDecision?.variation != nil else {
301+
return OptimizelyDecision.errorDecision(key: flagKey, user: user, reasons: decisionReasons.toReport())
302+
}
303+
300304
let userId = user.userId
301305
let attributes = user.attributes
302306
let flagEnabled = flagDecision?.variation?.featureEnabled ?? false

Sources/Optimizely+Decide/OptimizelyDecision.swift

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,16 @@ extension OptimizelyDecision {
5252
reasons: [error.reason])
5353
}
5454

55+
static func errorDecision(key: String, user: OptimizelyUserContext, reasons: [String]) -> OptimizelyDecision {
56+
return OptimizelyDecision(variationKey: nil,
57+
enabled: false,
58+
variables: OptimizelyJSON.createEmpty(),
59+
ruleKey: nil,
60+
flagKey: key,
61+
userContext: user,
62+
reasons: reasons)
63+
}
64+
5565
var hasFailed: Bool {
5666
return variationKey == nil
5767
}

Sources/Utils/LogMessage.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ extension LogMessage: CustomStringConvertible {
148148
case .failedToAssignValue: message = "Value for path could not be assigned to provided type."
149149
case .valueForKeyNotFound(let key): message = "Value for JSON key (\(key)) not found."
150150
case .lowPeriodicDownloadInterval: message = "Polling intervals below 30 seconds are not recommended."
151-
case .cmabFetchFailed(let key): message = "Failed to fetch CMAB data for experiment: \(key)."
151+
case .cmabFetchFailed(let key): message = "Failed to fetch CMAB data for experiment \(key)."
152152
case .cmabNotSupportedInSyncMode: message = "CMAB is not supported in sync mode."
153153
}
154154

0 commit comments

Comments
 (0)