Hi all, >This is a bit of a social norms thread.... > >I've been consistently asking for tests in reviews for a while now, and I get >the >occasional push-back. I think this falls into a few broad camps: > >A - there is no test suite at all, adding one in unreasonable B - this thing >cannot >be tested in this context (e.g. functional tests are defined in a different >tree) >C - this particular thing is very hard to test D - testing this won't offer >benefit E >- other things like this in the project don't have tests F - submitter doesn't >know how to write tests G - submitter doesn't have time to write tests > >Now, of these, I think it's fine not add tests in cases A, B, C in combination >with D, and D.
In my opinion C instead of being an acceptable reason for not having tests is a symptom of one of the two things: 1) F => Submitter doesn't know how to write tests, in this case someone else can help with suggestions 2) The code we are trying to test is too complicated so it's time to refactor it And about D, In my opinion tests always offer benefits, like code coverage or helping in understanding the code. What do you think? Regards -- Andrea Rosa _______________________________________________ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev