----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3931/#review13187 -----------------------------------------------------------
Ship it! I think the output is far from friendly right now, but it's a start. Please add this (or something like it) though, so we won't think the test passed by accident: @@ -381,6 +384,8 @@ ts.setAttribute("time", "%.2f" % self.total_time) ts.setAttribute("failures", str(self.total_failures)) ts.setAttribute("name", "AsteriskTestSuite") + if self.options.dry_run: + ts.setAttribute("dry-run", str(self.total_count)) for t in self.tests: if t.did_run is False: - wdoekes On Aug. 26, 2014, 4:50 p.m., Tzafrir Cohen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3931/ > ----------------------------------------------------------- > > (Updated Aug. 26, 2014, 4:50 p.m.) > > > Review request for Asterisk Developers. > > > Repository: testsuite > > > Description > ------- > > Adds option -d (--dry-run) to runtests.py to just list the tests that will be > run and not actually run them. > > > Diffs > ----- > > /asterisk/trunk/runtests.py 5520 > > Diff: https://reviewboard.asterisk.org/r/3931/diff/ > > > Testing > ------- > > > Thanks, > > Tzafrir Cohen > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev