736441 - fix test descriptions in mozmill/composition

Status: RESOLVED FIXED

(Thunderbird :: General, defect)

Description

[Magnus Melin [:mkmelin]]

Attachment: proposed fix

If you look at http://mxr.mozilla.org/comm-central/source/mail/test/mozmill/composition/ a lot of file descriptions are missing/wrong. Also true for other directories...

This fixes them for mozmill/composition.

Comment 1

[Mike Conley (:mconley) (:⚙️) (PTO June 8 - 12)]

Comment on attachment 606538 [details] [diff] [review]
proposed fix

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

This looks good.  Two nits.  Modulo those, r=me.

Thanks for your work,

-Mike

::: mail/test/mozmill/composition/test-attachment.js
@@ +35,5 @@
>   *
>   * ***** END LICENSE BLOCK ***** */
>  
> +/**
> + * Tests attachment handling functinality of the message compose window.

typo: functionality

::: mail/test/mozmill/composition/test-save-changes-on-quit.js
@@ +34,5 @@
>   *
>   * ***** END LICENSE BLOCK ***** */
>  
>  /**
> + * Tests what happens if you try to quit with a composition window that may

I'd prefer a description like,

"Tests that we prompt the user if they'd like to save their message when they try to quit with an open compose window with unsaved changes."

Comment 2

[Magnus Melin [:mkmelin]]

http://hg.mozilla.org/comm-central/rev/1a8835139dfc -> FIXED