ref(tests): Remove debug=True from tests (#2934)

Author: sentrivana

tests/integrations/asgi/test_asgi.py

@@ -322,7 +322,7 @@ async def test_trace_from_headers_if_performance_disabled(
 
 @pytest.mark.asyncio
 async def test_websocket(sentry_init, asgi3_ws_app, capture_events, request):
-    sentry_init(debug=True, send_default_pii=True)
+    sentry_init(send_default_pii=True)
 
     events = capture_events()
 
@@ -612,7 +612,6 @@ async def test_transaction_name(
     """
     sentry_init(
         traces_sample_rate=1.0,
-        debug=True,
     )
 
     envelopes = capture_envelopes()
@@ -674,7 +673,6 @@ def dummy_traces_sampler(sampling_context):
     sentry_init(
         traces_sampler=dummy_traces_sampler,
         traces_sample_rate=1.0,
-        debug=True,
     )
 
     app = SentryAsgiMiddleware(asgi3_app, transaction_style=transaction_style)

tests/integrations/asyncio/test_asyncio.py

@@ -67,7 +67,6 @@ async def test_create_task(
     sentry_init(
         traces_sample_rate=1.0,
         send_default_pii=True,
-        debug=True,
         integrations=[
             AsyncioIntegration(),
         ],
@@ -111,7 +110,6 @@ async def test_gather(
     sentry_init(
         traces_sample_rate=1.0,
         send_default_pii=True,
-        debug=True,
         integrations=[
             AsyncioIntegration(),
         ],
@@ -155,7 +153,6 @@ async def test_exception(
     sentry_init(
         traces_sample_rate=1.0,
         send_default_pii=True,
-        debug=True,
         integrations=[
             AsyncioIntegration(),
         ],

tests/integrations/celery/test_celery.py

@@ -361,7 +361,7 @@ def dummy_task(self):
 )
 @pytest.mark.forked
 def test_redis_backend_trace_propagation(init_celery, capture_events_forksafe):
-    celery = init_celery(traces_sample_rate=1.0, backend="redis", debug=True)
+    celery = init_celery(traces_sample_rate=1.0, backend="redis")
 
     events = capture_events_forksafe()
 

tests/integrations/falcon/test_falcon.py

@@ -112,7 +112,7 @@ def test_transaction_style(
 
 
 def test_unhandled_errors(sentry_init, capture_exceptions, capture_events):
-    sentry_init(integrations=[FalconIntegration()], debug=True)
+    sentry_init(integrations=[FalconIntegration()])
 
     class Resource:
         def on_get(self, req, resp):
@@ -140,7 +140,7 @@ def on_get(self, req, resp):
 
 
 def test_raised_5xx_errors(sentry_init, capture_exceptions, capture_events):
-    sentry_init(integrations=[FalconIntegration()], debug=True)
+    sentry_init(integrations=[FalconIntegration()])
 
     class Resource:
         def on_get(self, req, resp):
@@ -164,7 +164,7 @@ def on_get(self, req, resp):
 
 
 def test_raised_4xx_errors(sentry_init, capture_exceptions, capture_events):
-    sentry_init(integrations=[FalconIntegration()], debug=True)
+    sentry_init(integrations=[FalconIntegration()])
 
     class Resource:
         def on_get(self, req, resp):
@@ -188,7 +188,7 @@ def test_http_status(sentry_init, capture_exceptions, capture_events):
     This just demonstrates, that if Falcon raises a HTTPStatus with code 500
     (instead of a HTTPError with code 500) Sentry will not capture it.
     """
-    sentry_init(integrations=[FalconIntegration()], debug=True)
+    sentry_init(integrations=[FalconIntegration()])
 
     class Resource:
         def on_get(self, req, resp):

tests/integrations/fastapi/test_fastapi.py

@@ -59,7 +59,6 @@ async def test_response(sentry_init, capture_events):
         integrations=[StarletteIntegration(), FastApiIntegration()],
         traces_sample_rate=1.0,
         send_default_pii=True,
-        debug=True,
     )
 
     app = fastapi_app_factory()
@@ -196,7 +195,6 @@ async def test_original_request_not_scrubbed(sentry_init, capture_events):
     sentry_init(
         integrations=[StarletteIntegration(), FastApiIntegration()],
         traces_sample_rate=1.0,
-        debug=True,
     )
 
     app = FastAPI()
@@ -354,7 +352,6 @@ def test_transaction_name(
             FastApiIntegration(transaction_style=transaction_style),
         ],
         traces_sample_rate=1.0,
-        debug=True,
     )
 
     envelopes = capture_envelopes()
@@ -384,7 +381,6 @@ def test_route_endpoint_equal_dependant_call(sentry_init):
             FastApiIntegration(),
         ],
         traces_sample_rate=1.0,
-        debug=True,
     )
 
     app = fastapi_app_factory()
@@ -438,7 +434,6 @@ def dummy_traces_sampler(sampling_context):
         integrations=[StarletteIntegration(transaction_style=transaction_style)],
         traces_sampler=dummy_traces_sampler,
         traces_sample_rate=1.0,
-        debug=True,
     )
 
     app = fastapi_app_factory()
@@ -482,7 +477,6 @@ def test_transaction_name_in_middleware(
             FastApiIntegration(transaction_style=transaction_style),
         ],
         traces_sample_rate=1.0,
-        debug=True,
     )
 
     envelopes = capture_envelopes()

tests/integrations/flask/test_flask.py

@@ -125,7 +125,7 @@ def test_errors(
     testing,
     integration_enabled_params,
 ):
-    sentry_init(debug=True, **integration_enabled_params)
+    sentry_init(**integration_enabled_params)
 
     app.debug = debug
     app.testing = testing

tests/integrations/huey/test_huey.py

@@ -20,7 +20,6 @@ def inner():
             integrations=[HueyIntegration()],
             traces_sample_rate=1.0,
             send_default_pii=True,
-            debug=True,
         )
 
         return MemoryHuey(name="sentry_sdk")

tests/integrations/quart/test_quart.py

@@ -129,7 +129,7 @@ async def test_errors(
     app,
     integration_enabled_params,
 ):
-    sentry_init(debug=True, **integration_enabled_params)
+    sentry_init(**integration_enabled_params)
 
     @app.route("/")
     async def index():

tests/integrations/starlette/test_starlette.py

@@ -954,7 +954,6 @@ def test_transaction_name(
         auto_enabling_integrations=False,  # Make sure that httpx integration is not added, because it adds tracing information to the starlette test clients request.
         integrations=[StarletteIntegration(transaction_style=transaction_style)],
         traces_sample_rate=1.0,
-        debug=True,
     )
 
     envelopes = capture_envelopes()
@@ -1015,7 +1014,6 @@ def dummy_traces_sampler(sampling_context):
         integrations=[StarletteIntegration(transaction_style=transaction_style)],
         traces_sampler=dummy_traces_sampler,
         traces_sample_rate=1.0,
-        debug=True,
     )
 
     app = starlette_app_factory()
@@ -1057,7 +1055,6 @@ def test_transaction_name_in_middleware(
             StarletteIntegration(transaction_style=transaction_style),
         ],
         traces_sample_rate=1.0,
-        debug=True,
     )
 
     envelopes = capture_envelopes()

tests/test_basics.py

@@ -439,7 +439,7 @@ def test_integration_scoping(sentry_init, capture_events):
 def test_client_initialized_within_scope(sentry_init, caplog):
     caplog.set_level(logging.WARNING)
 
-    sentry_init(debug=True)
+    sentry_init()
 
     with push_scope():
         Hub.current.bind_client(Client())
@@ -455,7 +455,7 @@ def test_client_initialized_within_scope(sentry_init, caplog):
 def test_scope_leaks_cleaned_up(sentry_init, caplog):
     caplog.set_level(logging.WARNING)
 
-    sentry_init(debug=True)
+    sentry_init()
 
     old_stack = list(Hub.current._stack)
 
@@ -475,7 +475,7 @@ def test_scope_leaks_cleaned_up(sentry_init, caplog):
 def test_scope_popped_too_soon(sentry_init, caplog):
     caplog.set_level(logging.ERROR)
 
-    sentry_init(debug=True)
+    sentry_init()
 
     old_stack = list(Hub.current._stack)
 
@@ -519,7 +519,7 @@ def bar(event, hint):
 
 
 def test_capture_event_with_scope_kwargs(sentry_init, capture_events):
-    sentry_init(debug=True)
+    sentry_init()
     events = capture_events()
     capture_event({}, level="info", extras={"foo": "bar"})
     (event,) = events