1545591 - IndexedDB.jsm has no tests

Status: NEW

(Toolkit :: Async Tooling, task, P3)

Description

[Michael Cooper [:mythmon]]

The Promise-based wrapper of IndexedDB in toolkit/modules/IndexedDB.jsm does not have any associated tests. The initial implementation was landed in bug 1344590, and did not include any tests. It doesn't seem that any have been added in the intervening time either.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

Bugbug thinks this bug is a task, but please change it back in case of error.

Comment 2

[Marco Castelluccio [:marco]]

This is a task (see the bug type definition at https://mozilla.github.io/bug-handling/bug-types). I know the absence of tests could be considered as a defect of the project, but it is not a defect of the product (as it is not user-facing). While writing new tests could uncover defects, they should be filed as separate bugs, and the bug to track writing tests is not a defect by itself.

Comment 3

[Alex Thayer [:alexical] (she/her)]

Hey Andrew, you moved this from WebExtensions :: General to Toolkit :: General, but IndexedDB.jsm is listed as belonging to WebExtensions :: General. Do we need to change that or should this go back there?

Comment 4

[Andrew Swan [:aswan]]

Kris requested Async Tooling. I'll get moz.build updated (bug 1547045)