Bug 1264381 - Add comment explaining lack of SQLite forward-compatibility r?mcomella draft
authorAndrzej Hunt <ahunt@mozilla.com>
Wed, 13 Apr 2016 11:16:51 -0700
changeset 350430 8443f4169f75ecb11786055374fbf348a0f4fef4
parent 350429 bb951e0009b3b118441e9a7af9f2965a81e1e2df
child 518335 d77fdd7bd732949f8cbedf8527973f4dde650b3b
push id15342
push userahunt@mozilla.com
push dateWed, 13 Apr 2016 18:27:57 +0000
reviewersmcomella
bugs1264381
milestone48.0a1
Bug 1264381 - Add comment explaining lack of SQLite forward-compatibility r?mcomella MozReview-Commit-ID: J84s62BsoPJ
mobile/android/tests/browser/robocop/src/org/mozilla/gecko/tests/testBrowserDatabaseHelperUpgrades.java
--- a/mobile/android/tests/browser/robocop/src/org/mozilla/gecko/tests/testBrowserDatabaseHelperUpgrades.java
+++ b/mobile/android/tests/browser/robocop/src/org/mozilla/gecko/tests/testBrowserDatabaseHelperUpgrades.java
@@ -42,16 +42,21 @@ import java.util.ArrayList;
  *   * Copy the current version of the database into (robocop-assets)/browser_db_upgrade/v##.db database. You can do
  * this via Margaret's copy profile addon - take browser.db from the profile directory. This db copy should contain a
  * used profile - e.g. history items, bookmarks. A good way to get a used profile is to sign into sync.
  *   * MAKE SURE YOU COPY YOUR DB FIRST. Then make your changes to the DB schema code.
  *   * Test!
  *   * Note: when the application starts for testing, it may need to upgrade the database from your existing version. If
  * this fails, the application will crash and the test may fail to start.
  *
+ * IMPORTANT:
+ * Test DBs must be created on the oldest version of Android that is currently supported. SQLite
+ * is not forwards compatible. Uploading e.g. a DB created on a 6.0 device will cause failures
+ * when robocop tests running on 4.3 are unable to load it.
+ *
  * Implementation inspired by:
  *   http://riggaroo.co.za/automated-testing-sqlite-database-upgrades-android/
  */
 public class testBrowserDatabaseHelperUpgrades extends UITest {
     private static final int TEST_FROM_VERSION = 27; // We only started testing on this version.
 
     private ArrayList<File> temporaryDbFiles;