Skip to content

Commit 7fe1571

Browse files
committed
fix
1 parent cfc616d commit 7fe1571

File tree

3 files changed

+9
-1
lines changed

3 files changed

+9
-1
lines changed

android-sdk/src/test/java/com/optimizely/ab/android/sdk/OptimizelyManagerBuilderTest.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -442,6 +442,8 @@ public void testBuildWithVuidDisabled() throws Exception {
442442
verify(mockBuilder).withUserCommonIdentifiers(identifiersCaptor.capture());
443443
Map<String, String> identifiers = identifiersCaptor.getValue();
444444
assertFalse(identifiers.containsKey("vuid"));
445+
446+
when(ODPManager.builder()).thenCallRealMethod();
445447
}
446448

447449
@Test
@@ -464,5 +466,7 @@ public void testBuildWithVuidEnabled() throws Exception {
464466
verify(mockBuilder).withUserCommonIdentifiers(identifiersCaptor.capture());
465467
Map<String, String> identifiers = identifiersCaptor.getValue();
466468
assertEquals(identifiers.get("vuid"), "vuid_test");
469+
470+
when(ODPManager.builder()).thenCallRealMethod();
467471
}
468472
}

odp/src/androidTest/java/com/optimizely/ab/android/odp/VuidManagerTest.kt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ import android.content.Context
1818
import android.content.SharedPreferences
1919
import androidx.test.ext.junit.runners.AndroidJUnit4
2020
import androidx.test.platform.app.InstrumentationRegistry.getInstrumentation
21+
import com.optimizely.ab.android.shared.OptlyStorage
2122
import org.junit.After
2223
import org.junit.Assert.assertEquals
2324
import org.junit.Assert.assertFalse
@@ -138,9 +139,11 @@ class VuidManagerTest {
138139
cleanSharedPrefs()
139140
saveInSharedPrefs("vuid", "vuid_test")
140141
VuidManager.removeSharedForTesting()
142+
141143
vuidManager = VuidManager.getInstance()
142144
vuidManager.configure(false, context)
143145
assertNull(getFromSharedPrefs("vuid"))
146+
144147
assertEquals(vuidManager.vuid, "")
145148
}
146149

@@ -158,6 +161,7 @@ class VuidManagerTest {
158161
fun configureWithVuidEnabledWhenVuidDoesNotExist() {
159162
cleanSharedPrefs()
160163
VuidManager.removeSharedForTesting()
164+
assertNull(getFromSharedPrefs("vuid"))
161165
vuidManager = VuidManager.getInstance()
162166
vuidManager.configure(true, context)
163167
assertTrue(vuidManager.vuid.startsWith("vuid_"))

shared/src/main/java/com/optimizely/ab/android/shared/OptlyStorage.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,6 +93,6 @@ private SharedPreferences getReadablePrefs() {
9393
}
9494

9595
public void remove(String key) {
96-
getWritablePrefs().remove(key);
96+
getWritablePrefs().remove(key).apply();
9797
}
9898
}

0 commit comments

Comments
 (0)