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)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: whimboo, Assigned: whimboo)

References

()

Details

(Whiteboard: [needs-mozmill-1.5.2])

Attachments

(1 file)

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.
Well, bug 564537 should cover the refactoring work now.
Depends on: 564537
No longer depends on: 565733
Component: Mozmill → Mozmill Tests
QA Contact: mozmill → mozmilltests
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
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: hskupin → nobody
Component: Mozmill Tests → Mozmill Automation
QA Contact: mozmill-tests → mozmill-automation
Assignee: nobody → hskupin
Blocks: 626712
Blocks: 568726
Blocks: 637277
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
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)
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 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+
Attachment #519987 - Attachment description: Clean-up wrapper class v1 → Clean-up wrapper class v1 [checked-in]
No longer blocks: 626712
All remaining work now happens on bug 732134 and its dependencies.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: