Permanent orange: TEST-UNEXPECTED-FAIL | test-toolbar.js | test-toolbar.js::test_new_toolbar_with_default_tb3

RESOLVED FIXED in Thunderbird 12.0

Status

Product:
Thunderbird
Component:
Testing Infrastructure
Type:
defect
Status:
RESOLVED FIXED
Opened:
8 years ago
Updated:
7 years ago

People

(Reporter: mconley, Assigned: mconley)

Tracking

({intermittent-failure})

Version:
Trunk
Target:
Thunderbird 12.0
Platform:
x86
macOS
Dependency tree / graph

Thunderbird Tracking Flags

(thunderbird11 fixed)

Details

Attachments

(1 attachment)

Patch v1
1.19 KB, patch
standard8
: review+
standard8
: approval-comm-aurora+
Details | Diff | Splinter Review
Assignee

Description

•
8 years ago 
The patch for bug 715565 broke some of our tests.  We're seeing the following on trunk:

SUMMARY-UNEXPECTED-FAIL | test-toolbar.js | test-toolbar.js::test_new_toolbar_with_default_tb3
  EXCEPTION: The currentset is incorrect.: 'button-getmsg,button-newmsg,button-address,spacer,button-tag,spring,gloda-search' != 'button-getmsg,button-newmsg,button-address,spacer,button-tag,spring,gloda-search,throbber-box'.
    at: test-folder-display-helpers.js line 2842
       assert_true(false,"The currentset is incorrect.: 'button-getmsg,button-newmsg,button-address,spacer,button-tag,spring,gloda-search' != 'button-getmsg,button-newmsg,button-address,spacer,button-tag,spring,gloda-search,throbber-box'.") test-folder-display-helpers.js 2842
       assert_equals("button-getmsg,button-newmsg,button-address,spacer,button-tag,spring,gloda-search","button-getmsg,button-newmsg,button-address,spacer,button-tag,spring,gloda-search,throbber-box","The currentset is incorrect.") test-folder-display-helpers.js 2829
       assert_default_tb3_settings([object XULElement],true) test-toolbar.js 96
       test_new_toolbar_with_default_tb3() test-toolbar.js 164
            frame.js 557
            frame.js 626
            frame.js 669
            frame.js 497
            frame.js 675
            server.js 179
            server.js 183
SUMMARY-UNEXPECTED-FAIL | test-toolbar.js | test-toolbar.js::test_new_toolbar_with_custom_tb3
  EXCEPTION: The currentset is incorrect.: 'button-getmsg,button-newmsg,button-address,spacer,button-tag,spring,gloda-search' != 'button-getmsg,button-newmsg,button-address,spacer,button-tag,spring,gloda-search,throbber-box'.
    at: test-folder-display-helpers.js line 2842
       assert_true(false,"The currentset is incorrect.: 'button-getmsg,button-newmsg,button-address,spacer,button-tag,spring,gloda-search' != 'button-getmsg,button-newmsg,button-address,spacer,button-tag,spring,gloda-search,throbber-box'.") test-folder-display-helpers.js 2842
       assert_equals("button-getmsg,button-newmsg,button-address,spacer,button-tag,spring,gloda-search","button-getmsg,button-newmsg,button-address,spacer,button-tag,spring,gloda-search,throbber-box","The currentset is incorrect.") test-folder-display-helpers.js 2829
       assert_default_tb3_settings([object XULElement],false) test-toolbar.js 96
       test_new_toolbar_with_custom_tb3() test-toolbar.js 204
            frame.js 557
            frame.js 626
            frame.js 669
            frame.js 497
            frame.js 675
            server.js 179
            server.js 183
 Assignee

Updated

•
8 years ago
Blocks: 715565
Whiteboard: [tb-orange]
 Assignee

Comment 1

•
8 years ago
Posted patch Patch v1 — — Splinter Review 
This fixes the test for me, locally.

It's a bit strange that this test seems to test the migration to TB3..."TBCurrent" might be more accurate.
Assignee: nobody → mconley
Status: NEW → ASSIGNED
Attachment #591788 - Flags: review?(mbanner)
Attachment #591788 - Flags: review?(mbanner) → review+
 Assignee

Updated

•
8 years ago
Attachment #591788 - Flags: approval-comm-aurora?
 Assignee

Comment 2

•
8 years ago 
Landed on comm-central as http://hg.mozilla.org/comm-central/rev/09fa0d1410b9
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 12.0
Attachment #591788 - Flags: approval-comm-aurora? → approval-comm-aurora+
Whiteboard: [tb-orange]
You need to log in before you can comment on or make changes to this bug.