> On Dec. 16, 2012, 9:27 p.m., Yuri Zelikov wrote: > > LGTM > > Committed as r1422694
Thanks for the patch! Can you please close the review request? - Yuri ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8254/#review14566 ----------------------------------------------------------- On Nov. 29, 2012, 6:49 p.m., Nikolay Volnov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8254/ > ----------------------------------------------------------- > > (Updated Nov. 29, 2012, 6:49 p.m.) > > > Review request for wave. > > > Description > ------- > > Hi, > > I've started exporing a wave project recently and found that 3 tests in > AttachmentStoreTest failed when I try to run it localy (OS: window 7). It > could not delete a tempory directory in tearDown method > (AttachmentStoreTest:49). Some input streams weren't closed in tests and it > blocked deleting the folder. > > > Diffs > ----- > > test/org/waveprotocol/box/server/persistence/AttachmentStoreTestBase.java > b5aae36 > > Diff: https://reviews.apache.org/r/8254/diff/ > > > Testing > ------- > > run all tests, status: PASSED > > > Thanks, > > Nikolay Volnov > >
