Mock - Default HeartBeat Survey is not shown 7 days later

RESOLVED WORKSFORME

Status

Shield
General
RESOLVED WORKSFORME
a year ago
a year ago

People

(Reporter: Ovidiu, Unassigned)

Tracking

(Blocks: 1 bug)

unspecified

Firefox Tracking Flags

(firefox53 affected, firefox54 affected)

Details

(URL)

(Reporter)

Description

a year ago
[Affected versions]:

Firefox Nighlty 54.0a1(2017-02-23), Firefox Aurora 53.0a2(2017-02-23)

[Affected platforms]:

Mac OS X 10.10, Ubuntu 16.04, Win 10


[Steps to reproduce]:


Preconditions:

1. Obtain a copy of Firefox with the SHIELD recipe client system add-on installed. You can check about:support to ensure that you have it.
2. Set the extensions.shield-recipe-client.dev_mode preference to true to run recipes immediately on startup.
3. Set the extensions.shield-recipe-client.logging.level preference to 0 to enable more logging.
4. Set the security.content.signature.root_hash preference to 4C:35:B1:C3:E3:12:D9:55:E7:78:ED:D0:A7:E7:8A:38:83:04:EF:01:BF:FA:03:29:B2:46:9F:3C:C5:EC:36:04. This is the public hash used for verifying content signatures of the test data


Steps:

1 Open Firefox and Open about:config and set 
extensions.shield-recipe-client.api_url
to
https://normandy-mock.dev.mozaws.net/ShowHeartbeatButton/api/v1
2. Restart FF browser
3. A heartbeat Survey is shown.
4. Close firefox
5. Reopen Firefox -> Recipe is not executed again, the survey is not shown
6. Close Firefox and Modify system date and add 8 days to system time
7. Open Firefox


[Expected result]:

The survey is shown.


[Actual result]:

The survey doesn't appear.
(Reporter)

Comment 1

a year ago
I also tried to modify shield-recipe-client.json and rewinding the date with 8-10 days but the heartbeat wasn't triggered.
So from what I can tell, when we added repeat options for Heartbeat dialogs, we didn't update the mock-server testcases, so the mock-server recipes default to the "show once and never again" behavior, which matches the behavior you saw.

I think the bug here is to update the mock-server test cases to cover the three different repeat options. Once that's up, we can re-test and verify if they're behaving as expected. Does that sound good?
(Reporter)

Comment 3

a year ago
That is a great idea, I will make sure to cover all the options from the drop down. Please announce me when that's up. Thanks
(Reporter)

Comment 4

a year ago
Mike, just to make this clear, when you say "three different repeat options" on what recipes do you refer? 
Sorry for this but I want to make sure that I understand correctly. Thanks
Flags: needinfo?(mkelly)
(Reporter)

Comment 5

a year ago
I figured out what I wanted to ask you. I will create 3 test cases that will cover all the scenarios. Sorry for the spam.
Flags: needinfo?(mkelly)
Ovidiu: We still haven't gotten around to updating the mock server testcases, did you test this against the staging server? If the feature is working, we should reword or close this bug since functionally it's working as expected, I think.
Flags: needinfo?(ovidiu.boca)
(Reporter)

Comment 7

a year ago
We tested with a new profile on beta version FF 53.0b9 and everything works as expected. We also run a test with an old profile and on Nightly FF 55.0a1(2017-04-05) it worked but on beta with the same profile it didn't. We'll investigate this situation until then I will leave my NI? flag.
(Reporter)

Comment 8

a year ago
We figured out what was the problem, on the older profile the shield wasn't installed on Beta only on Nightly. 
I will mark this as resolved WFM based on comment 7.
Status: NEW → RESOLVED
Last Resolved: a year ago
Flags: needinfo?(ovidiu.boca)
Resolution: --- → WORKSFORME
Summary: Default HeartBeat Survey is not shown 7 days later → Mock - Default HeartBeat Survey is not shown 7 days later
You need to log in before you can comment on or make changes to this bug.