fix: assign self.deviceId correctly

amplitude · Apr 5, 2024 · 4a8978c · 4a8978c
1 parent cdb622c
commit 4a8978c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/Sources/Amplitude/Amplitude.m b/Sources/Amplitude/Amplitude.m
@@ -1686,7 +1686,7 @@ - (void)initializeDeviceInfo {
         if (self.deviceId == nil) {
             self.deviceId = [self.dbHelper getValue:DEVICE_ID];
             if (![self isValidDeviceId:self.deviceId]) {
-                self.deviceId = [self _getDeviceId];
+                self->_deviceId = [self _getDeviceId];
                 [self.dbHelper insertOrReplaceKeyValue:DEVICE_ID value:self.deviceId];
             }
         }

diff --git a/Tests/AmplitudeTests.m b/Tests/AmplitudeTests.m
@@ -1708,9 +1708,9 @@ - (void)testOpenURLFiresDeepLinkEvent {
 - (void)testGetDeviceIdBeforeInit {
     NSString *instanceName = @"testGetDeviceIdBeforeAndAfterInit";
 
-    // Clear device ID from db
+    // Delete DB before running the test to start fresh
     AMPDatabaseHelper *dbHelper = [AMPDatabaseHelper getDatabaseHelper:instanceName];
-    [dbHelper insertOrReplaceKeyValue:@"device_id" value:nil];
+    [dbHelper deleteDB];
 
     // Device ID should get generated before api key init
     Amplitude *client = [Amplitude instanceWithName:instanceName];