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

Commit

Permalink
Fix failing autoplay tests
Browse files Browse the repository at this point in the history
Auditors: @darkdh

The default was changed here:
c5f4409

So this makes the tests more robust to default changes.
  • Loading branch information
bbondy committed May 18, 2017
1 parent 3a307f4 commit 907f2a7
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions test/bravery-components/notificationBarTest.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
const Brave = require('../lib/brave')
const {notificationBar, titleBar, urlInput, reloadButton} = require('../lib/selectors')
const {autoplayOption} = require('../../app/common/constants/settingsEnums')
const {AUTOPLAY_MEDIA} = require('../../js/constants/settings')

describe('notificationBar permissions', function () {
function * setup (client) {
Expand Down Expand Up @@ -220,6 +221,7 @@ describe('Autoplay test', function () {
yield client
.waitForBrowserWindow()
.waitForVisible(urlInput)
.changeSetting(AUTOPLAY_MEDIA, autoplayOption.ALWAYS_ASK)
}

Brave.beforeEach(this)
Expand Down

1 comment on commit 907f2a7

@darkdh
Copy link
Member

@darkdh darkdh commented on 907f2a7 May 19, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

++

Please sign in to comment.