Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Manual test run on Windows x64 for 0.21.x w/ Chromium 65 (Release Channel) #13506

Closed
23 of 29 tasks
kjozwiak opened this issue Mar 19, 2018 · 4 comments
Closed
23 of 29 tasks

Comments

@kjozwiak
Copy link
Member

kjozwiak commented Mar 19, 2018

Per release specialty tests

  • Missing Properties on Publisher Object Breaks Brave. (#13330)

Installer

  • Check that installer is close to the size of last release
  • Check signature: If OS Run spctl --assess --verbose /Applications/Brave.app/ and make sure it returns accepted. If Windows right click on the installer exe and go to Properties, go to the Digital Signatures tab and double click on the signature. Make sure it says "The digital signature is OK" in the popup window
  • Check Brave, muon, and libchromiumcontent version in about:brave and make sure it is EXACTLY as expected

Ledger

  • Verify wallet is auto created after enabling payments
  • Verify monthly budget and account balance shows correct BAT and USD value
  • Click on add funds and click on each currency and verify it shows wallet address and QR Code
  • Verify that Brave BAT wallet address can be copied
  • Verify adding funds via any of the currencies flows into BAT Wallet after specified amount of time
  • Verify adding funds to an existing wallet with amount, adjusts the BAT value appropriately
  • Change min visit and min time in advance setting and verify if the publisher list gets updated based on new setting
  • Visit nytimes.com for a few seconds and make sure it shows up in the Payments table
  • Check that disabling payments and enabling them again does not lose state
  • Upgrade from older version
    • Verify the wallet overlay is shown when wallet transition is happening upon upgrade
    • Verify transition overlay is shown post upgrade even if the payment is disabled before upgrade
    • Verify publishers list is not lost after upgrade when payment is disabled in the older version

Ledger Media (Please also ensure Twitch is working as well)

  • Visit any YouTube video in a normal/session tab and ensure the video publisher name is listed in ledger table
  • Visit any YouTube video in a private tab and ensure the video publisher name is not listed in ledger table
  • Visit any live YouTube video and ensure the time spent is shown under ledger table
  • Visit any embeded YouTube video and ensure the video publisher name is listed in ledger table
  • Ensure total time spent is correctly calculated for each publisher video
  • Ensure total time spent is correctly calculated when switching to YouTube video from an embeded video
  • Ensure YouTube publishers are not listed when Allow contributions to video is disabled in adavanced settings
  • Ensure existing YouTube publishers are not lost when Allow contributions to video is disabled in adavanced settings
  • Ensure YouTube publishers is listed but not included when auto-include is disabled
  • Update Advanced settings to different time/visit value and ensure YouTube videos are added to ledger table once criteria is met

Update tests

  • Test that updating using BRAVE_UPDATE_VERSION=0.8.3 env variable works correctly.
  • Test that using BRAVE_ENABLE_PREVIEW_UPDATES=TRUE env variable works and prompts for preview build updates.
@kjozwiak
Copy link
Member Author

We'll also need to double check and ensure that the reconcile/contribution process is working as expected.

@srirambv
Copy link
Collaborator

Update tests works as expected. Reconcile worked for me but the amount wasn't deducted but payment went through and generated payment history.

@srirambv
Copy link
Collaborator

Verified on a clean profile, reconcile worked properly and balance was deducted correctly. Verified on staging profile.

@kjozwiak
Copy link
Member Author

Verified last night that reconciling/contributions worked on production.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants