Wow... I meant
"replace them with real world assertions (that touch the FS)". :-) On Tue, Nov 12, 2013 at 1:55 PM, Anis KADRI <anis.ka...@gmail.com> wrote: > 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.