1590788 - Update test case referencing social-tracking-protection-digest256

Status: RESOLVED FIXED

(Core :: Privacy: Anti-Tracking, task)

Description

[Se Yeon Kim]

After/during bug 1590779 the test cases in sections:

• Social Tracking Protection
• Social Tracking Protection and Fingerprinting
• Social Tracking Protection and Cryptomining
• Social Tracking Protection and Fingerprinting and Cryptomining
  created by :englehardt on https://senglehardt.com/test/trackingprotection/test_pages/ should be updated to check for domains in social-tracking-protection-facebook-digest256, social-tracking-protection-linkedin-digest256, social-tracking-protection-twitter-digest256

Should the testrail cases (https://testrail.stage.mozaws.net/index.php?/suites/view/354&group_by=cases:section_id&group_id=1173&group_order=asc) also be updated?

Comment 1

[(no longer active)]

Steven, do you mind updating these test pages please?

Comment 2

[Steven Englehardt [:englehardt]]

Se Yeon has been working on an overhaul of all of these test pages, so I'm going to redirect this request to her. Se Yeon, are you up for updating this as well?

Comment 3

[Arturo Mejia [:amejia]]

Should we also update the sections that reference the combined list base-track-digest256 as now we are using individual lists?

Comment 4

[Se Yeon Kim]

Ehsan, Steven
Yes, I will pick this up!

Arturo,
Can you clarify what you mean by update the sections? Are we talking about sections listed under https://senglehardt.com/test/trackingprotection/test_pages
If so, yes I intend to go through all the tests and remove the tests referencing base-track-digest256 and update to the separate list test, if it does not already exists.

Comment 5

[Arturo Mejia [:amejia]]

Exactly that was what I was referring to.
Thanks!

Comment 6

[Chenxia Liu [:liuche]]

Hi Se Yeon, is this something that we could get bumped? QA team is using these pages to test but they are not showing the right behavior.

Comment 7

[Se Yeon Kim]

Hi Chenxia,

Steven and I are working on the fix. We should get this done, at the latest, by this week.

Comment 8

[Steven Englehardt [:englehardt]]

Se Yeon's overhaul of the test pages is on hold for now due to higher priority work. I went ahead at swapped out the test domains for all of the test pages mentioned in Comment 0. These pages will now give the correct result for Nightly.

72 Beta requires https://github.com/mozilla-services/shavar-list-creation/pull/112 to land before these pages will work.

Comment 9

[Steven Englehardt [:englehardt]]

(In reply to Arturo Mejia from comment #3)

Should we also update the sections that reference the combined list base-track-digest256 as now we are using individual lists?

I also took care of this. All test pages on https://senglehardt.com/test/trackingprotection/test_pages/ no longer reference the base-track-digest256 list. Instead, the individual lists are used (ads-, analytics-, and social-).

Comment 10

[Steven Englehardt [:englehardt]]

Closing. The test rail cases will be updated separately.

Comment 11

[Se Yeon Kim]

https://github.com/mozilla-services/shavar-list-creation/pull/112 was merged and I tested that the pages work using the Staging environment. It's in Prod as well now.