gabbi icon indicating copy to clipboard operation
gabbi copied to clipboard

add wsgi_intercept.STRICT_RESPONSE_HEADERS as a flag

Open cdent opened this issue 9 years ago • 2 comments

Add a flag to build_tests that allows turning wsgi_intercept.STRICT_RESPONSE_HEADERS on and in the wsgi intercept context manager fixture.

The challenge here is getting the flag value to the fixture.

Also the number of parameters on build_tests is getting absurd. It's time to explore some new models.

/cc @jd

cdent avatar Oct 02 '16 13:10 cdent

What struck me when I took a look at that, is more the global variable that is provided by wsgi_intercept. That might be the first thing to change.

jd avatar Oct 03 '16 08:10 jd

@FND and I talked about that in IRC some time last week and it was basically what I said to you: The use of a global is because of the multiple entry points into wsgi-intercept's big mess, which will be a large effort to fix.

We can hide that complexity in gabbi (without regard to the implementation being what it is now or how it changes later) now, with something like this flag.

cdent avatar Oct 03 '16 09:10 cdent