made refactoring and added test for browser_instance fixture

Made changes related to https://github.com/paylogic/pytest-splinter/issues/5
let's rename the fixture to reflect what it returns
also i think we should make it session scoped, as patching happens there and we return a function, so it's safe and efficient to make it session scoped