From 365e79dc8fdeeed324c3e6f4f20ce200f4d28c3c Mon Sep 17 00:00:00 2001 From: Tom Clegg Date: Tue, 31 Oct 2023 10:29:55 -0400 Subject: [PATCH] 20846: Fix kwargs variable name. Arvados-DCO-1.1-Signed-off-by: Tom Clegg --- services/api/test/test_helper.rb | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/services/api/test/test_helper.rb b/services/api/test/test_helper.rb index e907367446..0255d8907d 100644 --- a/services/api/test/test_helper.rb +++ b/services/api/test/test_helper.rb @@ -179,21 +179,21 @@ class ActionController::TestCase end [:get, :post, :put, :patch, :delete].each do |method| - define_method method do |action, **args| + define_method method do |action, **kwargs| check_counter action # After Rails 5.0 upgrade, some params don't get properly serialized. # One case are filters: [['attr', 'op', 'val']] become [['attr'], ['op'], ['val']] # if not passed upstream as a JSON string. - if args[:params].is_a?(Hash) - args[:params].each do |key, _| + if kwargs[:params].is_a?(Hash) + kwargs[:params].each do |key, _| next if key == :exclude_script_versions # Job Reuse tests # Keys could be: :filters, :where, etc - if [Array, Hash].include?(args[:params][key].class) - args[:params][key] = SafeJSON.dump(args[:params][key]) + if [Array, Hash].include?(kwargs[:params][key].class) + kwargs[:params][key] = SafeJSON.dump(kwargs[:params][key]) end end end - super action, **args + super action, **kwargs end end -- 2.30.2