1605030 - Remove mozmill tests and infrastructure from the tree

Status: RESOLVED FIXED

(Thunderbird :: Testing Infrastructure, task)

Description

[Geoff Lankow (:darktrojan)]

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.

Comment 1

[Geoff Lankow (:darktrojan)]

Attachment: 1605030-remove-mozmill-tasks-1.diff

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.

Comment 2

[Rob Lemley [:rjl]]

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?

Comment 3

[Geoff Lankow (:darktrojan)]

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?

Comment 4

[Geoff Lankow (:darktrojan)]

Attachment: Bug 1605030 - Remove mozmill infrastructure from the tree r?#firefox-build-system-reviewers

Comment 5

[Magnus Melin [:mkmelin]]

Comment on attachment 9117487 [details] [diff] [review]
1605030-remove-mozmill-tasks-1.diff

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

Wohoo! r=mkmelin

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/f760a8ee7478
Remove mozmill infrastructure from the tree. r=rjl,mkmelin

Comment 7

[Geoff Lankow (:darktrojan)]

Attachment: 1605030-remove-mozmill-tests-1.diff

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.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/69a27f586f5d
Remove mozmill tests from the tree. r=mkmelin

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7dc8f80d45a8
Remove mozmill infrastructure from the tree r=firefox-build-system-reviewers,mshal

Comment 10

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/7dc8f80d45a8