Skip to content

Commit a615482

Browse files
committed
fix: tests
1 parent 7342d22 commit a615482

2 files changed

Lines changed: 7 additions & 13 deletions

File tree

tests/test_serverless/test_modules/test_logger.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -127,9 +127,9 @@ def test_log_job_id(self):
127127
# Test with endpoint id set
128128
os.environ["RUNPOD_ENDPOINT_ID"] = "test_endpoint_id"
129129
logger.log("test_message", "INFO", job_id)
130-
os.environ.pop("RUNPOD_ENDPOINT_ID")
131130

132-
mock_print.assert_called_once_with(
131+
mock_print.assert_called_with(
133132
'{"requestId": "test_job_id", "message": "test_message", "level": "INFO"}',
134133
flush=True
135134
)
135+
os.environ.pop("RUNPOD_ENDPOINT_ID")

tests/test_serverless/test_worker.py

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
import os
55
import platform
66
import argparse
7+
from unittest import mock
78
from unittest.mock import patch, mock_open, Mock, MagicMock
89

910
from unittest import IsolatedAsyncioTestCase
@@ -18,24 +19,17 @@
1819

1920

2021
class TestWorker(IsolatedAsyncioTestCase):
21-
""" Tests for runpod | serverless| worker """
22+
"""Tests for RunPod serverless worker."""
2223

2324
def setUp(self):
24-
self.mock_handler = Mock()
25-
self.mock_handler.return_value = "test"
26-
27-
self.mock_config = Mock()
25+
self.mock_handler = mock.Mock(return_value="test")
2826
self.mock_config = {
2927
"handler": self.mock_handler,
30-
"rp_args": {
31-
"test_input": None,
32-
}
28+
"rp_args": {"test_input": None},
3329
}
3430

3531
def test_get_auth_header(self):
36-
'''
37-
Test _get_auth_header
38-
'''
32+
"""Test retrieval of the authentication header from _get_auth_header."""
3933
os_info = f"{platform.system()} {platform.release()}; {platform.machine()}"
4034
with patch("runpod.serverless.worker.os") as mock_os:
4135
mock_os.environ.get.return_value = "test"

0 commit comments

Comments
 (0)