Merge pull request #22264 from Fryguy/fix_ruby3_issue_in_ansible_runner

Author: jrafanie

lib/ansible/runner.rb

@@ -195,7 +195,7 @@ def run_via_cli(hosts, credentials, env_vars, extra_vars, tags: nil, ansible_run
 
         validate_params!(env_vars, extra_vars, tags, ansible_runner_method, playbook_or_role_args)
 
-        base_dir = Pathname.new(Dir.mktmpdir("ansible-runner"))
+        base_dir = Pathname.new(Dir.mktmpdir("ansible-runner")).realpath
         debug    = verbosity.to_i >= 5 || env_vars["ANSIBLE_KEEP_REMOTE_FILES"]
 
         cred_command_line, cred_env_vars, cred_extra_vars = credentials_info(credentials, base_dir)

lib/ansible/runner/response_async.rb

@@ -54,10 +54,11 @@ def dump
 
       # Creates the Ansible::Runner::ResponseAsync object from hash data
       #
-      # @param [Hash] Dumped Ansible::Runner::ResponseAsync object
+      # @param hash [Hash] Dumped Ansible::Runner::ResponseAsync object
       # @return [Ansible::Runner::ResponseAsync] Ansible::Runner::ResponseAsync Object created from hash data
-      def self.load(kwargs)
-        new(kwargs)
+      def self.load(hash)
+        # Dump dumps a hash and load accepts a hash, so we must expand the hash to kwargs as new expects kwargs
+        new(**hash)
       end
     end
   end

spec/lib/ansible/runner/response_async_spec.rb

@@ -0,0 +1,18 @@
+RSpec.describe Ansible::Runner::ResponseAsync do
+  subject { described_class.new(:base_dir => "/path/to/results") }
+
+  it "#dump" do
+    expect(subject.dump).to eq(
+      :base_dir => "/path/to/results",
+      :debug    => false,
+      :ident    => "result"
+    )
+  end
+
+  it ".load" do
+    response = described_class.load(subject.dump)
+
+    expect(response).to be_a(described_class)
+    expect(response.base_dir).to eq("/path/to/results")
+  end
+end