Skip to content

Commit 2121554

Browse files
Remove redundant abTest name from enum case.
1 parent a90b9e5 commit 2121554

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

Experiments/Experiments/ABTest.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ public enum ABTest: String, CaseIterable {
1919
/// A/B test to measure the sign-in success rate when native Jetpack installation experience is enabled
2020
/// Experiment ref: pbxNRc-29W-p2
2121
///
22-
case abTestNativeJetpackSetupFlow = "woocommerceios_login_jetpack_setup_flow"
22+
case nativeJetpackSetupFlow = "woocommerceios_login_jetpack_setup_flow"
2323

2424
/// A/B test for the Products Onboarding banner on the My Store dashboard.
2525
/// Experiment ref: pbxNRc-26F-p2
@@ -39,7 +39,7 @@ public enum ABTest: String, CaseIterable {
3939
/// When adding a new experiment, add it to the appropriate case depending on its context (logged-in or logged-out experience).
4040
public var context: ExperimentContext {
4141
switch self {
42-
case .productsOnboardingBanner, .productsOnboardingTemplateProducts, .abTestNativeJetpackSetupFlow:
42+
case .productsOnboardingBanner, .productsOnboardingTemplateProducts, .nativeJetpackSetupFlow:
4343
return .loggedIn
4444
case .aaTestLoggedOut, .abTestLoginWithWPComOnly:
4545
return .loggedOut

WooCommerce/Classes/Authentication/AuthenticationManager.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -761,7 +761,7 @@ private extension AuthenticationManager {
761761
}
762762

763763
// Shows the native Jetpack flow during the site discovery flow.
764-
if ABTest.abTestNativeJetpackSetupFlow.variation != .control {
764+
if ABTest.nativeJetpackSetupFlow.variation != .control {
765765
return jetpackSetupUI(for: site.url,
766766
connectionMissingOnly: site.hasJetpack && site.isJetpackActive,
767767
in: navigationController)

0 commit comments

Comments
 (0)