Nice branch. This is a very good second step in order to reduce the amount of boilerplate we have everywhere in our tests.
The only bit I don't like is the many "if self.ssl_settings" scattered around. What about splitting the WebServer class into HTTPS and HTTP classes?
« Back to merge proposal
Nice branch. This is a very good second step in order to reduce the amount of boilerplate we have everywhere in our tests.
The only bit I don't like is the many "if self.ssl_settings" scattered around.
What about splitting the WebServer class into HTTPS and HTTP classes?