Fixes middle click not working for home button

[NejcZdovc]

Test Plan:

• middle click on Home button
• new tab is opened

Description

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 #9562

Auditors: @bsclifton

Reviewer Checklist:

• 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]

Code works good; I tested with single and multiple (separated with |) and it worked as expected. You can of course also use cmd + click

However, the test is failing for me locally (reproducible each time):
npm run test -- --grep="opens home page in a new tab when middle mouse button is clicked"

It looks OK on travis:
https://travis-ci.org/brave/browser-laptop/jobs/244416657#L3093

[NejcZdovc]

@bsclifton thing is that this whole block is connected so you need to run the whole block (npm run test -- --grep="home button when enabled") otherwise number of open tabs will be wrong. I added this test into this existing one. What we can do is do beforeEach instead of before and adjust tests accordingly.

[bsclifton]

Re-ran tests (thanks for the heads up that they're part of a block). Changes LGTM 😄 👍

[bsclifton]

Ideal situation would be each test has what it needs (ex: get # of tabs, do a +1, then expect that number... or restructure to before each as mentioned above). For now, this totally works though 😄 Merging!