Make redirect_true default to True

salt/modules/cmdmod.py

@@ -1053,7 +1053,7 @@ def run(
     ignore_retcode=False,
     saltenv=None,
     use_vt=False,
-    redirect_stderr=False,
+    redirect_stderr=True,
     bg=False,
     password=None,
     encoded_cmd=False,
@@ -1191,7 +1191,7 @@ def run(
 
     :param bool redirect_stderr: If set to ``True``, then stderr will be
         redirected to stdout. This is helpful for cases where obtaining both
-        the retcode and output is desired.
+        the retcode and output is desired. Default is ``True``
 
         .. versionadded:: 3006.9
 

tests/pytests/functional/modules/cmd/test_powershell.py

@@ -178,7 +178,7 @@ def test_cmd_run_encoded_cmd(shell, cmd, expected, encoded_cmd):
     """
     Ensure that cmd.run supports running shell='powershell'
     """
-    ret = cmdmod.run(cmd=cmd, shell=shell, encoded_cmd=encoded_cmd)
+    ret = cmdmod.run(cmd=cmd, shell=shell, encoded_cmd=encoded_cmd, redirect_stderr=False)
     assert ret == expected
 
 

tests/pytests/functional/modules/test_win_useradd.py

@@ -320,13 +320,13 @@ def test_setpassword_int(user, account_int):
         ("logonscript", "\\\\server\\script.cmd", "", None),
         ("expiration_date", "3/19/2024", "", "2024-03-19 00:00:00"),
         ("expiration_date", "Never", "", None),
-        ("expired", True, "", None),
-        ("expired", False, "", None),
         ("account_disabled", True, "", None),
         ("account_disabled", False, "", None),
         ("unlock_account", True, "account_locked", False),
         ("password_never_expires", True, "", None),
         ("password_never_expires", False, "", None),
+        ("expired", True, "", None),
+        ("expired", False, "", None),
         ("disallow_change_password", True, "", None),
         ("disallow_change_password", False, "", None),
     ],