Merge branch 'richardgarcar-master'

developernotes · developernotes · commit badfe79ed5e0 · 2023-06-28T08:44:49.000-05:00
diff --git a/sqlcipher/src/androidTest/java/net/zetetic/database/sqlcipher_cts/SupportHelperTest.java b/sqlcipher/src/androidTest/java/net/zetetic/database/sqlcipher_cts/SupportHelperTest.java
@@ -48,7 +48,7 @@ public void shouldCreateDatabaseNormallyWithInitialVersion() {
     }
 
     @Test
-    public void shouldRunUpgradeFromVersion1ToVersion2() {
+    public void shouldRunUpgradeFromVersion1ToVersion2WhenMinSupportedVersionIsProvided() {
         FakeCallback initialCallback = new FakeCallback(1);
 
         SupportHelper initialHelper = new SupportHelper(createConfiguration(initialCallback), null, null, true);
@@ -71,6 +71,29 @@ public void shouldRunUpgradeFromVersion1ToVersion2() {
         assertEquals(1, callbackWrapper.callbackCount[UPGRADE_INDEX]);
     }
 
+    @Test
+    public void shouldRunUpgradeFromVersion1ToVersion2() {
+        FakeCallback initialCallback = new FakeCallback(1);
+
+        SupportHelper initialHelper = new SupportHelper(createConfiguration(initialCallback), null, null, true);
+
+        initialHelper.getWritableDatabase();
+        initialHelper.close();
+
+        assertEquals(1, initialCallback.callbackCount[CREATION_INDEX]);
+        assertEquals(0, initialCallback.callbackCount[UPGRADE_INDEX]);
+
+        FakeCallback callbackWrapper = new FakeCallback(2);
+
+        SupportHelper helper = new SupportHelper(createConfiguration(callbackWrapper), null, null, true);
+
+        helper.getWritableDatabase();
+        helper.close();
+
+        assertEquals(0, callbackWrapper.callbackCount[CREATION_INDEX]);
+        assertEquals(1, callbackWrapper.callbackCount[UPGRADE_INDEX]);
+    }
+
     private SupportSQLiteOpenHelper.Configuration createConfiguration(SupportSQLiteOpenHelper.Callback callback) {
         Context context = InstrumentationRegistry.getInstrumentation().getTargetContext();
         return SupportSQLiteOpenHelper.Configuration.builder(context)
diff --git a/sqlcipher/src/main/java/net/zetetic/database/sqlcipher/SupportHelper.java b/sqlcipher/src/main/java/net/zetetic/database/sqlcipher/SupportHelper.java
@@ -10,7 +10,7 @@ public class SupportHelper implements SupportSQLiteOpenHelper {
 
   public SupportHelper(final Configuration configuration, byte[] password, SQLiteDatabaseHook hook,
                        boolean enableWriteAheadLogging) {
-    this(configuration, password, hook, enableWriteAheadLogging, configuration.callback.version);
+    this(configuration, password, hook, enableWriteAheadLogging, 0);
   }
 
   public SupportHelper(final Configuration configuration, byte[] password, SQLiteDatabaseHook hook,

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ public class SupportHelper implements SupportSQLiteOpenHelper {`
`10`	`10`
`11`	`11`	`public SupportHelper(final Configuration configuration, byte[] password, SQLiteDatabaseHook hook,`
`12`	`12`	`boolean enableWriteAheadLogging) {`
`13`		`- this(configuration, password, hook, enableWriteAheadLogging, configuration.callback.version);`
	`13`	`+ this(configuration, password, hook, enableWriteAheadLogging, 0);`
`14`	`14`	`}`
`15`	`15`
`16`	`16`	`public SupportHelper(final Configuration configuration, byte[] password, SQLiteDatabaseHook hook,`