Okay, So I did a refactoring of the Default GET servlet bundle a while back and 
missed validating against the integration tests.

As I'm working through issues that popped up, I have one where the test was 
failing because a boolean value was being accessed as a String, and that no 
longer worked, because the boolean value is now a bool. So my knee jerk 
reaction to integration tests is that they are the truth, and that I shouldn't 
be changing an integration test to fix something I did. However, this seems to 
be clearly a case where the integration test should not have been checking for 
a String. 

I changed the integration test. But I wanted to bring this up for awareness in 
the case that one of you is going "no --- that will break x,y, and z"

- Jason

Reply via email to