Closed Bug 900273 Opened 7 years ago Closed 7 years ago

mozharness should pass test-plugin-path to runxpcshelltests.py

Categories

(Testing :: XPCShell Harness, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mihneadb, Assigned: mihneadb)

References

Details

Attachments

(1 file, 1 obsolete file)

This is necessary so we can get rid of the hardcoded defaulting that xpcshell.cpp does.
Blocks: 899857
Assignee: nobody → mihneadb
Ash run with the changes in this bug and in bug 899857 applied:
https://tbpl.mozilla.org/?tree=Ash&rev=46ca10eb1fb3
mozharness-ash seems to be busted (cannot install mozrunner)
Attachment #784096 - Flags: review?(rail)
Attachment #784096 - Flags: review?(rail) → review+
Rail, do I need to do something else to get this landed/deployed?

Thanks
Flags: needinfo?(rail)
(In reply to Mihnea Dobrescu-Balaur (:mihneadb) from comment #4)
> Rail, do I need to do something else to get this landed/deployed?
> 
> Thanks

Once buildduty merges default to production (today, afaik) the patch will be live.
Flags: needinfo?(rail)
In other words, no further action required here.
(In reply to Rail Aliiev [:rail] from comment #6)
> In other words, no further action required here.

Great, thanks, I thought I needed to add a keyword or something.
Comment on attachment 784096 [details] [diff] [review]
mozharness should pass test-plugin-path to runxpcshelltests.py

pushed to https://hg.mozilla.org/build/mozharness/rev/dab6d1772285
and merged to production https://hg.mozilla.org/build/mozharness/rev/8c511238602b
Backed out

https://hg.mozilla.org/build/mozharness/rev/1ce7afcbbf12
https://hg.mozilla.org/build/mozharness/rev/c57bdead999d

b2g_emulator_vm mozilla-inbound opt test xpcshell on 2013-08-02 11:13:08 PDT for push 834374d5596f

11:14:22    FATAL - Uncaught exception: Traceback (most recent call last):
11:14:22    FATAL -   File "/builds/slave/test/scripts/mozharness/base/script.py", line 1048, in run
11:14:22    FATAL -     self.run_action(action)
11:14:22    FATAL -   File "/builds/slave/test/scripts/mozharness/base/script.py", line 990, in run_action
11:14:22    FATAL -     self._possibly_run_method(method_name, error_if_missing=True)
11:14:22    FATAL -   File "/builds/slave/test/scripts/mozharness/base/script.py", line 931, in _possibly_run_method
11:14:22    FATAL -     return getattr(self, method_name)()
11:14:22    FATAL -   File "scripts/scripts/b2g_emulator_unittest.py", line 333, in run_tests
11:14:22    FATAL -     cmd = self._query_abs_base_cmd(suite)
11:14:22    FATAL -   File "scripts/scripts/b2g_emulator_unittest.py", line 270, in _query_abs_base_cmd
11:14:22    FATAL -     option = option % str_format_values
11:14:22    FATAL - KeyError: 'test_plugin_path'
11:14:22    FATAL - Exiting -1
Problem was I made the change to the b2g script as well, but it turns out that
one is not set up in the same way.

Please don't land it yet, need to investigate a failure that popped up with this
patch applied.
Attachment #785134 - Flags: review?(rail)
Attachment #784096 - Attachment is obsolete: true
Attachment #785134 - Flags: review?(rail) → review+
No longer blocks: 899857
Depends on: 899857
The change in bug 899857 will allow the transition to passing in --test-plugin-path.
You can retrigger xpcshell tests on cedar to test this change out.
Just did, thanks!
Comment on attachment 785134 [details] [diff] [review]
mozharness should pass test-plugin-path to runxpcshelltests.py

Merged to production.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.