Fixes some failing tests #9639

NejcZdovc · 2017-06-21T11:07:44Z

Submitter Checklist:

Submitted a ticket for my issue if one did not already exist.
Used Github auto-closing keywords in the commit message.
Added/updated tests for this change (for new code or code which already has tests).
Ran git rebase -i to squash commits (if needed).
Tagged reviewers and labelled the pull request as needed.

Resolves #9638

Test Plan:

Reviewer Checklist:

Tests

Adequate test coverage exists to prevent regressions
Tests should be independent and work correctly when run individually or as a suite ref
New files have MPL2 license header

@bsclifton

Resolves brave#9638 Auditors: @bsclifton Test Plan: - tests should be green

bsclifton

++ 😄

NejcZdovc added the automated-tests label Jun 21, 2017

NejcZdovc added this to the 0.19.x (Nightly Channel) milestone Jun 21, 2017

NejcZdovc self-assigned this Jun 21, 2017

NejcZdovc force-pushed the hotfix/#9638-tests branch from d710f44 to f3832f9 Compare June 21, 2017 12:15

NejcZdovc added the PR/work-in-progress ⚒ label Jun 21, 2017

Fixes some failing tests

838b44e

Resolves brave#9638 Auditors: @bsclifton Test Plan: - tests should be green

NejcZdovc force-pushed the hotfix/#9638-tests branch from f3832f9 to 838b44e Compare June 23, 2017 10:50

NejcZdovc mentioned this pull request Jun 23, 2017

Local failing tests #9638

Closed

NejcZdovc removed the PR/work-in-progress ⚒ label Jun 23, 2017

NejcZdovc merged commit 1102e38 into brave:master Jun 23, 2017

luixxiul requested a review from bsclifton June 23, 2017 14:09

bsclifton reviewed Jun 23, 2017

View reviewed changes

Provide feedback