Skip to content

Commit

Permalink
Use correct tabByIndex in autofill tests
Browse files Browse the repository at this point in the history
Auditors: @bsclifton

This probably broke because we weren't finding about prefrences tabs correctly before in tabByIndex calls.  Since we fixed that, we need to fix wrong callers using wrong a wrong index. There might be follow up similar things that need fixing
  • Loading branch information
bbondy committed Nov 5, 2016
1 parent 6fb5c34 commit 951a127
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/components/autofillTest.js
Original file line number Diff line number Diff line change
Expand Up @@ -435,12 +435,12 @@ describe('Autofill', function () {
.click('[name="04fullname"]')
.windowByUrl(Brave.browserWindowUrl)
.waitForExist('contextMenuItemText', 500, true)
.tabByIndex(0)
.tabByIndex(1)
.getValue('[name="04fullname"]').should.eventually.be.equal('')
.click('[name="41ccnumber"]')
.click('[name="41ccnumber"]')
.waitForExist('contextMenuItemText', 500, true)
.tabByIndex(0)
.tabByIndex(1)
.getValue('[name="41ccnumber"]').should.eventually.be.equal('')
})
})
Expand Down Expand Up @@ -609,7 +609,7 @@ describe('Autofill', function () {
.click('[name="04fullname"]')
.windowByUrl(Brave.browserWindowUrl)
.waitForExist('contextMenuItemText', 500, true)
.tabByIndex(0)
.tabByIndex(1)
.getValue('[name="04fullname"]').should.eventually.be.equal('')
})
})
Expand Down

0 comments on commit 951a127

Please sign in to comment.