627108 - Release improved Mozmill Crowd v0.1.1 for localizers

Status: RESOLVED FIXED

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

Description

[Henrik Skupin [:whimboo][⌚️UTC+1]]

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.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Attachment: Store screenshots in storage folder [checked-in]

Simple patch to store the screenshots in the storage folder. This patch depends on the work on bug 627205.

Comment 2

[Aaron Train [:aaronmt]]

Attachment: Patch v1 - ('Path not found' string change) [checked-in]

String change on the path not found property

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+1]]

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

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Attachment: Show log information after the test-run [checked-in]

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Attachment: Reset local mozmill-automation to get latest updates [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.

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Attachment: Version bump [checked-in]

Final patch to bump the version to 0.1.1 and make it compatible with all 4.0.* releases of Firefox.

Comment 7

[Aaron Train [:aaronmt]]

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.

Comment 8

[Henrik Skupin [:whimboo][⌚️UTC+1]]

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

Comment 9

[Henrik Skupin [:whimboo][⌚️UTC+1]]

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.

Comment 10

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Oh, landed as:
https://github.com/whimboo/mozmill-crowd/commit/ba4c0d0975357c0f860299375392f2a17cef6eb7

Comment 11

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Comment on attachment 505289 [details] [diff] [review]
Version bump [checked-in]

Landed as:
https://github.com/whimboo/mozmill-crowd/commit/bdb74e0925d79f363bc376d2b89478feda301001

Comment 12

[Henrik Skupin [:whimboo][⌚️UTC+1]]

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.

Comment 13

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Attachment: Disable slow script warning during runtime [checked-in]

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.

Comment 14

[Aaron Train [:aaronmt]]

Comment on attachment 505471 [details] [diff] [review]
Disable slow script warning during runtime [checked-in]

Looks fine, r+. Let's get this in.

Comment 15

[Henrik Skupin [:whimboo][⌚️UTC+1]]

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

Comment 16

[Henrik Skupin [:whimboo][⌚️UTC+1]]

New version is up on AMO:
https://addons.mozilla.org/de/firefox/addon/mozmill-crowd/