Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migration to Core has broken URL comparison for landed ad, easter egg and search engine state #3680

Closed
tmancey opened this issue Mar 12, 2019 · 2 comments · Fixed by brave/brave-core#1914

Comments

@tmancey
Copy link
Contributor

tmancey commented Mar 12, 2019

Description

Migration of bat-native-ads to Core has broken URL comparison for landed ad, easter egg and search engine state

Steps to Reproduce broken URL for search engine

  1. Open a search engine, i.e. www.google.com

Actual result:

Search engine state is not updated in Default/ads_service/client.json

Expected result:

Search engine state should be updated in Default/ads_service/client.json

Steps to Reproduce broken URL for landed Ad

  1. View an Ad
  2. Navigate to another tab
  3. Wait 10+ seconds

Actual result:

Ad is Landed

Expected result:

Ad should not be landed as user navigated away before 10 seconds had elapsed

Steps to Reproduce broken URL for easter egg

  1. Open www.iab.com

Actual result:

Easter egg is not activated

Expected result:

Easter egg should be activated

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

0.63.x

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds? Dev

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields? N/A
  • Is the issue reproducible on the latest version of Chrome? N/A

Additional Information

@tmancey tmancey added this to the 0.64.x - Nightly milestone Mar 12, 2019
@tmancey tmancey self-assigned this Mar 12, 2019
@tmancey tmancey added the QA/Yes label Mar 12, 2019
@tmancey tmancey added the bug label Mar 12, 2019
@tmancey tmancey changed the title Migration to Core has broken URL comparison for easter egg and search engines Migration to Core has broken URL comparison for landed ad, easter egg and search engines Mar 12, 2019
@tmancey tmancey changed the title Migration to Core has broken URL comparison for landed ad, easter egg and search engines Migration to Core has broken URL comparison for landed ad, easter egg and search engine state Mar 12, 2019
@bsclifton
Copy link
Member

Updated milestone after brave/brave-core#1925 was merged
@tmancey can you update the top post here to include a test plan?

@srirambv
Copy link
Contributor

srirambv commented Apr 17, 2019

Verification passed on

Brave 0.63.44 Chromium: 74.0.3729.75 (Official Build) beta(64-bit)
Revision fdb7915642fef8cf997beac2554709d148e3c187-refs/branch-heads/3729@{#754}
OS Linux
  • Search engine entry is updated on client.json file
    image
  • Verified after viewing an ad navigating a new tab doesn't trigger an ad
    image
  • Verified Ad shown on iab.com based on category business and not business-advertising. Verified with @jsecretan that this is expected

Verification passed by Geeta on

Brave 0.63.45 Chromium: 74.0.3729.75 (Official Build) beta (64-bit)
Revision fdb7915642fef8cf997beac2554709d148e3c187-refs/branch-heads/3729@{#754}
OS Windows 10 OS Build 17134.523

3680
3680_1

Verified passed with

Brave 0.63.45 Chromium: 74.0.3729.75 (Official Build) beta(64-bit)
Revision fdb7915642fef8cf997beac2554709d148e3c187-refs/branch-heads/3729@{#754}
OS Mac OS X
  • Verified test cases from description:
  • Search engine entry is updated on client.json file
  • Verified after viewing an ad navigating a new tab doesn't trigger an ad
  • Verified Ad shown on iab.com based on category business and not business-advertising. Verified with @jsecretan that this is expected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment