test pilot thunderbird 'all your user studies' shows each more than once

RESOLVED FIXED in Thunderbird 13.0

Status

Thunderbird
General
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Gregg Lind (Fx Strategy and Insights - Shield - Heartbeat ), Assigned: Gregg Lind (Fx Strategy and Insights - Shield - Heartbeat ))

Tracking

unspecified
Thunderbird 13.0

Thunderbird Tracking Flags

(thunderbird11 fixed, thunderbird12 fixed)

Details

Attachments

(2 attachments)

Created attachment 593887 [details]
screenshot showing doubled printing of results.

To reproduce:

1.  studies in ``https://testpilot.mozillalabs.com/testcases/index-tb-dev.json``
2.  tbird with test pilot 1.2:  Preferences > advanced > config editor:  

    extensions.testpilot.indexFileName -> index-tb-dev.json

3.  (clear the cached download - OSX)... 

    $ rm ~/Library/Thunderbird/Profiles/*.default//TestPilotExperimentFiles/index.json

4.  Restart Tbird.
5.  GOOD - doorhanger appears about new study
6.  BAD - ``tools > test pilot > all your user studies > [current]`` 
    (studies show up multiple times)

To investigate...

Starting with ``https://github.com/mozsquib/testpilot.git``, going to look for issues in ``setup.js``

Cf:  https://wiki.mozilla.org/Labs/Test_Pilot/debug
after uninstalling and reinstalling the latest version of the extension (1.3.4), this problem goes away.  @rebron, what version comes with TBird?
1.3.4 currently, but 1.3.5 soon.

I believe I've been seeing this in 1.3.4 though.
Created attachment 594237 [details] [diff] [review]
patch for bug.  Neeeds review.

Uses the ``observer`` correctly, rather than the kludgey "assume only one window" hacks that were there before.  

As per Mark Banner:

09:24 <Standard8> updated patch: http://pastebin.mozilla.org/1473807
09:25 <Standard8> the magic is in chrome.manifest
09:25 <Standard8> it sets up TestPilotComponent as a component, and registers it in the profile-after-change category
09:26 <Standard8> when that is going to be called out to the observers, the component gets created as a service, and 
                  then the observer function gets called with profile-after-change
09:26 <Standard8> at which point it registers iteself for mail-startup-done, which is the thing you probably really want 
                  for  Thundebird
Attachment #594237 - Flags: review?(squibblyflabbetydoo)

Comment 4

5 years ago
Comment on attachment 594237 [details] [diff] [review]
patch for bug.  Neeeds review.

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

I wasn't able to reproduce the specific issue in comment 0, but this works for me. We should remove the dump() statements though.
Attachment #594237 - Flags: review?(squibblyflabbetydoo) → review+
Checked in for Thunderbird:

http://hg.mozilla.org/comm-central/rev/fc91f140aae6

I also bumped the version that comes with Thunderbird so I can get this up on AMO:

http://hg.mozilla.org/comm-central/rev/fb4163ac866a

Moving this to the Thunderbird product so that we can track it is fixed there, if we unify the extensions, then we'll probably want a different method.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Component: Test Pilot → General
Product: Mozilla Labs → Thunderbird
QA Contact: test-pilot → general
Resolution: --- → FIXED
Attachment #594237 - Flags: approval-comm-beta?
Attachment #594237 - Flags: approval-comm-aurora?
Target Milestone: --- → Thunderbird 13.0
Attachment #594237 - Flags: approval-comm-beta?
Attachment #594237 - Flags: approval-comm-beta+
Attachment #594237 - Flags: approval-comm-aurora?
Attachment #594237 - Flags: approval-comm-aurora+
Checked into branches:

http://hg.mozilla.org/releases/comm-aurora/rev/3afe7fc35f3
http://hg.mozilla.org/releases/comm-aurora/rev/1f916fceb081
http://hg.mozilla.org/releases/comm-beta/rev/b6fad7040c8e
http://hg.mozilla.org/releases/comm-beta/rev/1257dccef380
status-thunderbird11: --- → fixed
status-thunderbird12: --- → fixed

Comment 7

5 years ago
(In reply to Mark Banner (:standard8) from comment #5)
> http://hg.mozilla.org/comm-central/rev/fc91f140aae6

This still has the debug dump() statements in it. We should remove those.
You need to log in before you can comment on or make changes to this bug.