fix: unused parameter

This commit is contained in:
HugoPBrito
2025-11-07 13:58:26 +00:00
parent b68097ebea
commit a97fb3d993
2 changed files with 5 additions and 5 deletions
@@ -1,5 +1,4 @@
import os
from typing import Optional
from prowler.lib.logger import logger
from prowler.lib.powershell.powershell import PowerShellSession
@@ -124,7 +123,7 @@ class M365PowerShell(PowerShellSession):
'$graphToken = Invoke-RestMethod -Uri "https://login.microsoftonline.com/$tenantID/oauth2/v2.0/token" -Method POST -Body $graphtokenBody | Select-Object -ExpandProperty Access_Token'
)
def execute_connect(self, command: str, timeout: Optional[int] = None) -> str:
def execute_connect(self, command: str) -> str:
"""
Execute a PowerShell connect command ensuring empty responses surface as timeouts.
@@ -566,7 +566,7 @@ class Testm365PowerShell:
)
session = M365PowerShell(credentials, identity)
session.execute = MagicMock(return_value=None)
session.execute = MagicMock(side_effect=[None, ""])
session.execute_connect = MagicMock(return_value="")
result = session.test_teams_connection()
@@ -598,8 +598,8 @@ class Testm365PowerShell:
)
session = M365PowerShell(credentials, identity)
session.execute = MagicMock(return_value=None)
session.execute_connect = MagicMock(return_value="Permission denied")
session.execute = MagicMock(side_effect=[None, "Permission denied"])
session.execute_connect = MagicMock()
with patch("prowler.lib.logger.logger.error") as mock_error:
result = session.test_teams_connection()
@@ -608,6 +608,7 @@ class Testm365PowerShell:
mock_error.assert_called_once_with(
"Microsoft Teams connection failed: Permission denied"
)
session.execute_connect.assert_not_called()
session.close()
@patch("subprocess.Popen")