Closed Bug 1264381 Opened 4 years ago Closed 4 years ago
Document lack of forward compatibility in test
Browser Database Helper Upgrades
MozReview Request: Bug 1264381 - Add comment explaining lack of SQLite forward-compatibility r?mcomella
58 bytes, text/x-review-board-request
See Bug 1253340, we need to use older devices/emulators to create our test DBs. We should document this in the test.
Review commit: https://reviewboard.mozilla.org/r/46161/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/46161/
Attachment #8741064 - Flags: review?(michael.l.comella)
Comment on attachment 8741064 [details] MozReview Request: Bug 1264381 - Add comment explaining lack of SQLite forward-compatibility r?mcomella https://reviewboard.mozilla.org/r/46161/#review44335 I'm surprised we didn't add this sooner! ::: mobile/android/tests/browser/robocop/src/org/mozilla/gecko/tests/testBrowserDatabaseHelperUpgrades.java:52 (Diff revision 1) > * * 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 nit: "e.g. Uploading a..."
Attachment #8741064 - Flags: review?(michael.l.comella) → review+
https://hg.mozilla.org/integration/fx-team/rev/19954943888496006f660e90b3eee0c2482f5fca Bug 1264381 - Add comment explaining lack of SQLite forward-compatibility r=mcomella
You need to log in before you can comment on or make changes to this bug.