Closed
Bug 566314
Opened 15 years ago
Closed 13 years ago
Improve automation scripts and rewrite most of the MozmillWrapper class
Categories
(Mozilla QA Graveyard :: Mozmill Automation, defect)
Mozilla QA Graveyard
Mozmill Automation
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: whimboo, Assigned: whimboo)
References
()
Details
(Whiteboard: [needs-mozmill-1.5.2])
Attachments
(1 file)
(deleted),
patch
|
k0scist
:
review+
|
Details | Diff | Splinter Review |
Once all the refactoring work has happened on bug 565733 we will have to remove the MozmillWrapperCLI class which is a temporarily workaround for my Firefox automation scripts.
Assignee | ||
Comment 1•14 years ago
|
||
Well, bug 564537 should cover the refactoring work now.
Assignee | ||
Updated•14 years ago
|
Component: Mozmill → Mozmill Tests
QA Contact: mozmill → mozmilltests
Assignee | ||
Comment 2•14 years ago
|
||
Move of Mozmill Test related project bugs to newly created components. You can
filter out those emails by using "Mozmill-Tests-to-MozillaQA" as criteria.
Product: Testing → Mozilla QA
Version: Trunk → unspecified
Assignee | ||
Comment 3•14 years ago
|
||
With Mozmill 1.5.2 we should be able to remove the wrapper now. In the same step we have to rewrite all the scripts.
Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Depends on: 629107
Summary: Remove MozmillWrapperCLI class from automation scripts → Rewrite automation scripts and remove MozmillWrapperCLI class
Whiteboard: [needs-mozmill-1.5.2]
Assignee | ||
Updated•14 years ago
|
Assignee: hskupin → nobody
Component: Mozmill Tests → Mozmill Automation
QA Contact: mozmill-tests → mozmill-automation
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → hskupin
Assignee | ||
Comment 4•14 years ago
|
||
As it looks like right now we can't get rid of the wrapper class by using Mozmill 1.5.2. So I will refactor a lot of our existing code to make it better usable and to integrate some new features.
Summary: Rewrite automation scripts and remove MozmillWrapperCLI class → Improve automation scripts and rewrite most of the MozmillWrapper class
Assignee | ||
Comment 5•14 years ago
|
||
This patch mostly cleans up the wrapper class to align it better with Mozmill 1.5.2. It removes some unnecessary properties which can be directly set.
I don't want to move large portions of code around right now, but I will check in one of the next patches how to better align the wrapper to the API capabilities Mozmill 1.5.2 offers. Keep in mind the current implementation is from the very beginning.
Attachment #519987 -
Flags: review?(jhammel)
Comment 6•14 years ago
|
||
Note that bug 607111 is the "last" remaining change necessary to make inhereting from the CLI classes unnecessary (and, strictly speaking, nonsensical if you're using the e.g. MozMill class as an API). This will be done for 2.0 and I'll get on it as soon as I clean up a few other bugs.
Comment 7•14 years ago
|
||
Comment on attachment 519987 [details] [diff] [review]
Clean-up wrapper class v1 [checked-in]
This looks fine. Wrapper mozmill will become much different -- and easier -- in 2.0, so probably anything done at the wrapper level shouldn't be a deep refactor for 1.5.2 since that will need to be done anyway for 2.0
Attachment #519987 -
Flags: review?(jhammel) → review+
Assignee | ||
Comment 8•14 years ago
|
||
Comment on attachment 519987 [details] [diff] [review]
Clean-up wrapper class v1 [checked-in]
Landed as:
http://hg.mozilla.org/qa/mozmill-automation/rev/e4f3af55bf9a
Attachment #519987 -
Attachment description: Clean-up wrapper class v1 → Clean-up wrapper class v1 [checked-in]
Assignee | ||
Comment 9•13 years ago
|
||
All remaining work now happens on bug 732134 and its dependencies.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•