1310022 - cleanup test-charset-upgrade.js

Status: RESOLVED FIXED

(Thunderbird :: Testing Infrastructure, enhancement)

Description

[:aceman]

1. We can use some existing helpers instead of open-coding in test-charset-upgrade.js, like setup_msg_contents
2. There is teardownmodule() function twice.
3. Calling Send later then does not wait for window close.

Comment 1

[:aceman]

Attachment: patch

Comment 2

[Jorg K (CEST = GMT+2)]

Comment on attachment 8800866 [details] [diff] [review]
patch

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

Drive-by review. I guess it's all going to change again in bug 1310024, so let's get this one here done.

::: mail/test/mozmill/composition/test-charset-upgrade.js
@@ -45,5 @@
> -                   .localFoldersServer
> -                   .rootFolder
> -                   .containsChildNamed("Outbox")) {
> -     create_folder("Outbox", [Ci.nsMsgFolderFlags.Outbox]);
> -  }

Why are you removing this? What are you doing in bug 1310024 for this one?
 outboxFolder = get_special_folder(Ci.nsMsgFolderFlags.Queue);
right? So that equates to creating it.
Am I missing something?

Comment 3

[:aceman]

(In reply to Jorg K (GMT+2) from comment #2)
> ::: mail/test/mozmill/composition/test-charset-upgrade.js
> @@ -45,5 @@
> > -                   .localFoldersServer
> > -                   .rootFolder
> > -                   .containsChildNamed("Outbox")) {
> > -     create_folder("Outbox", [Ci.nsMsgFolderFlags.Outbox]);
> > -  }
> 
> Why are you removing this? What are you doing in bug 1310024 for this one?
>  outboxFolder = get_special_folder(Ci.nsMsgFolderFlags.Queue);
> right? So that equates to creating it.
> Am I missing something?

Yes - "missing". For Outbox we do not create it automatically, I made an exception in the function.
We just expect it is there, or we fail.

Comment 4

[Jorg K (CEST = GMT+2)]

OK. You have my r+ already. Just put a big comment in bug 1310024 to explain when/why you're creating folders and when/why you're not.

Comment 5

[:aceman]

Thanks. Comment added.
https://hg.mozilla.org/comm-central/rev/118a8202a7457b3d9ffecbbe97feff4e8287bf70