Fix tests

tests/unit/plugins/action/network/test_network.py

@@ -63,8 +63,9 @@ def test_backup_options(plugin, backup_dir, backup_file, role_path):
 
  # This doesn't need to be conditional, but doing so tests the equivalent
  # `if backup_options:` in the action plugin itself.
+ backup_options = None
  if backup_dir or backup_file:
- plugin._task.args["backup_options"] = {
+ backup_options = {
  "dir_path": backup_dir,
  "filename": backup_file,
  }
@@ -79,7 +80,7 @@ def test_backup_options(plugin, backup_dir, backup_file, role_path):
 
  try:
  # result is updated in place, nothing is returned
- plugin._handle_backup_option(result, task_vars)
+ plugin._handle_backup_option(result, task_vars, backup_options)
  assert not result.get("failed")
 
  with open(result["backup_path"]) as backup_file_obj:
@@ -108,7 +109,7 @@ def test_backup_options(plugin, backup_dir, backup_file, role_path):
  if backup_file:
  # check for idempotency
  result = {"__backup__": content}
- plugin._handle_backup_option(result, task_vars)
+ plugin._handle_backup_option(result, task_vars, backup_options)
  assert not result.get("failed")
  assert result["changed"] is False
 
@@ -123,21 +124,19 @@ def test_backup_no_content(plugin):
  with pytest.raises(
  AnsibleError, match="Failed while reading configuration backup"
  ):
- plugin._handle_backup_option(result, task_vars)
+ plugin._handle_backup_option(result, task_vars, backup_options=None)
 
 
 def test_backup_options_error(plugin):
  result = {"__backup__": ""}
  task_vars = {}
 
  with tempfile.NamedTemporaryFile() as existing_file:
- plugin._task.args = {
- "backup_options": {
- "dir_path": existing_file.name,
- "filename": "backup_file",
- }
+ backup_options = {
+ "dir_path": existing_file.name,
+ "filename": "backup_file",
  }
- plugin._handle_backup_option(result, task_vars)
+ plugin._handle_backup_option(result, task_vars, backup_options)
 
  assert result["failed"] is True
  assert result["msg"] == (