Skip to content

Commit

Permalink
run-regression-test: fix to stop the consumer on stop-on-failure mode
Browse files Browse the repository at this point in the history
  • Loading branch information
komainu8 committed Nov 7, 2018
1 parent b27811b commit 335653c
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions lib/groonga-query-log/server-verifier.rb
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ def run_consumers
@options.n_clients.times.collect do
Thread.new do
loop do
break if run_consumer
break if run_consumer_stop?
end
end
end
Expand Down Expand Up @@ -136,10 +136,14 @@ def target_command?(command)
@options.target_command_name?(command.command_name)
end

def different_or_error_results?
def different_or_error_results?
@same and !@client_error_is_occurred
end

def run_consumer_stop?
run_consumer or @options.stop_on_failure?
end

def verify_command(groonga1_client, groonga2_client, command)
command["cache"] = "no" if @options.disable_cache?
command["output_type"] = "json"
Expand Down

0 comments on commit 335653c

Please sign in to comment.