> On May 23, 2013, 6:15 a.m., Prasanna Santhanam wrote: > > Can you create a bug report for this? Otherwise the patch LGTM. > > Ashutosh Kelkar wrote: > Created a bug report CLOUDSTACK-2703
Thanks, this will correct the TestCase logger when running tests using python -m marvin.deployAndRun. The nose test runner however doesn't duplicate the log messages. - Prasanna ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11340/#review20930 ----------------------------------------------------------- On May 28, 2013, 5:40 a.m., Ashutosh Kelkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11340/ > ----------------------------------------------------------- > > (Updated May 28, 2013, 5:40 a.m.) > > > Review request for cloudstack, Prasanna Santhanam and Girish Shilamkar. > > > Description > ------- > > In unit tests runs the same log output line is repeated for number tests in a > class, making log unreadable. > This is because we are creating a logger by class name for each test load in > loadTestsFromFile() > > with this fix now logger will log the test case and class name. > > > This addresses bug CLOUDSTACK-2703. > > > Diffs > ----- > > tools/marvin/marvin/TestCaseExecuteEngine.py 5743868 > > Diff: https://reviews.apache.org/r/11340/diff/ > > > Testing > ------- > > Testing Done. > > > Thanks, > > Ashutosh Kelkar > >
