The committers listed above are authorized under a signed CLA.

• ✅ login: therealbrad / name: Brad DerManouelian (98bbbc8, 4b020c9)

@therealbrad thanks for taking a stab at this. Let's make sure we add proper unit tests so we don't run into any regressions moving on. There are already existing unit tests for this case. Also we need you to have the CLA signed. Thank you!

/easycla

@therealbrad there seems to be an failure in the test?

@christian-bromann My bad. I meant to remove that test because I didn't see a way to run unit tests and set browser.options.hostname to null. Is there a way to do that?

I thought browser.options.hostname = 'local' would do the trick because it is the same value as this.instance if not we should track down where this.instance is defined and find a way to modify that value for our test purposes. You are always free to write an individual unit test for the action class itself if it makes writing the test easier.

@christian-bromann I changed it to explicitly check if running locally via hostname instead of assuming it. Seems to work but I don't know if this breaks anything else.

Thank you @therealbrad