clean up

Author: muzahidul-opti

Sources/ODP/OdpManager.swift

@@ -127,7 +127,7 @@ public class OdpManager {
 
         var identifiersUpdated = identifiers
 
-        if identifiers[Constants.ODP.keyForVuid] == nil, let _vuid = vuid {
+        if identifiers[Constants.ODP.keyForVuid] == nil, let _vuid = vuid, VuidManager.isVuid(_vuid) {
             identifiersUpdated[Constants.ODP.keyForVuid] = _vuid
         }
 

Sources/Optimizely/VuidManager.swift

@@ -25,14 +25,12 @@ public class VuidManager {
     public static let shared = VuidManager()
 
     public func configure(enable: Bool) {
-        lock.async {
-            self.enable = enable
-            if enable {
-                self._vuid = self.load()
-            } else {
-                self.remove()
-                self._vuid = ""
-            }
+        self.enable = enable
+        if enable {
+            self._vuid = self.load()
+        } else {
+            self.remove()
+            self._vuid = ""
         }
     }
 
@@ -54,9 +52,12 @@ public class VuidManager {
 // MARK: - VUID Store
 
 extension VuidManager {
-    public var vuid: String {
-        lock.sync {
-            return self._vuid
+    public var vuid: String? {
+        if self.enable {
+            return _vuid
+        } else {
+            logger.w("VUID is not enabled.")
+            return nil
         }
     }
 

Tests/OptimizelyTests-APIs/OptimizelyClientTests_ODP.swift

@@ -190,13 +190,12 @@ class OptimizelyClientTests_ODP: XCTestCase {
     func testVuidEnabled() {
         let settings = OptimizelySdkSettings(enableVuid: true)
         optimizely = OptimizelyClient(sdkKey: OTUtils.randomSdkKey, settings: settings)
-        XCTAssertTrue(optimizely.enableVuid)
+        XCTAssertNotNil(optimizely.vuid)
         XCTAssert(optimizely.vuid!.starts(with: "vuid_"))
     }
 
     func testVuidDiabled() {
         // Default client vuid diabled
-        XCTAssertFalse(optimizely.enableVuid)
         XCTAssertNil(optimizely.vuid)
     }
 

Tests/OptimizelyTests-Common/VuidManagerTests.swift

@@ -52,8 +52,8 @@ class VuidManagerTests: XCTestCase {
         let vuid2 = manager.vuid
 
         XCTAssertTrue(vuid1 == vuid2)
-        XCTAssert(VuidManager.isVuid(vuid1))
-        XCTAssert(VuidManager.isVuid(vuid2))
+        XCTAssert(VuidManager.isVuid(vuid1!))
+        XCTAssert(VuidManager.isVuid(vuid2!))
 
         UserDefaults.standard.removeObject(forKey: "optimizely-vuid")