Add ability to add multiple hosts
See original GitHub issueThe Problem
I have some tests that run across multiple environments and I would like to codify that into the test. Initially I’ve left of the ForHost
declaration but that introduces potential issues with it running against any service that matches the other builder routes configured.
Proposed Solution I’m just wondering if there’s been thought/discussions around adding the following overload to enable you to configure multiple hosts:
ForHost(params string[] host)
Alternatives
One alternative I was considering (which I feel could be a better option) is a regular expression overload as specifying multiple hosts feels a bit inconsistent with the idea of mocking out an HTTP endpoint (as opposed to many, which host implies).
Would be happy to contribute the PR if this is something you feel is valuable outside of my single use case.
Issue Analytics
- State:
- Created 4 years ago
- Reactions:1
- Comments:15 (13 by maintainers)
Top GitHub Comments
Yeah, I’m sorry but I’ve still not had a chance to finish it off. It’s almost there but I wouldn’t want to hold 2.0.2 up.
@slang25 Yes, you’re right. That’s the “god mode” I was thinking of.