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

Add Brave Search beta to the list of available search engines #15663

Closed
bsclifton opened this issue May 6, 2021 · 3 comments · Fixed by brave/brave-core#8966
Closed

Add Brave Search beta to the list of available search engines #15663

bsclifton opened this issue May 6, 2021 · 3 comments · Fixed by brave/brave-core#8966

Comments

@bsclifton
Copy link
Member

bsclifton commented May 6, 2021

Test plan

(for iOS, see brave/brave-ios#3745)

@brave/legacy_qa please also see brave/brave-core#8966 (comment) re: test cases for Android/Desktop.

Onboarding experience (Desktop)

  1. Fresh profile
  2. Upon launching Brave, welcome screen will show
  3. Navigate to fourth screen (Set default search engine)
    a. (US and Canada) Brave should show as an option in the dropdown
    b. (not US or Canada) Brave should not be in the dropdown

Onboarding experience (Android)

  1. Fresh profile
  2. Upon launching Brave, onboarding will show
  3. Confirm that:
    a. (US and Canada) Brave Search beta should show as the 2nd search option
    b. (not US or Canada) Brave should not show as an option won't be shown in this list

Search engines screen (all platforms)

  1. Fresh profile
  2. Launch Brave and go to settings
  3. Under settings, pick Search engines
  4. In settings pick Manage search engines
  5. Brave should show under Default search engines area. Go ahead and choose it as default
  6. Do a search in omnibox and verify search is going to search.brave.com SERP

Autocomplete suggestions test (all platforms)

  1. Fresh profile
  2. Launch Brave and go to settings
  3. Under settings, pick Privacy and security
  4. Set the toggle for Autocomplete searches and URLs to true
  5. Perform steps above (Search engines screen (all platforms)) to set Brave search as default
  6. Type in the omnibox. Try something like how do i and verify autocomplete results are being shown

Description

We recently announced having a search (https://brave.com/brave-search/) and this issue is intended to capture adding this to the browser as an alternate search. It would be in the Default search engines list in brave://settings/searchEngines

Because this search is currently limited (https://brave.com/search/ is currently waitlist'ing folks; team is in Beta phase), we should put this behind a flag in brave://flags. When disabled (default setting) it wouldn't show search in Default search engines list. If true (user needs to set manually) then it will.

When we're ready to expand search to a larger audience we can change the value of this using Griffin or just remove the flag

@bsclifton bsclifton added feature-request OS/Android Fixes related to Android browser functionality OS/Desktop labels May 6, 2021
@bsclifton bsclifton self-assigned this May 6, 2021
@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label May 6, 2021
bsclifton added a commit to brave/brave-core that referenced this issue May 19, 2021
bsclifton added a commit to brave/brave-core that referenced this issue May 25, 2021
bsclifton added a commit to brave/brave-core that referenced this issue May 27, 2021
bsclifton added a commit to brave/brave-core that referenced this issue Jun 11, 2021
bsclifton added a commit to brave/brave-core that referenced this issue Jun 11, 2021
bsclifton added a commit to brave/brave-core that referenced this issue Jun 14, 2021
bsclifton added a commit to brave/brave-core that referenced this issue Jun 16, 2021
bsclifton added a commit to brave/brave-core that referenced this issue Jun 17, 2021
@bsclifton bsclifton added this to the 1.28.x - Nightly milestone Jun 17, 2021
@kjozwiak kjozwiak removed this from the 1.28.x - Nightly milestone Jun 19, 2021
@kjozwiak
Copy link
Member

kjozwiak commented Jun 21, 2021

@brave/legacy_qa I've also added a bunch of more test cases that we ran via iOS that should also be checked via Android & Desktop. Please find them via brave/brave-core#8966 (comment). Combination of what @bsclifton provided via #15663 (comment) and what was checked via iOS.

Also important note, re: Default Callback API, we can now use search.brave.com rather than search-dev.brave.com.

@rebron rebron changed the title Add Brave Search as an alternative search Add Brave Search beta to the list of available search engines Jun 21, 2021
@srirambv srirambv added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Jun 21, 2021
@stephendonner
Copy link

stephendonner commented Jun 21, 2021

Verified PASSED using

Brave 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
Revision 4bb19460e8d88c3446b360b0df8fd991fee49c0b-refs/branch-heads/4472@{#1496}
OS macOS Version 11.4 (Build 20F71)

Onboarding/Setting Test Cases

  • ensured that Brave Search beta is only displayed under brave://welcome onboarding for Canada & USA
    • ensured that Japan, German & France weren't displaying Brave Search beta via search onboarding
  • ensured that Brave Search beta appears via brave://settings/search & brave://settings/searchEngines for all regions
  • ensured that the query URL appears as https://search.brave.com/search?q=%s&source=desktop under brave://settings/searchEngines
Japan onboarding Germany onboarding France onboarding USA & Canada onboarding
スクリーンショット 2021-06-21 10 53 20 Bildschirmfoto 2021-06-21 um 11 01 31 Capture d’écran 2021-06-21 à 11 05 17 Screen Shot 2021-06-21 at 10 26 39 AM
Japan SE settings Germany SE settings France SE settings USA & Canada settings
スクリーンショット 2021-06-21 10 54 23 Bildschirmfoto 2021-06-21 um 11 00 57 Capture d’écran 2021-06-21 à 11 05 33 Screen Shot 2021-06-21 at 11 08 59 AM

URL Query Test Cases

Screen Shot 2021-06-21 at 11 11 41 AM

Default Callback Test Cases

Test Case 1: (Setting as default via page)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • don't run through the search onboarding and leave Google as the default browser
  • login into https://search-dev.brave.com
  • search for Brave Browser via the text field
  • click on the Set default button via the default modal within https://search-dev.brave.com
  • you should see the button turn from Set default --> Done and disappear
  • check and ensure https://search-dev.brave.com is set as the default via brave://settings/search
example example example
Screen Shot 2021-06-21 at 11 16 43 AM Screen Shot 2021-06-21 at 11 17 09 AM Screen Shot 2021-06-21 at 11 17 17 AM

Test Case 2: (Private Browsing)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • don't run through the search onboarding and leave Google as the default browser
  • launch open * login into https://search-dev.brave.com via Private Browsing
  • search for Basic Attention Token via the text field
  • shouldn't see any Brave Search Default modals
Screen Shot 2021-06-21 at 11 23 05 AM

Test Case 3: (Default modal retries per session)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • don't run through the search onboarding and leave Google as the default browser
  • login into https://search-dev.brave.com
  • search for Brave Browser via the text field
  • the default modal should appear
  • refresh the page two times (should see the default modal appear
  • refresh again and you shouldn't see the default modal appear for this session
  • open a new tab, open https://search-dev.brave.com and try another search (shouldn't see any more modals for this session)

As per the above, this checks to make sure that the Default modal only appears three times per session.

Test Case 4: (Maximum 10 retries)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • don't run through the search onboarding and leave Google as the default browser
  • login into https://search-dev.brave.com
  • search for Brave Browser via the text field
  • the default modal should appear
  • refresh the page two times (should see the default modal appear
  • refresh again and you shouldn't see the default modal appear for this session
  • open a new tab, open https://search-dev.brave.com and try another search (shouldn't see any more modals for this session)
  • close Brave and move the time forward by 24hrs

Repeat the above. The basic logic for the above is that on desktop/android, we should only see 3 modals per session per day to a maximum of 10 in a span of four days.

Test Case 5: (Setting Brave Search as default via onboarding)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • Set Brave Search beta as the default SE via brave://welcome
  • login into https://search-dev.brave.com
  • search for Brave Browser via the text field
  • ensure that the default modal isn't being displayed

Test Case 6: (Setting Brave Search as default via brave://settings/search)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • Set Brave Search beta as the default SE via brave://settings/search
  • login into https://search-dev.brave.com
  • search for Brave Browser via the text field
  • ensure that the default modal isn't being displayed

Fallback Test Cases

  • ensured that the Enable Google Fallback mixing appears when searching for terms like brave 123 456 789
  • ensured that tapping on Dismiss correctly closes the Enable Google Fallback mixing card
    • ensured that it re-appears when another term that needs fallback is used
  • ensured that Learn more. opens https://search.brave.com/help/google-fallback
  • ensured that https://www.google.com isn't being contact when fallback is disabled
  • ensured that https://www.google.com is being contact once fallback has been enabled
example example example
Screen Shot 2021-06-21 at 12 42 10 PM Screen Shot 2021-06-22 at 11 45 28 AM Screen Shot 2021-06-22 at 11 45 46 AM

Verified PASSED using

Brave 1.26.65 Chromium: 91.0.4472.114 (Official Build) (64-bit)
Revision 4bb19460e8d88c3446b360b0df8fd991fee49c0b-refs/branch-heads/4472@{#1496}
OS Windows 10 OS Version 2009 (Build 21390.2025)

Onboarding/Setting Test Cases

  • ensured that Brave Search beta is only displayed under brave://welcome onboarding for Canada & USA
    • ensured that Japan, German & France weren't displaying Brave Search beta via search onboarding
  • ensured that Brave Search beta appears via brave://settings/search & brave://settings/searchEngines for all regions
  • ensured that the query URL appears as https://search.brave.com/search?q=%s&source=desktop under brave://settings/searchEngines
Japan onboarding Germany onboarding France onboarding USA & Canada onboarding
ja-onboarding deonboarding fronboarding us-onboarding
Japan SE settings Germany SE settings France SE settings USA & Canada settings
ja-settings desettings frsettings us-settings

URL Query Test Cases

Default Callback Test Cases

Test Case 1: (Setting as default via page)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • don't run through the search onboarding and leave Google as the default browser
  • login into https://search-dev.brave.com
  • search for Brave Browser via the text field
  • click on the Set default button via the default modal within https://search-dev.brave.com
  • you should see the button turn from Set default --> Done and disappear
  • check and ensure https://search-dev.brave.com is set as the default via brave://settings/search
example example example
set-default search-button-gone made-default

Test Case 2: (Private Browsing)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • don't run through the search onboarding and leave Google as the default browser
  • launch open * login into https://search-dev.brave.com via Private Browsing
  • search for Basic Attention Token via the text field
  • shouldn't see any Brave Search Default modals

private-search

Test Case 3: (Default modal retries per session)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • don't run through the search onboarding and leave Google as the default browser
  • login into https://search-dev.brave.com
  • search for Brave Browser via the text field
  • the default modal should appear
  • refresh the page two times (should see the default modal appear
  • refresh again and you shouldn't see the default modal appear for this session
  • open a new tab, open https://search-dev.brave.com and try another search (shouldn't see any more modals for this session)

As per the above, this checks to make sure that the Default modal only appears three times per session.

Test Case 4: (Maximum 10 retries)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • don't run through the search onboarding and leave Google as the default browser
  • login into https://search-dev.brave.com
  • search for Brave Browser via the text field
  • the default modal should appear
  • refresh the page two times (should see the default modal appear
  • refresh again and you shouldn't see the default modal appear for this session
  • open a new tab, open https://search-dev.brave.com and try another search (shouldn't see any more modals for this session)
  • close Brave and move the time forward by 24hrs

Repeat the above. The basic logic for the above is that on desktop/android, we should only see 3 modals per session per day to a maximum of 10 in a span of four days.

Test Case 5: (Setting Brave Search as default via onboarding)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • Set Brave Search beta as the default SE via brave://welcome
  • login into https://search-dev.brave.com
  • search for Brave Browser via the text field
  • ensure that the default modal isn't being displayed

Test Case 6: (Setting Brave Search as default via brave://settings/search)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • Set Brave Search beta as the default SE via brave://settings/search
  • login into https://search-dev.brave.com
  • search for Brave Browser via the text field
  • ensure that the default modal isn't being displayed

Fallback Test Cases

  • ensured that the Enable Google Fallback mixing appears when searching for terms like brave 123 456 789
  • ensured that tapping on Dismiss correctly closes the Enable Google Fallback mixing card
    • ensured that it re-appears when another term that needs fallback is used
  • ensured that Learn more. opens https://search.brave.com/help/google-fallback
  • ensured that https://www.google.com isn't being contact when fallback is disabled
  • ensured that https://www.google.com is being contact once fallback has been enabled
example example
no-fallback-wireshark fallback-google

Verification PASSED

Brave | 1.26.67 Chromium: 91.0.4472.114 (Official Build) (64-bit)
--- | ---
Revision | 4bb19460e8d88c3446b360b0df8fd991fee49c0b-refs/branch-heads/4472@{#1496}
OS | Linux

Onboarding/Setting Test Cases

  • ensured that Brave Search beta is only displayed under brave://welcome onboarding for Canada & USA
    • ensured that Japan, German & France weren't displaying Brave Search beta via search onboarding
  • ensured that Brave Search beta appears via brave://settings/search & brave://settings/searchEngines for all regions
  • ensured that the query URL appears as https://search.brave.com/search?q=%s&source=desktop under brave://settings/searchEngines
Japan onboarding Germany onboarding France onboarding USA & Canada onboarding
image image image image
Japan SE settings Germany SE settings France SE settings USA & Canada settings
image image image image

URL Query Test Cases

Default Callback Test Cases

Test Case 1: (Setting as default via page)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • don't run through the search onboarding and leave Google as the default browser
  • login into https://search-dev.brave.com
  • search for Brave Browser via the text field
  • click on the Set default button via the default modal within https://search-dev.brave.com
  • you should see the button turn from Set default --> Done and disappear
  • check and ensure https://search-dev.brave.com is set as the default via brave://settings/search
example example example
image image image

Test Case 2: (Private Browsing)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • don't run through the search onboarding and leave Google as the default browser
  • launch open * login into https://search-dev.brave.com via Private Browsing
  • search for Brave browser via the text field
  • shouldn't see any Brave Search Default modals

image

Test Case 3: (Default modal retries per session)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • don't run through the search onboarding and leave Google as the default browser
  • login into https://search-dev.brave.com
  • search for Brave Browser via the text field
  • the default modal should appear
  • refresh the page two times (should see the default modal appear
  • refresh again and you shouldn't see the default modal appear for this session
  • open a new tab, open https://search-dev.brave.com and try another search (shouldn't see any more modals for this session)

As per the above, this checks to make sure that the Default modal only appears three times per session.

Test Case 4: (Maximum 10 retries)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • don't run through the search onboarding and leave Google as the default browser
  • login into https://search-dev.brave.com
  • search for Brave Browser via the text field
  • the default modal should appear
  • refresh the page two times (should see the default modal appear
  • refresh again and you shouldn't see the default modal appear for this session
  • open a new tab, open https://search-dev.brave.com and try another search (shouldn't see any more modals for this session)
  • close Brave and move the time forward by 24hrs

Repeat the above. The basic logic for the above is that on desktop/android, we should only see 3 modals per session per day to a maximum of 10 in a span of four days.

Test Case 5: (Setting Brave Search as default via onboarding)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • Set Brave Search beta as the default SE via brave://welcome
  • login into https://search-dev.brave.com
  • search for Brave Browser via the text field
  • ensure that the default modal isn't being displayed
    image

Test Case 6: (Setting Brave Search as default via brave://settings/search)

  • install 1.26.65 Chromium: 91.0.4472.114 (Official Build) (x86_64)
  • Set Brave Search beta as the default SE via brave://settings/search
  • login into https://search-dev.brave.com
  • search for Brave Browser via the text field
  • ensure that the default modal isn't being displayed

image

Fallback Test Cases

  • ensured that the Enable Google Fallback mixing appears when searching for terms like brave 123 456 789
  • ensured that tapping on Dismiss correctly closes the Enable Google Fallback mixing card
    • ensured that it re-appears when another term that needs fallback is used
  • ensured that Learn more. opens https://search.brave.com/help/google-fallback
  • ensured that https://www.google.com isn't being contact when fallback is disabled
  • ensured that https://www.google.com is being contact once fallback has been enabled
example example example
image image image

@srirambv
Copy link
Contributor

srirambv commented Jun 22, 2021

Verification passed on OnePlus 6T with Android 10 running 1.26.65 x64 build & 1.26.67 x64 build

On-boarding

  • Verified Brave Search beta is on shown in SE onboarding screen only for US and Canada
  • Verified Brave Search beta is shown second on the list during onboarding
  • Verified Brave Search beta is not listed for other locales for SE onboarding
  • Verified selecting Brave Search beta during onboarding sets Brave search as default for both normal and private tabs

Search Engine Setting

  • Verified Brave Search beta is shown second on the list in search engine selection screen
  • Verified selecting Brave Search beta from the SE list sets it as default
  • Verified able to set Brave Search beta as default for both normal and private tabs from search engine selection screen
  • Verified upgrading from 1.25.x to 1.26.65 adds Brave Search beta to the list at the bottom

URL Tests

  • Verified setting Brave Search beta as default and doing a search from omnibox adds &source=android to the query URL
    Ex: https://search.brave.com/?q=test&source=android
  • Verified performing a search on the search page doesn't append any &source=android to the query URL

Default Callback

Case 1: via Search page

  • Install 1.26.65
  • Select URL bar to trigger SE onboarding
  • Select continue so Google is set as default for both normal and private tab
  • Visit https://search.brave.com and perform a search
  • Click on Set as defaultand wait for the modal to change to Done and then auto-dismiss
  • Verified Brave Search beta is set as default for normal mode only and not for private mode

Case 2: via Search Page on Private tab

  • Install 1.26.65
  • Select URL bar to trigger SE onboarding
  • Select continue so Google is set as default for both normal and private tab
  • Visit https://search.brave.com and perform a search in private tab
  • Verified no Set as default modal is shown on SERP
  • Verified search engines settings remain same as before in settings

Case 3: Default modal retries

  • Install 1.26.65
  • Select URL bar to trigger SE onboarding
  • Select continue so Google is set as default for both normal and private tab
  • Visit https://search.brave.com and perform a search
  • Set as default modal should be shown
  • Reload the page two times, modal should still be shown
  • Reload the page a third time, modal should be removed
  • Open a new tab and perform a search on search.brave.com, no default modal is shown
  • Clear browsing data and try again, still no default modal is shown

Case 4: Default modal retries (max 10)

  • Install 1.26.65
  • Select URL bar to trigger SE onboarding
  • Select continue so Google is set as default for both normal and private tab
  • Visit https://search.brave.com and perform a search
  • Set as default modal should be shown
  • Reload the page two times, modal should still be shown
  • Reload the page a third time, modal should be removed
  • Open a new tab and perform a search on search.brave.com, no default modal is shown
  • Close browser and move device time by 24hr
  • Search again on search.brave.com, default modal should be shown

Fallback Test Cases

  • Verified Enable Google Fallback mixing shows up when searching for term like brave 123 456
  • Verified tapping on Dismiss hides the fallback modal
  • Verified modal reappears every time a weird search term is used like above
  • Verified Learn more. opens https://search.brave.com/help/google-fallback
  • Verified https://www.google.com isn't being contact when fallback is disabled
  • Verified https://www.google.com is being contact once fallback has been enabled
image image

Verification passed on OnePlus 6T with Android 10 running 1.26.65 x64 build & 1.26.67 x64 build

On-boarding

  • Verified Brave Search beta is on shown in SE onboarding screen only for US and Canada
  • Verified Brave Search beta is shown second on the list during onboarding
  • Verified Brave Search beta is not listed for other locales for SE onboarding
  • Verified selecting Brave Search beta during onboarding sets Brave search as default for both normal and private tabs

Search Engine Setting

  • Verified Brave Search beta is shown second on the list in search engine selection screen
  • Verified selecting Brave Search beta from the SE list sets it as default
  • Verified able to set Brave Search beta as default for both normal and private tabs from search engine selection screen
  • Verified upgrading from 1.25.x to 1.26.65 adds Brave Search beta to the list at the bottom

URL Tests

  • Verified setting Brave Search beta as default and doing a search from omnibox adds &source=android to the query URL
    Ex: https://search.brave.com/?q=test&source=android
  • Verified performing a search on the search page doesn't append any &source=android to the query URL

Default Callback

Case 1: via Search page

  • Install 1.26.65
  • Select URL bar to trigger SE onboarding
  • Select continue so Google is set as default for both normal and private tab
  • Visit https://search.brave.com and perform a search
  • Click on Set as defaultand wait for the modal to change to Done and then auto-dismiss
  • Verified Brave Search beta is set as default for normal mode only and not for private mode

Case 2: via Search Page on Private tab

  • Install 1.26.65
  • Select URL bar to trigger SE onboarding
  • Select continue so Google is set as default for both normal and private tab
  • Visit https://search.brave.com and perform a search in private tab
  • Verified no Set as default modal is shown on SERP
  • Verified search engines settings remain same as before in settings

Case 3: Default modal retries

  • Install 1.26.65
  • Select URL bar to trigger SE onboarding
  • Select continue so Google is set as default for both normal and private tab
  • Visit https://search.brave.com and perform a search
  • Set as default modal should be shown
  • Reload the page two times, modal should still be shown
  • Reload the page a third time, modal should be removed
  • Open a new tab and perform a search on search.brave.com, no default modal is shown
  • Clear browsing data and try again, still no default modal is shown

Case 4: Default modal retries (max 10)

  • Install 1.26.65
  • Select URL bar to trigger SE onboarding
  • Select continue so Google is set as default for both normal and private tab
  • Visit https://search.brave.com and perform a search
  • Set as default modal should be shown
  • Reload the page two times, modal should still be shown
  • Reload the page a third time, modal should be removed
  • Open a new tab and perform a search on search.brave.com, no default modal is shown
  • Close browser and move device time by 24hr
  • Search again on search.brave.com, default modal should be shown

Fallback Test Cases

  • Verified Enable Google Fallback mixing shows up when searching for term like brave 123 456
  • Verified tapping on Dismiss hides the fallback modal
  • Verified modal reappears every time a weird search term is used like above
  • Verified Learn more. opens https://search.brave.com/help/google-fallback
  • Verified https://www.google.com isn't being contact when fallback is disabled
  • Verified https://www.google.com is being contact once fallback has been enabled

@kjozwiak kjozwiak removed the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Jun 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment