1046998 - Modify test_copyThenMoveManual.js to use Promises

Status: RESOLVED FIXED

(MailNews Core :: Testing Infrastructure, defect)

Description

[Suyash Agarwal (:sshagarwal)]

We intend to replace async calls with promises.
This is another attempt in the same direction.

Comment 1

[Suyash Agarwal (:sshagarwal)]

Attachment: b1046998.patch

Comment 2

[Kent James (:rkent)]

Comment on attachment 8465701 [details] [diff] [review]
b1046998.patch

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

r=me with one nit fixed.

::: mailnews/base/test/unit/test_copyThenMoveManual.js
@@ +85,5 @@
>      // operation was a move
>      do_check_eq(folderCount(localAccountUtils.inboxFolder), 0);
> +  },
> +  function endTest() {
> +    // Cleanup, null out everything, close all cached connections and stop the 

Nit: trailing space

Comment 3

[Suyash Agarwal (:sshagarwal)]

Attachment: Patch for check-in

Thanks.

Comment 4

[Mark Banner (:standard8)]

https://hg.mozilla.org/comm-central/rev/0ba79665d6fc