Fix lint error

Author: mattxw

SplunkRumWorkspace/SmokeTest/SmokeTestUITests/SmokeTestUITests.swift

@@ -35,6 +35,8 @@ var lastTimestamp: CFTimeInterval = CACurrentMediaTime()
 class SmokeTestUITests: XCTestCase {
 
     override func setUpWithError() throws {
+        try super.setUpWithError()
+
         // Put setup code here. This method is called before the invocation of each test method in the class.
 
         // In UI tests it is usually best to stop immediately when a failure occurs.

SplunkRumWorkspace/SplunkRum/SplunkRum/SessionBasedSampler.swift

@@ -18,55 +18,57 @@ import Foundation
 
 struct BoolDecision: Decision {
     var isSampled: Bool
-    var attributes: [String : AttributeValue] = [:]
+    var attributes: [String: AttributeValue] = [:]
 }
 
 class SessionBasedSampler: Sampler {
-    
+
     var probability: Double = 1.0
     var currentlySampled: Bool?
     var lock: Lock = Lock()
-    
+
     init(ratio: Double) {
         probability = ratio
         observeSessionIdChange()
     }
-    
-    func shouldSample(parentContext: SpanContext?, traceId: TraceId, name: String, kind: SpanKind, attributes: [String : AttributeValue], parentLinks: [SpanData.Link]) -> Decision {
+
+    // swiftlint:disable function_parameter_count
+    func shouldSample(parentContext: SpanContext?, traceId: TraceId, name: String, kind: SpanKind, attributes: [String: AttributeValue], parentLinks: [SpanData.Link]) -> Decision {
         return lock.withLock({
             return self.getDecision()
         })
-        
+
     }
-    
+    // swiftlint:enable function_parameter_count
+
     var description: String {
         return "SessionBasedSampler, Ratio: \(probability)"
     }
-    
+
     private func observeSessionIdChange() {
         addSessionIdCallback { [weak self] in
             self?.lock.withLockVoid {
                 self?.currentlySampled = self?.shouldSampleNewSession()
             }
         }
     }
-    
+
     private func getDecision() -> Decision {
-        
+
         if let currentlySampled = self.currentlySampled {
             return BoolDecision(isSampled: currentlySampled)
         }
-        
+
         let isSampled = self.shouldSampleNewSession()
         self.currentlySampled = isSampled
         return BoolDecision(isSampled: isSampled)
     }
-    
+
     /**Check if session will be sampled or not.**/
     private func shouldSampleNewSession() -> Bool {
-        
+
         var result = false
-        
+
         switch probability {
         case 0.0:
             result = false
@@ -75,8 +77,8 @@ class SessionBasedSampler: Sampler {
         default:
             result = Double.random(in: 0.0...1.0) <= probability
         }
-        
+
         return result
     }
-    
+
 }

SplunkRumWorkspace/SplunkRum/SplunkRum/SplunkRum.swift

@@ -243,7 +243,7 @@ var splunkRumInitializeCalledTime = Date()
             .add(spanProcessor: GlobalAttributesProcessor())
             .with(sampler: SessionBasedSampler(ratio: options?.sessionSamplingRatio ?? 1.0))
             .build()
-        
+
         OpenTelemetry.registerTracerProvider(tracerProvider: tracerProvider)
 
         if options != nil {
@@ -341,7 +341,7 @@ var splunkRumInitializeCalledTime = Date()
             .add(spanProcessor: GlobalAttributesProcessor(appName: appName))
             .with(sampler: SessionBasedSampler(ratio: options.sessionSamplingRatio))
             .build()
-        
+
         OpenTelemetry.registerTracerProvider(tracerProvider: tracerProvider)
 
         configuredOptions = SplunkRumOptions(opts: options)

SplunkRumWorkspace/SplunkRum/SplunkRumTests/SessionSamplingTests.swift

@@ -29,10 +29,10 @@ class SessionSamplingTests: XCTestCase {
             .globalAttributes(globalAttributes: [:])
             .sessionSamplingRatio(samplingRatio: 0.2)
             .build()
-        
+
         let provider = OpenTelemetry.instance.tracerProvider as! TracerProviderSdk
         let sampler = provider.getActiveSampler() as! SessionBasedSampler
-        
+
         XCTAssertTrue(sampler.probability >= 0.0 && sampler.probability <= 1.0)
         XCTAssertEqual(sampler.probability, 0.2)
         resetRUM()
@@ -48,11 +48,11 @@ class SessionSamplingTests: XCTestCase {
             .globalAttributes(globalAttributes: [:])
             .sessionSamplingRatio(samplingRatio: 1.0)
             .build()
-        
+
         let provider = OpenTelemetry.instance.tracerProvider as! TracerProviderSdk
         let sampler = provider.getActiveSampler() as! SessionBasedSampler
         let shouldSample = sampler.shouldSample(parentContext: nil, traceId: .random(), name: "Span Example", kind: .client, attributes: [:], parentLinks: [])
-        
+
         XCTAssertTrue(shouldSample.isSampled)
         resetRUM()
     }
@@ -67,11 +67,11 @@ class SessionSamplingTests: XCTestCase {
             .globalAttributes(globalAttributes: [:])
             .sessionSamplingRatio(samplingRatio: 0.0)
             .build()
-        
+
         let provider = OpenTelemetry.instance.tracerProvider as! TracerProviderSdk
         let sampler = provider.getActiveSampler() as! SessionBasedSampler
         let shouldSample = sampler.shouldSample(parentContext: nil, traceId: .random(), name: "Span Example", kind: .client, attributes: [:], parentLinks: [])
-        
+
         XCTAssertFalse(shouldSample.isSampled)
         resetRUM()
     }
@@ -86,16 +86,16 @@ class SessionSamplingTests: XCTestCase {
             .globalAttributes(globalAttributes: [:])
             .sessionSamplingRatio(samplingRatio: 0.5)
             .build()
-        
+
         let provider = OpenTelemetry.instance.tracerProvider as! TracerProviderSdk
         let sampler = provider.getActiveSampler() as! SessionBasedSampler
-        
+
         func testSampling() -> Bool {
             _ = getRumSessionId(forceNewSessionId: true)
             let shouldSample = sampler.shouldSample(parentContext: nil, traceId: .random(), name: "Span Example", kind: .client, attributes: [:], parentLinks: [])
             return shouldSample.isSampled
         }
-        
+
         var countSpans = 0
         for _ in 1...100 where testSampling() {
             countSpans += 1