fix: Check for provider ID in social organization signal
Summary
add_user_to_socialapp_organization was only checking for the provider string which leads to exclude Open ID providers or similar that relies on the provider ID. So no Open ID users are added to the org through the social app.
This MR update the queries to check for the provider ID string before falling back to the provider string
PS: This branch has mostly the same changes on the test side (besides the query number change to show the user assertion fail) to show that the test fails with an Open ID provider
Checklist
-
I am respecting Sentry's proprietary Business Source License (BSL) by implementing code in a clean room environment. I have not read any BSL Sentry code. It is OK to read MIT licensed code.
Edited by tytan652