Closed Bug 1605030 Opened 7 months ago Closed 6 months ago

Remove mozmill tests and infrastructure from the tree

Categories

(Thunderbird :: Testing Infrastructure, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 73.0

People

(Reporter: darktrojan, Assigned: darktrojan)

References

Details

(Keywords: leave-open)

Attachments

(3 files)

This is more than just hg rm mail/test/mozmill mail/test/resources. There are some bits we still need to have somewhere. I also need to pull out the CI configuration for running the tests.

This removes the TaskCluster stuff but not the tests themselves. There's a corresponding patch to remove a bunch of things from mozilla-central but I'm not posting that now as my tree is in use.

Attachment #9117487 - Flags: review?(rob)
Comment on attachment 9117487 [details] [diff] [review]
1605030-remove-mozmill-tasks-1.diff

Review of attachment 9117487 [details] [diff] [review]:
-----------------------------------------------------------------

I'm good with the Taskcluster and build files. Javascript stuff you might want a second opinion.

::: taskcluster/ci/source-test/mozlint.yml
@@ -75,5 @@
>      run:
>          mach: >
>              lint -f treeherder -l flake8 -n
>              comm/taskcluster
> -            comm/mail/test/resources/installmozmill.py

Just removes that file from source test, right?
Attachment #9117487 - Flags: review?(rob) → review+

Yes it does. That source test should be doing a lot more, but that's a story for another bug.

Also, you know we can't see what you marked as reviewed, right?

Attachment #9117487 - Flags: review?(mkmelin+mozilla)
Comment on attachment 9117487 [details] [diff] [review]
1605030-remove-mozmill-tasks-1.diff

Review of attachment 9117487 [details] [diff] [review]:
-----------------------------------------------------------------

Wohoo! r=mkmelin
Attachment #9117487 - Flags: review?(mkmelin+mozilla) → review+
Keywords: leave-open
Target Milestone: --- → Thunderbird 73.0
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/f760a8ee7478
Remove mozmill infrastructure from the tree. r=rjl,mkmelin

To remove the tests I'll apply this patch, plus remove the remaining contents of mail/test/mozmill and mail/test/resources. I didn't put that part of the patch in because it's rather pointless uploading 40000 lines of code to be deleted.

Attachment #9117761 - Flags: review?(mkmelin+mozilla)
Attachment #9117761 - Flags: review?(mkmelin+mozilla) → review+
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/69a27f586f5d
Remove mozmill tests from the tree. r=mkmelin
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/integration/autoland/rev/7dc8f80d45a8
Remove mozmill infrastructure from the tree r=firefox-build-system-reviewers,mshal
You need to log in before you can comment on or make changes to this bug.