On Thu, 15 May 2014 15:39:45 +0900 Yuichi Ito <[email protected]> wrote:
> On Wed, 14 May 2014 21:48:18 +0900 > Chiu,Cheng-Han <[email protected]> wrote: > >> From: "Chiu,Cheng-Han" <[email protected]> >> >> Signed-off-by: Chiu,Cheng-Han <[email protected]> >> --- >> ryu/tests/switch/tester.py | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py >> index a6734ce..499f8ff 100644 >> --- a/ryu/tests/switch/tester.py >> +++ b/ryu/tests/switch/tester.py >> @@ -480,7 +480,7 @@ class OfTester(app_manager.RyuApp): >> for result_type in sorted(report.keys()): >> test_descriptions = report[result_type] >> if result_type == TEST_OK: >> - ok_count = len(test_descriptions) >> + ok_count += len(test_descriptions) >> continue >> error_count += len(test_descriptions) >> self.logger.info('%s(%d)', result_type, len(test_descriptions)) >> > > Thank you for sending a patch. > > Well, I think there's no need to calculate ok_count in the loop. > (ok_count is assigned only when result_type is TEST_OK.) Applied, thanks again! ------------------------------------------------------------------------------ "Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE Instantly run your Selenium tests across 300+ browser/OS combos. Get unparalleled scalability from the best Selenium testing platform available Simple to use. Nothing to install. Get started now for free." http://p.sf.net/sfu/SauceLabs _______________________________________________ Ryu-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ryu-devel
