Skip to content

Commit d02855a

Browse files
committed
lint and move self.response=None into try block
1 parent d60572b commit d02855a

File tree

2 files changed

+5
-9
lines changed

2 files changed

+5
-9
lines changed

datadog_lambda/wrapper.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -153,9 +153,8 @@ def __call__(self, event, context, **kwargs):
153153
self._after(event, context)
154154

155155
def _before(self, event, context):
156-
self.response = None
157156
try:
158-
157+
self.response = None
159158
set_cold_start()
160159
submit_invocations_metric(context)
161160
self.trigger_tags = extract_trigger_tags(event, context)

tests/test_wrapper.py

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -275,22 +275,19 @@ def lambda_handler(event, context):
275275
]
276276
)
277277

278-
@patch('datadog_lambda.wrapper.extract_trigger_tags')
278+
@patch("datadog_lambda.wrapper.extract_trigger_tags")
279279
def test_5xx_sends_errors_metric_and_set_tags(self, mock_extract_trigger_tags):
280280
mock_extract_trigger_tags.return_value = {
281281
"function_trigger.event_source": "api-gateway",
282-
"function_trigger.event_source_arn":
283-
"arn:aws:apigateway:us-west-1::/restapis/1234567890/stages/prod",
282+
"function_trigger.event_source_arn": "arn:aws:apigateway:us-west-1::/restapis/1234567890/stages/prod",
284283
"http.url": "70ixmpl4fl.execute-api.us-east-2.amazonaws.com",
285284
"http.url_details.path": "/prod/path/to/resource",
286285
"http.method": "GET",
287286
}
287+
288288
@datadog_lambda_wrapper
289289
def lambda_handler(event, context):
290-
return {
291-
"statusCode": 500,
292-
"body": "fake response body"
293-
}
290+
return {"statusCode": 500, "body": "fake response body"}
294291

295292
lambda_event = {}
296293

0 commit comments

Comments
 (0)