Skip to content

Commit

Permalink
perf tests: fix arg passing
Browse files Browse the repository at this point in the history
- now that we wrap the perf tests in a Rails wrapper, the outer `ARGV`
  needs to be passed to the inner one
- fix straggler that wasn't previously converted to use an iteration
  count
  • Loading branch information
fallwith committed Sep 15, 2023
1 parent d995111 commit 68cdb9b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion test/performance/script/runner
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,4 @@ require_relative '../rails_app/config/boot'
require 'rails/command'
require_relative '../lib/performance'

Rails::Command.invoke(:runner, %w[Performance::Runner.new.run_and_report])
Rails::Command.invoke(:runner, %w[Performance::Runner.new.run_and_report] + ARGV)
2 changes: 1 addition & 1 deletion test/performance/suites/agent_module.rb
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ class AgentModuleTest < Performance::TestCase
ITERATIONS = 50_000

def test_increment_metric_by_1
measure do
measure(ITERATIONS) do
NewRelic::Agent.increment_metric(METRIC)
end
end
Expand Down

0 comments on commit 68cdb9b

Please sign in to comment.