Fix SwiftLint 0.47.0 warning

DuckDuckGo/Statistics/ATB/VariantManager.swift

@@ -24,27 +24,27 @@ enum FeatureName: String {
 
     // Used for unit tests
     case dummy
-    
+
 }
 
 struct Variant {
-    
+
     struct When {
         static let always = { return true }
 
         static let inRequiredCountry = { return ["AU", "AT", "DK", "FI", "FR", "DE", "IT", "IE", "NZ", "NO", "ES", "SE", "GB"]
-                .contains(where: { Locale.current.regionCode == $0 }) }
-        
+            .contains(where: { Locale.current.regionCode == $0 }) }
+
         static let inEnglish = { return Locale.current.languageCode == "en" }
     }
-    
+
     static let doNotAllocate = 0
-    
+
     // Note: Variants with `doNotAllocate` weight, should always be included so that previous installations are unaffected
     static let defaultVariants: [Variant] = [
 
     ]
-    
+
     let name: String
     let weight: Int
     let isIncluded: () -> Bool
@@ -53,30 +53,30 @@ struct Variant {
 }
 
 protocol VariantRNG {
-    
+
     func nextInt(upperBound: Int) -> Int
-    
+
 }
 
 protocol VariantManager {
-    
+
     var currentVariant: Variant? { get }
     func assignVariantIfNeeded(_ newInstallCompletion: (VariantManager) -> Void)
     func isSupported(feature: FeatureName) -> Bool
-    
+
 }
 
 final class DefaultVariantManager: VariantManager {
-    
+
     var currentVariant: Variant? {
         let variantName = ProcessInfo.processInfo.environment["VARIANT", default: storage.variant ?? "" ]
         return variants.first(where: { $0.name == variantName })
     }
-    
+
     private let variants: [Variant]
     private let storage: StatisticsStore
     private let rng: VariantRNG
-    
+
     init(variants: [Variant] = Variant.defaultVariants,
          storage: StatisticsStore = LocalStatisticsStore(),
          rng: VariantRNG = Arc4RandomUniformVariantRNG()) {
@@ -88,21 +88,21 @@ final class DefaultVariantManager: VariantManager {
     func isSupported(feature: FeatureName) -> Bool {
         return currentVariant?.features.contains(feature) ?? false
     }
-    
+
     func assignVariantIfNeeded(_ newInstallCompletion: (VariantManager) -> Void) {
         guard !storage.hasInstallStatistics else {
             os_log("ATB: No new variant needed for existing user", type: .debug)
             return
         }
-        
+
         if let variant = currentVariant {
             os_log("ATB: Already assigned variant: %s", type: .debug, String(describing: variant))
             return
         }
-        
+
         guard let variant = selectVariant() else {
             os_log("ATB: Failed to assign variant", type: .debug)
-            
+
             // it's possible this failed because there are none to assign, we should still let new install logic execute
             _ = newInstallCompletion(self)
             return
@@ -111,30 +111,30 @@ final class DefaultVariantManager: VariantManager {
         storage.variant = variant.name
         newInstallCompletion(self)
     }
-    
+
     private func selectVariant() -> Variant? {
         let totalWeight = variants.reduce(0, { $0 + $1.weight })
         let randomPercent = rng.nextInt(upperBound: totalWeight)
-        
+
         var runningTotal = 0
         for variant in variants {
             runningTotal += variant.weight
             if randomPercent < runningTotal {
                 return variant.isIncluded() ? variant : nil
             }
         }
-        
+
         return nil
     }
-    
+
 }
 
 final class Arc4RandomUniformVariantRNG: VariantRNG {
-    
+
     init() { }
-    
+
     func nextInt(upperBound: Int) -> Int {
-        return Int(arc4random_uniform(UInt32(upperBound)))
+        return .random(in: 0..<upperBound)
     }
-    
+
 }