-
Notifications
You must be signed in to change notification settings - Fork 16
update domain -> host config references in stripe and sentry test infrastructure #698
update domain -> host config references in stripe and sentry test infrastructure #698
Conversation
Thanks for hopping on this @adamsachs + @galvana 🙏 |
@@ -54,6 +54,6 @@ identity_email = "" | |||
|
|||
[zendesk] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixing names here, not necessary for our build pipeline but this needs to match if you're testing locally.
@@ -27,7 +27,7 @@ def test_saas_access_request_task( | |||
"""Full access request based on the Hubspot SaaS config""" | |||
|
|||
privacy_request = PrivacyRequest( | |||
id=f"test_saas_access_request_task_{random.randint(0, 1000)}" | |||
id=f"test_hubspot_access_request_task_{random.randint(0, 1000)}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To uniquely identify the test
f"Operational Error connecting to '{mailchimp_connection_config.key}'." | ||
== body["failure_reason"] | ||
f"Operational Error connecting to '{mailchimp_connection_config.key}'" | ||
in body["failure_reason"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed from a ==
to in
since dev mode will cause the error to contain additional context.
@@ -1241,10 +1241,9 @@ def test_saas_connection_connect_with_components( | |||
db: Session, | |||
generate_auth_header, | |||
mailchimp_connection_config, | |||
mailchimp_secrets, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed this to prevent the secrets from being output to the logs during an error.
f"Operational Error connecting to '{mailchimp_connection_config.key}'." | ||
== body["failure_reason"] | ||
f"Operational Error connecting to '{mailchimp_connection_config.key}'" | ||
in body["failure_reason"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same fix here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @adamsachs for starting this off, not sure if I should be approving this since I wrote part of it too but it's just a handful of simple changes.
…rastructure (#698) Co-authored-by: Adam Sachs <adam@Adams-MacBook-Pro.local> Co-authored-by: Adrian Galvan <adriang430@gmail.com>
No description provided.