Closed Bug 627108 Opened 14 years ago Closed 14 years ago

Release improved Mozmill Crowd v0.1.1 for localizers

Categories

(Mozilla QA Graveyard :: Mozmill Crowd Extension, defect)

Product:
Mozilla QA Graveyard
Component:
Mozmill Crowd Extension
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: whimboo, Assigned: whimboo)

References

Details

(Whiteboard: [0.1.1])

Attachments

(6 files)

Store screenshots in storage folder [checked-in]
1.17 KB, patch
aaronmt
: review+
Details | Diff | Splinter Review
Patch v1 - ('Path not found' string change) [checked-in]
849 bytes, patch
whimboo
: review+
Details | Diff | Splinter Review
Show log information after the test-run [checked-in]
2.90 KB, patch
aaronmt
: review+
Details | Diff | Splinter Review
Reset local mozmill-automation to get latest updates [checked-in]
976 bytes, patch
aaronmt
: review+
Details | Diff | Splinter Review
Version bump [checked-in]
848 bytes, patch
aaronmt
: review+
Details | Diff | Splinter Review
Disable slow script warning during runtime [checked-in]
1.02 KB, patch
aaronmt
: review+
Details | Diff | Splinter Review
As given by the feedback from Axel, it would help a lot to fix the following items before we want to ask our localizers to run the tests: * Show a better message why the extension folder is needed * Save screenshots in the extension folder, best under 'screenshots' * Show the log in the textarea after the testrun has been completed All that shouldn't be that hard to implement and we can cover that in this bug.
Depends on: 627205
Simple patch to store the screenshots in the storage folder. This patch depends on the work on bug 627205.
Attachment #505213 - Flags: review?(aaron.train)
Attachment #505213 - Flags: review?(aaron.train) → review+
String change on the path not found property
Attachment #505265 - Flags: review?(hskupin)
Attachment #505265 - Flags: review?(hskupin) → review+
Attachment #505265 - Attachment description: Patch v1 - ('Path not found' string change) → Patch v1 - ('Path not found' string change) [checked-in]
Comment on attachment 505213 [details] [diff] [review] Store screenshots in storage folder [checked-in] Landed as: https://github.com/whimboo/mozmill-crowd/commit/328f963a7bf1eaa0c53235d0cb218cd6bbb0d712 The last patch landed as: https://github.com/whimboo/mozmill-crowd/commit/6a1c594fc250d2e810c9f5dcaef037709633b957
Attachment #505213 - Attachment description: Store screenshots in storage folder → Store screenshots in storage folder [checked-in]
We don't have a way to call environment.run with the path of the automation repository set as current working directory. For now lets re-clone the repository.
Attachment #505287 - Flags: review?(aaron.train)
Final patch to bump the version to 0.1.1 and make it compatible with all 4.0.* releases of Firefox.
Attachment #505289 - Flags: review?(aaron.train)
Attachment #505287 - Flags: review?(aaron.train) → review+
Attachment #505289 - Flags: review?(aaron.train) → review+
Comment on attachment 505282 [details] [diff] [review] Show log information after the test-run [checked-in] Looks fine, r+. Any cleanup in the output I suppose will be for a later time.
Attachment #505282 - Flags: review?(aaron.train) → review+
Comment on attachment 505287 [details] [diff] [review] Reset local mozmill-automation to get latest updates [checked-in] Landed as: https://github.com/whimboo/mozmill-crowd/commit/785ac20569ffeb8a0a94a3757e1495798b7b8e13
Attachment #505287 - Attachment description: Reset local mozmill-automation to get latest updates → Reset local mozmill-automation to get latest updates [checked-in]
Comment on attachment 505282 [details] [diff] [review] Show log information after the test-run [checked-in] (In reply to comment #7) > Any cleanup in the output I suppose will be for a later time. Yes, we will parse the output later and display it in a more friendly way.
Attachment #505282 - Attachment description: Show log information after the test-run → Show log information after the test-run [checked-in]
Comment on attachment 505289 [details] [diff] [review] Version bump [checked-in] Landed as: https://github.com/whimboo/mozmill-crowd/commit/bdb74e0925d79f363bc376d2b89478feda301001
Attachment #505289 - Attachment description: Version bump → Version bump [checked-in]
Axel, I have uploaded the new version to my people account: http://people.mozilla.com/~hskupin/mozmill-crowd/mozmill-crowd.xpi Would be great if you could have a quick check, and tell me if those changes fit your expectations before we can announce it to localizers. Thanks.
This patch simply disable the slow script warning for chrome scripts while the external process is executed. It gives a much better user experience so we should include it in 0.1.1.
Attachment #505471 - Flags: review?(aaron.train)
Comment on attachment 505471 [details] [diff] [review] Disable slow script warning during runtime [checked-in] Looks fine, r+. Let's get this in.
Attachment #505471 - Flags: review?(aaron.train) → review+
Comment on attachment 505471 [details] [diff] [review] Disable slow script warning during runtime [checked-in] Landed as: https://github.com/whimboo/mozmill-crowd/commit/6152ff59c159148ff98d77250e7acf59e5e4218d
Attachment #505471 - Attachment description: Disable slow script warning during runtime → Disable slow script warning during runtime [checked-in]
New version is up on AMO: https://addons.mozilla.org/de/firefox/addon/mozmill-crowd/
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: [0.1.1]
Target Milestone: --- → 0.1
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: