Release improved Mozmill Crowd v0.1.1 for localizers

RESOLVED FIXED in 0.1

Status

RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: whimboo, Assigned: whimboo)

Tracking

unspecified

Details

(Whiteboard: [0.1.1])

Attachments

(6 attachments)

(Assignee)

Description

8 years ago
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.
(Assignee)

Updated

8 years ago
Depends on: 627205
(Assignee)

Comment 1

8 years ago
Created attachment 505213 [details] [diff] [review]
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.
Attachment #505213 - Flags: review?(aaron.train)

Updated

8 years ago
Attachment #505213 - Flags: review?(aaron.train) → review+
Created attachment 505265 [details] [diff] [review]
Patch v1 - ('Path not found' string change) [checked-in]

String change on the path not found property
Attachment #505265 - Flags: review?(hskupin)
(Assignee)

Updated

8 years ago
Attachment #505265 - Flags: review?(hskupin) → review+
(Assignee)

Updated

8 years ago
Attachment #505265 - Attachment description: Patch v1 - ('Path not found' string change) → Patch v1 - ('Path not found' string change) [checked-in]
(Assignee)

Comment 3

8 years ago
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]
(Assignee)

Comment 4

8 years ago
Created attachment 505282 [details] [diff] [review]
Show log information after the test-run [checked-in]
Attachment #505282 - Flags: review?(aaron.train)
(Assignee)

Comment 5

8 years ago
Created attachment 505287 [details] [diff] [review]
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.
Attachment #505287 - Flags: review?(aaron.train)
(Assignee)

Comment 6

8 years ago
Created attachment 505289 [details] [diff] [review]
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.
Attachment #505289 - Flags: review?(aaron.train)

Updated

8 years ago
Attachment #505287 - Flags: review?(aaron.train) → review+

Updated

8 years ago
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+
(Assignee)

Comment 8

8 years ago
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]
(Assignee)

Comment 9

8 years ago
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]
(Assignee)

Comment 11

8 years ago
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]
(Assignee)

Comment 12

8 years ago
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.
(Assignee)

Comment 13

8 years ago
Created attachment 505471 [details] [diff] [review]
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.
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+
(Assignee)

Comment 15

8 years ago
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]
(Assignee)

Comment 16

8 years ago
New version is up on AMO:
https://addons.mozilla.org/de/firefox/addon/mozmill-crowd/
Status: ASSIGNED → RESOLVED
Last Resolved: 8 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.