Document lack of forward compatibility in testBrowserDatabaseHelperUpgrades

RESOLVED FIXED in Firefox 48

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: ahunt, Assigned: ahunt)

Tracking

Trunk
Firefox 48
Points:
---

Firefox Tracking Flags

(firefox48 fixed)

Details

Attachments

(1 attachment)

See Bug 1253340, we need to use older devices/emulators to create our test DBs. We should document this in the test.
Assignee: nobody → ahunt
Status: NEW → ASSIGNED
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
https://hg.mozilla.org/mozilla-central/rev/199549438884
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
You need to log in before you can comment on or make changes to this bug.