Skip to content

Commit b890306

Browse files
committed
use @patch and reformat
1 parent 0ab4dc1 commit b890306

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

datadog_lambda/tracing.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -344,12 +344,14 @@ def extract_context_custom_extractor(extractor, event, lambda_context):
344344

345345
return None, None, None
346346

347+
347348
def is_authorizer_response(response) -> bool:
348349
try:
349350
return response["principalId"] and response["policyDocument"]
350351
except:
351352
return False
352353

354+
353355
def get_injected_authorizer_data(event, is_http_api) -> dict:
354356
try:
355357
authorizer_headers = event.get("requestContext", {}).get("authorizer")

tests/test_wrapper.py

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -545,10 +545,9 @@ def lambda_handler(event, context):
545545
self.assertEquals(inject_data[TraceHeader.SAMPLING_PRIORITY], "1")
546546
self.assertEquals(result["context"]["scope"], "still here")
547547

548-
def test_different_return_type_no_error(self):
549-
TEST_RESULTS = ["a str to return", 42, {'value': 42}, ['A', 42]]
550-
import traceback as _importedTraceback
551-
_importedTraceback.print_exc = MagicMock()
548+
@patch("traceback.print_exc")
549+
def test_different_return_type_no_error(self, MockPrintExc):
550+
TEST_RESULTS = ["a str to return", 42, {"value": 42}, ["A", 42]]
552551
mock_context = get_mock_context()
553552
for test_result in TEST_RESULTS:
554553

@@ -558,5 +557,4 @@ def return_a_str(event, context):
558557

559558
result = return_a_str({}, mock_context)
560559
self.assertEquals(result, test_result)
561-
_importedTraceback.print_exc.assert_not_called()
562-
560+
self.assertFalse(MockPrintExc.called)

0 commit comments

Comments
 (0)