776724 - test_voicemail_statuschanged.py loads ril_consts.js from arbitrary in-tree location

Status: RESOLVED FIXED

(Core :: DOM: Device Interfaces, defect)

Description

[Jonathan Griffin (:jgriffin)]

The Marionette test test_voicemail_statuschanged.py loads the file 'ril_consts.js' from the relative path '../../../system/gonk'.

This works fine when running the test in-tree, but in continuous integration, we don't run tests that way...we will run them via packaged tests, and that file will not be available.

We should update this test to include the needed constants within the test's .js file.

Comment 1

[Marshall Culpepper [:marshall_law]]

There is a large chunk of constants needed by this test (specifically the language tables for translating the PDUs). Maybe the test could import the ril_consts module from Components.utils instead..

Comment 2

[Philipp von Weitershausen [:philikon]]

There should be no need for test_voicemail_statuschanged.py. If the test needs to load a file that ships with Gecko, it can do something like:

  var Cu = SpecialPowers.unwrap(Components).utils;
  var RIL = {};
  Cu.import("resource://gre/modules/ril_consts.js, RIL);

I'm not 100% sure what we can for loading pdu_builder.js. We could have Marionette map the current test's location to a resource alias, or we could provide a file:/// URI as a global variable in the test scope. jgriffin?

Comment 3

[Jonathan Griffin (:jgriffin)]

(In reply to Philipp von Weitershausen [:philikon] from comment #2)
> There should be no need for test_voicemail_statuschanged.py. If the test
> needs to load a file that ships with Gecko, it can do something like:
> 
>   var Cu = SpecialPowers.unwrap(Components).utils;
>   var RIL = {};
>   Cu.import("resource://gre/modules/ril_consts.js, RIL);
> 
> I'm not 100% sure what we can for loading pdu_builder.js. We could have
> Marionette map the current test's location to a resource alias, or we could
> provide a file:/// URI as a global variable in the test scope. jgriffin?

It's probably a good idea to add dyanmic resource url mappings for cases like this.  I'll file another bug for that.

Comment 4

[Marshall Culpepper [:marshall_law]]

Attachment: vm test pathing fix - v1

Comment 5

[Marshall Culpepper [:marshall_law]]

http://hg.mozilla.org/integration/mozilla-inbound/rev/bc9be59cca93

Comment 6

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/bc9be59cca93