Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add stripe setupIntent api #1083

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

suejung-sentry
Copy link
Contributor

To use the Stripe PaymentElement UI component, need to pass a clientSecret based on an established "setup intent". This PR adds the corresponding api to fetch the needed clientSecret.

@suejung-sentry suejung-sentry force-pushed the sshin/stripe-setup-intent branch from 9f023ab to 058e85f Compare January 7, 2025 00:35
Copy link

codecov bot commented Jan 7, 2025

❌ 15 Tests Failed:

Tests completed Failed Passed Skipped
2674 15 2659 7
View the top 3 failed tests by shortest run time
services/tests/test_billing.py::BillingServiceTests::test_get_subscription
Stack Traces | 0.002s run time
self = <services.tests.test_billing.BillingServiceTests testMethod=test_get_subscription>

    def setUp(self):
>       self.mock_payment_service = MockPaymentService()
E       TypeError: Can't instantiate abstract class MockPaymentService without an implementation for abstract method 'create_setup_intent'

services/tests/test_billing.py:1875: TypeError
services/tests/test_billing.py::BillingServiceTests::test_list_filtered_invoices_calls_payment_service_list_filtered_invoices_with_limit
Stack Traces | 0.002s run time
self = <services.tests.test_billing.BillingServiceTests testMethod=test_list_filtered_invoices_calls_payment_service_list_filtered_invoices_with_limit>

    def setUp(self):
>       self.mock_payment_service = MockPaymentService()
E       TypeError: Can't instantiate abstract class MockPaymentService without an implementation for abstract method 'create_setup_intent'

services/tests/test_billing.py:1875: TypeError
services/tests/test_billing.py::BillingServiceTests::test_update_plan_modifies_subscription_if_user_plan_and_subscription_exists
Stack Traces | 0.002s run time
self = <services.tests.test_billing.BillingServiceTests testMethod=test_update_plan_modifies_subscription_if_user_plan_and_subscription_exists>

    def setUp(self):
>       self.mock_payment_service = MockPaymentService()
E       TypeError: Can't instantiate abstract class MockPaymentService without an implementation for abstract method 'create_setup_intent'

services/tests/test_billing.py:1875: TypeError

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

@codecov-notifications
Copy link

codecov-notifications bot commented Jan 7, 2025

❌ 15 Tests Failed:

Tests completed Failed Passed Skipped
2675 15 2660 6
View the top 3 failed tests by shortest run time
services/tests/test_billing.py::BillingServiceTests::test_get_subscription
Stack Traces | 0.002s run time
self = <services.tests.test_billing.BillingServiceTests testMethod=test_get_subscription>

    def setUp(self):
>       self.mock_payment_service = MockPaymentService()
E       TypeError: Can't instantiate abstract class MockPaymentService without an implementation for abstract method 'create_setup_intent'

services/tests/test_billing.py:1875: TypeError
services/tests/test_billing.py::BillingServiceTests::test_list_filtered_invoices_calls_payment_service_list_filtered_invoices_with_limit
Stack Traces | 0.002s run time
self = <services.tests.test_billing.BillingServiceTests testMethod=test_list_filtered_invoices_calls_payment_service_list_filtered_invoices_with_limit>

    def setUp(self):
>       self.mock_payment_service = MockPaymentService()
E       TypeError: Can't instantiate abstract class MockPaymentService without an implementation for abstract method 'create_setup_intent'

services/tests/test_billing.py:1875: TypeError
services/tests/test_billing.py::BillingServiceTests::test_update_plan_modifies_subscription_if_user_plan_and_subscription_exists
Stack Traces | 0.002s run time
self = <services.tests.test_billing.BillingServiceTests testMethod=test_update_plan_modifies_subscription_if_user_plan_and_subscription_exists>

    def setUp(self):
>       self.mock_payment_service = MockPaymentService()
E       TypeError: Can't instantiate abstract class MockPaymentService without an implementation for abstract method 'create_setup_intent'

services/tests/test_billing.py:1875: TypeError

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

@codecov-qa
Copy link

codecov-qa bot commented Jan 7, 2025

❌ 15 Tests Failed:

Tests completed Failed Passed Skipped
2674 15 2659 7
View the top 3 failed tests by shortest run time
services/tests/test_billing.py::BillingServiceTests::test_get_subscription
Stack Traces | 0.002s run time
self = <services.tests.test_billing.BillingServiceTests testMethod=test_get_subscription>

    def setUp(self):
>       self.mock_payment_service = MockPaymentService()
E       TypeError: Can't instantiate abstract class MockPaymentService without an implementation for abstract method 'create_setup_intent'

services/tests/test_billing.py:1875: TypeError
services/tests/test_billing.py::BillingServiceTests::test_list_filtered_invoices_calls_payment_service_list_filtered_invoices_with_limit
Stack Traces | 0.002s run time
self = <services.tests.test_billing.BillingServiceTests testMethod=test_list_filtered_invoices_calls_payment_service_list_filtered_invoices_with_limit>

    def setUp(self):
>       self.mock_payment_service = MockPaymentService()
E       TypeError: Can't instantiate abstract class MockPaymentService without an implementation for abstract method 'create_setup_intent'

services/tests/test_billing.py:1875: TypeError
services/tests/test_billing.py::BillingServiceTests::test_update_plan_modifies_subscription_if_user_plan_and_subscription_exists
Stack Traces | 0.002s run time
self = <services.tests.test_billing.BillingServiceTests testMethod=test_update_plan_modifies_subscription_if_user_plan_and_subscription_exists>

    def setUp(self):
>       self.mock_payment_service = MockPaymentService()
E       TypeError: Can't instantiate abstract class MockPaymentService without an implementation for abstract method 'create_setup_intent'

services/tests/test_billing.py:1875: TypeError

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

Copy link

❌ 15 Tests Failed:

Tests completed Failed Passed Skipped
2675 15 2660 6
View the top 3 failed tests by shortest run time
services/tests/test_billing.py::BillingServiceTests::test_get_subscription
Stack Traces | 0.002s run time
self = <services.tests.test_billing.BillingServiceTests testMethod=test_get_subscription>

    def setUp(self):
>       self.mock_payment_service = MockPaymentService()
E       TypeError: Can't instantiate abstract class MockPaymentService without an implementation for abstract method 'create_setup_intent'

services/tests/test_billing.py:1875: TypeError
services/tests/test_billing.py::BillingServiceTests::test_list_filtered_invoices_calls_payment_service_list_filtered_invoices_with_limit
Stack Traces | 0.002s run time
self = <services.tests.test_billing.BillingServiceTests testMethod=test_list_filtered_invoices_calls_payment_service_list_filtered_invoices_with_limit>

    def setUp(self):
>       self.mock_payment_service = MockPaymentService()
E       TypeError: Can't instantiate abstract class MockPaymentService without an implementation for abstract method 'create_setup_intent'

services/tests/test_billing.py:1875: TypeError
services/tests/test_billing.py::BillingServiceTests::test_update_plan_modifies_subscription_if_user_plan_and_subscription_exists
Stack Traces | 0.002s run time
self = <services.tests.test_billing.BillingServiceTests testMethod=test_update_plan_modifies_subscription_if_user_plan_and_subscription_exists>

    def setUp(self):
>       self.mock_payment_service = MockPaymentService()
E       TypeError: Can't instantiate abstract class MockPaymentService without an implementation for abstract method 'create_setup_intent'

services/tests/test_billing.py:1875: TypeError

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

Copy link
Contributor

github-actions bot commented Jan 7, 2025

❌ 15 Tests Failed:

Tests completed Failed Passed Skipped
2681 15 2660 6
View the top 3 failed tests by shortest run time
test_get_subscription
Stack Traces | 0.002s run time
self = &lt;services.tests.test_billing.BillingServiceTests testMethod=test_get_subscription&gt;

    def setUp(self):
&gt;       self.mock_payment_service = MockPaymentService()
E       TypeError: Can't instantiate abstract class MockPaymentService without an implementation for abstract method 'create_setup_intent'

services/tests/test_billing.py:1875: TypeError
test_list_filtered_invoices_calls_payment_service_list_filtered_invoices_with_limit
Stack Traces | 0.002s run time
self = &lt;services.tests.test_billing.BillingServiceTests testMethod=test_list_filtered_invoices_calls_payment_service_list_filtered_invoices_with_limit&gt;

    def setUp(self):
&gt;       self.mock_payment_service = MockPaymentService()
E       TypeError: Can't instantiate abstract class MockPaymentService without an implementation for abstract method 'create_setup_intent'

services/tests/test_billing.py:1875: TypeError
test_update_plan_modifies_subscription_if_user_plan_and_subscription_exists
Stack Traces | 0.002s run time
self = &lt;services.tests.test_billing.BillingServiceTests testMethod=test_update_plan_modifies_subscription_if_user_plan_and_subscription_exists&gt;

    def setUp(self):
&gt;       self.mock_payment_service = MockPaymentService()
E       TypeError: Can't instantiate abstract class MockPaymentService without an implementation for abstract method 'create_setup_intent'

services/tests/test_billing.py:1875: TypeError

📣 Thoughts on this report? Let Codecov know! | Powered by Codecov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant