On Tue, Nov 12, 2013 at 1:49 PM, Andrew Grieve <agri...@chromium.org> wrote: > Want to give a big +1 to deleting tests that are testing implementation > rather than results. Having such tests is a real drain on productivity.
I tend to agree with this statement. Maybe not delete but add/replace them real world assertions to them (that touch the FS). In some cases it makes sense to test the implementation to make sure that an action goes through the proper flow.