selftest/subunit: Remove open_tests argument from parse_results.
authorJelmer Vernooij <jelmer@samba.org>
Fri, 5 Jun 2009 14:36:10 +0000 (16:36 +0200)
committerJelmer Vernooij <jelmer@samba.org>
Thu, 11 Jun 2009 17:59:59 +0000 (19:59 +0200)
selftest/Subunit.pm
selftest/Subunit/Filter.pm
selftest/filter-subunit.pl
selftest/format-subunit.pl

index 71d65f0ee533ca7a0c6e4d2c9611b0c3f6655843..16aab73926242504a3153a8d332fa85f97250487 100644 (file)
@@ -23,14 +23,14 @@ require Exporter;
 
 use strict;
 
 
 use strict;
 
-sub parse_results($$$$)
+sub parse_results($$$)
 {
 {
-       my ($msg_ops, $statistics, $fh, $open_tests) = @_;
+       my ($msg_ops, $statistics, $fh) = @_;
        my $unexpected_ok = 0;
        my $expected_fail = 0;
        my $unexpected_fail = 0;
        my $unexpected_err = 0;
        my $unexpected_ok = 0;
        my $expected_fail = 0;
        my $unexpected_fail = 0;
        my $unexpected_err = 0;
-       my $orig_open_len = $#$open_tests;
+       my $open_tests = [];
 
        while(<$fh>) {
                if (/^test: (.+)\n/) {
 
        while(<$fh>) {
                if (/^test: (.+)\n/) {
@@ -103,7 +103,7 @@ sub parse_results($$$$)
                }
        }
 
                }
        }
 
-       while ($#$open_tests > $orig_open_len) {
+       while ($#$open_tests+1 > 0) {
                $msg_ops->end_test(pop(@$open_tests), "error", 1,
                                   "was started but never finished!");
                $statistics->{TESTS_ERROR}++;
                $msg_ops->end_test(pop(@$open_tests), "error", 1,
                                   "was started but never finished!");
                $statistics->{TESTS_ERROR}++;
index 799b5dd7fb20ecde95ded6c2d0705572748118b8..3a9e4f9409535dc9386d489acb775073cf12b3e8 100644 (file)
@@ -40,17 +40,6 @@ sub find_in_list($$)
        return undef;
 }
 
        return undef;
 }
 
-my $statistics = {
-       SUITES_FAIL => 0,
-
-       TESTS_UNEXPECTED_OK => 0,
-       TESTS_EXPECTED_OK => 0,
-       TESTS_UNEXPECTED_FAIL => 0,
-       TESTS_EXPECTED_FAIL => 0,
-       TESTS_ERROR => 0,
-       TESTS_SKIP => 0,
-};
-
 sub control_msg()
 {
        # We regenerate control messages, so ignore this
 sub control_msg()
 {
        # We regenerate control messages, so ignore this
index cbc078765e43620e2367527e289163e4ea2219c1..912070a0d4e5a29b5417dbdd4d99782f5d2a04de 100755 (executable)
@@ -93,6 +93,6 @@ my $statistics = {
 
 my $msg_ops = new Subunit::Filter($opt_prefix, \@expected_failures);
 
 
 my $msg_ops = new Subunit::Filter($opt_prefix, \@expected_failures);
 
-parse_results($msg_ops, $statistics, *STDIN, []);
+parse_results($msg_ops, $statistics, *STDIN);
 
 0;
 
 0;
index acaac697aa0e00792ddd23f8341ac22e8f9d9f2d..77e4b34f3820366df698a767359cdde58ec83769 100755 (executable)
@@ -92,7 +92,7 @@ if ($opt_format eq "buildfarm") {
        die("Invalid output format '$opt_format'");
 }
 
        die("Invalid output format '$opt_format'");
 }
 
-my $expected_ret = parse_results($msg_ops, $statistics, *STDIN, []);
+my $expected_ret = parse_results($msg_ops, $statistics, *STDIN);
 
 $msg_ops->summary();
 
 
 $msg_ops->summary();