1733883 - Allow tests to import more modules from ExtensionStorageSyncKinto.jsm

Status: RESOLVED FIXED

(WebExtensions :: General, task, P3)

Description

[Mathew Hodson]

+++ This bug was initially created as a clone of Bug #1733871 +++

In order to stop using ChromeUtils.import with null as the second parameter, ExtensionStorageSyncKinto.jsm needs to be refactored so tests can import more modules from it.

It's used in these tests:

• services/sync/tests/unit/test_extension_storage_engine_kinto.js
• services/sync/tests/unit/test_extension_storage_migration_telem.js
• services/sync/tests/unit/test_extension_storage_tracker_kinto.js
• toolkit/components/extensions/test/xpcshell/test_ExtensionStorageSync_migration_kinto.js
• toolkit/components/extensions/test/xpcshell/test_ext_storage_sync_kinto.js
• toolkit/components/extensions/test/xpcshell/test_ext_storage_sync_kinto_crypto.js

Comment 1

[Mathew Hodson]

Attachment: Bug 1733883 - Allow tests to import more modules from ExtensionStorageSyncKinto.jsm. r?robwu

Depends on D127387

Comment 2

[Luca Greco [:rpl] [:luca] [:lgreco]]

Changing severity to N/A because this is a task.

Comment 3

[Rob Wu [:robwu]]

ExtensionStorageSyncKinto.jsm is deprecated and may be removed eventually. So unlike the other bugs, I'm willing to accept a patch that adds a bunch of extra names to EXPORTED_SYMBOLS here.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1f9e27d95626
Allow tests to import more modules from ExtensionStorageSyncKinto.jsm. r=robwu

Comment 5

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/1f9e27d95626