732334 - Need test for bug 731907

Status: NEW

(MailNews Core :: Composition, defect)

Description

[Hiroyuki Ikezoe (:hiro)]

Attachment: A test

No more enbug[1]!

[1] http://enbug.org/about

Comment 1

[Hiroyuki Ikezoe (:hiro)]

Comment on attachment 602265 [details]
A test

Clearing review flag.

This test hits the assertion at http://mxr.mozilla.org/comm-central/source/mailnews/compose/src/nsMsgAttachmentHandler.cpp#1003.

Ummm, the assertion is really necessary..?

Comment 2

[:aceman]

Could the test cover both bug 235432 and bug 731907?
As in, check that temp files were deleted but sources of attachments were not (you already covered this second part)?

Comment 3

[Hiroyuki Ikezoe (:hiro)]

(In reply to :aceman from comment #2)
> Could the test cover both bug 235432 and bug 731907?
> As in, check that temp files were deleted but sources of attachments were
> not (you already covered this second part)?

First of all, I do not actually prefer a test covers plural test cases.

The test for bug 235432 I've been trying to write can not be achieved yet because I can not find the timing the temporary file (nsqmail.tmp or something) still exits from the test.

Comment 4

[Wayne Mery (:wsmwk)]

aceman, could you polish this off and land?