Closed Bug 1375376 Opened 7 years ago Closed 7 years ago

Write a test for clicking on addbook: link from vCard displayed inline (test for bug 1374779)

Categories

(Thunderbird :: Message Reader UI, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 56.0

People

(Reporter: jorgk-bmo, Assigned: aceman)

References

Details

Attachments

(1 file, 1 obsolete file)

+++ This bug was initially created as a clone of Bug #1374779 +++

Write a test for clicking on addbook: link from vCard displayed inline

STR:
Set attachments to display inline.
Import and open message from attachment 8879692 [details].
Click on the vCard link.

Expected result:
chrome://messenger/content/addressbook/abNewCardDialog.xul opens.
Flags: needinfo?(acelists)
Assignee: nobody → acelists
Status: NEW → ASSIGNED
Flags: needinfo?(acelists)
OS: Unspecified → All
Hardware: Unspecified → All
Attached patch patch (obsolete) — Splinter Review
This should do it. I have confirmed backing out bug 1374779 makes the test fail.
Attachment #8882836 - Flags: review?(jorgk)
Comment on attachment 8882836 [details] [diff] [review]
patch

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

Thanks!

Now over to bug 1352740 and bug 1352739? ;-)

::: mail/test/mozmill/message-window/test-vcard-actions.js
@@ +4,5 @@
> +
> +/**
> + * Tests for attached vcards.
> + */
> +

Please add:
// make SOLO_TEST=message-window/test-vcard-actions.js mozmill-one

@@ +49,5 @@
> +  // Check new card was created from the vcard.
> +  newcards = get_cards_in_all_address_books_for_email("meister@example.com");
> +  assert_equals(newcards.length, 1);
> +
> +  msgc.sleep(5000);

Why sleep here?
Attachment #8882836 - Flags: review?(jorgk) → review+
Attached patch patch v1.1Splinter Review
Debugging leftover :)
Attachment #8882836 - Attachment is obsolete: true
Attachment #8882837 - Flags: review+
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/7d0b8cd8b49d7e192ecc10c0c70a13af6a2a8f87
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 56.0
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: