Closed
Bug 524493
Opened 15 years ago
Closed 15 years ago
Update Mozmill/Mozrunner/Jsbridge to allow software update tests for multiple builds
Categories
(Testing Graveyard :: Mozmill, defect)
Testing Graveyard
Mozmill
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: whimboo, Assigned: whimboo)
References
Details
(Whiteboard: [verified-mozmill-1.2.4][mozmill-doc-needed])
Attachments
(3 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
All three components of Mozmill (Mozmill/Mozrunner/JSBridge) have to be updated to allow multiple runs for different builds. This bug will track this work. I have created branches on Github which will show the latest state:
http://github.com/whimboo/mozmill/tree/parser
http://github.com/whimboo/mozrunner/tree/parser
http://github.com/whimboo/jsbridge/tree/parser
I still have to get the following components working:
* Exchange persisted data between Python and the tests itself
* Reset all the data for the next test run
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•15 years ago
|
||
(In reply to comment #0)
> * Exchange persisted data between Python and the tests itself
I was able to get this part working. The initial implementation for persisted data was broken when you wanted to initialize the object in the Python script itself. Nothing has been send to the test. The trick with persisted.__dict__ is the right way.
> * Reset all the data for the next test run
A big chunk of code in all modules have been refactored to make it usable for multiple test-runs and that consumers can make it more configurable. All those parts are necessary for my software update tests.
Some more comment to the changes:
* the pre-filled lists for global and normal listeners should be part of MozMill itself and should be called directly when the network is created. That way sub-classes don't have to take care about the assignment to the internal back channel. Mikeal, can you point me to a doc or the code how other listeners could be implemented. Also having the possibility to add callback to Python code would be great.
* Probably we should also get rid of the mozmill_class declaration in each sub class. I believe this should be a part of the constructor and would make it more obvious. I haven't changed it yet. Instead I have added the instantiation directly to the constructor so the .mozmill attribute is always available and can be used in any sub class too, e.g. to set values for the persisted attribute. With a local object as it has been used so far there was no way for me to supply user defined data.
* I have also removed the second call to clear_profile which has been called if tests fail during the test run.
Mikeal, it would be great when you could have a look at the current parser branch version.
Assignee | ||
Comment 2•15 years ago
|
||
That's the final patch for Mozmill
Assignee | ||
Comment 3•15 years ago
|
||
That's the final patch for JSBridge
Assignee | ||
Comment 4•15 years ago
|
||
That's the final patch for Mozrunner
Assignee | ||
Updated•15 years ago
|
Attachment #408710 -
Attachment is patch: true
Assignee | ||
Comment 5•15 years ago
|
||
My branch has been merged into Mikeal's master:
http://github.com/mikeal/mozmill/commits/78dac228712a3aa41cb1798d10f37800ca2a2b72
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 6•15 years ago
|
||
This will need updates to http://mikeal.github.com/mozrunner/#mozrunner.CLI.parser_options
Whiteboard: [mozmill-1.2.4][mozmill-doc-needed]
Assignee | ||
Comment 7•15 years ago
|
||
Yesterdays update tests pass without any issues. Marking verified fixed.
Status: RESOLVED → VERIFIED
Whiteboard: [mozmill-1.2.4][mozmill-doc-needed] → [verified-mozmill-1.2.4][mozmill-doc-needed]
Updated•8 years ago
|
Product: Testing → Testing Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•