chore(sentry apps): Audit Sentry App Errors

[Christinarlong]

not going to get merged, I just need to see (and copy paste) the changes to new PRs :oopsie

[codecov]

❌ 49 Tests Failed:

Tests completed	Failed	Passed	Skipped
23381	49	23332	214

View the top 3 failed tests by shortest run time

tests.sentry.sentry_apps.api.endpoints.test_sentry_apps.PostSentryAppsTest::test_missing_webhook_url

Stack Traces | 1.47s run time

#x1B[1m#x1B[.../api/endpoints/test_sentry_apps.py#x1B[0m:694: in test_missing_webhook_url
    assert "webhookUrl" in response.data
#x1B[1m#x1B[31mE   assert 'webhookUrl' in {'error': "{'webhookUrl': 'webhookUrl required for public integrations'}"}#x1B[0m
#x1B[1m#x1B[31mE    +  where {'error': "{'webhookUrl': 'webhookUrl required for public integrations'}"} = <Response status_code=400, "application/json">.data#x1B[0m

tests.sentry.sentry_apps.api.endpoints.test_sentry_apps.PostSentryAppsTest::test_cannot_create_app_without_correct_permissions

Stack Traces | 1.54s run time

#x1B[1m#x1B[.../api/endpoints/test_sentry_apps.py#x1B[0m:589: in test_cannot_create_app_without_correct_permissions
    assert response.data == {"events": ["issue webhooks require the event:read permission."]}
#x1B[1m#x1B[31mE   assert {'error': "{'...ermission.'}"} == {'events': ['...permission.']}#x1B[0m
#x1B[1m#x1B[31mE     #x1B[0m
#x1B[1m#x1B[31mE     Left contains 1 more item:#x1B[0m
#x1B[1m#x1B[31mE     {'error': "{'events': 'issue webhooks require the event:read permission.'}"}#x1B[0m
#x1B[1m#x1B[31mE     Right contains 1 more item:#x1B[0m
#x1B[1m#x1B[31mE     {'events': ['issue webhooks require the event:read permission.']}#x1B[0m
#x1B[1m#x1B[31mE     #x1B[0m
#x1B[1m#x1B[31mE     Full diff:#x1B[0m
#x1B[1m#x1B[31mE       {#x1B[0m
#x1B[1m#x1B[31mE     -     'events': [#x1B[0m
#x1B[1m#x1B[31mE     -         'issue webhooks require the event:read permission.',#x1B[0m
#x1B[1m#x1B[31mE     ?     ^^^#x1B[0m
#x1B[1m#x1B[31mE     +     'error': "{'events': 'issue webhooks require the event:read permission.'}",#x1B[0m
#x1B[1m#x1B[31mE     ?     ^^^^^^^^^^^^^^^^^^^^                                                    ++#x1B[0m
#x1B[1m#x1B[31mE     -     ],#x1B[0m
#x1B[1m#x1B[31mE       }#x1B[0m

tests.sentry.sentry_apps.api.endpoints.test_sentry_apps.PostSentryAppsTest::test_no_author_public_integration

Stack Traces | 1.54s run time

#x1B[1m#x1B[.../api/endpoints/test_sentry_apps.py#x1B[0m:717: in test_no_author_public_integration
    assert response.data == {"author": ["author required for public integrations"]}
#x1B[1m#x1B[31mE   assert {'error': "{'...tegrations'}"} == {'author': ['...ntegrations']}#x1B[0m
#x1B[1m#x1B[31mE     #x1B[0m
#x1B[1m#x1B[31mE     Left contains 1 more item:#x1B[0m
#x1B[1m#x1B[31mE     {'error': "{'author': 'author required for public integrations'}"}#x1B[0m
#x1B[1m#x1B[31mE     Right contains 1 more item:#x1B[0m
#x1B[1m#x1B[31mE     {'author': ['author required for public integrations']}#x1B[0m
#x1B[1m#x1B[31mE     #x1B[0m
#x1B[1m#x1B[31mE     Full diff:#x1B[0m
#x1B[1m#x1B[31mE       {#x1B[0m
#x1B[1m#x1B[31mE     -     'author': [#x1B[0m
#x1B[1m#x1B[31mE     -         'author required for public integrations',#x1B[0m
#x1B[1m#x1B[31mE     ?     ^^^^#x1B[0m
#x1B[1m#x1B[31mE     +     'error': "{'author': 'author required for public integrations'}",#x1B[0m
#x1B[1m#x1B[31mE     ?     ^^^^^^^^^^^       ++++++++++                                  ++#x1B[0m
#x1B[1m#x1B[31mE     -     ],#x1B[0m
#x1B[1m#x1B[31mE       }#x1B[0m

To view more test analytics, go to the Test Analytics Dashboard
📢 Thoughts on this report? Let us know!