Back out --fast mode changes that leaked in, handle defects correctly

in testrail-results

bin/testrail-results

@@ -133,8 +133,6 @@ sub run {
         'd|defect=s@'     => \$opts->{'defects'},
         'v|version=s@'    => \$opts->{'versions'},
         'c|cachefile=s'   => \$opts->{'cachefile'},
-        'f|fast'          => \$opts->{'fast'},
-        't|testsuite'     => \$opts->{'testsuite'},
         'json'            => \$opts->{'json'},
         'h|help'          => \$opts->{'help'},
     );
@@ -167,12 +165,6 @@ sub run {
         $opts->{'plan_ids'} = $prior_plans;
     }
 
-    if ($opts->{'fast'}) {
-        my $project = $tr->getProjectByName($opts->{project});
-        $opts->{testsuite} = $tr->getTestsuiteByName($project->{id},$opts->{testsuite});
-        confess("Could not determine the relevant testsuite!") unless ref($opts->{testsuite}) eq 'HASH';
-    }
-
     my ($res,$seen_plans) = TestRail::Utils::Find::getResults($tr,$opts,$prior_runs,@{$params{'args'}});
 
     #Make sure subsequent runs keep ignoring the prior plans
@@ -203,7 +195,7 @@ sub run {
                 $casetotals->{$result->{status_id}}++;
                 $versions_by_status->{$result->{status_id}} //= [];
                 push(@{$versions_by_status->{$result->{status_id}}},$result->{version}) if $result->{version};
-                push(@$defects, @{$result->{defects}}) if $result->{defects};
+                push(@$defects, $result->{defects}) if $result->{defects};
                 push(@$elapsetotals,_elapsed2secs($result->{'elapsed'}));
             }
         }

lib/TestRail/Utils/Find.pm

@@ -345,8 +345,6 @@ Valid Options:
 
 =item B<defects> - ARRAYREF of defects of which at least one must be present in a result.
 
-=item B<fast> - Whether to get only the latest result from the test in your run(s).  This can significantly speed up operations.
-
 =back
 
 =cut