feat: add feedback when canceling

debtcollective · Oct 15, 2019 · 8c17bc2 · 8c17bc2
1 parent 7326544
commit 8c17bc2
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 4 deletions.
diff --git a/app/javascript/bundles/User/components/SubscriptionCancel.jsx b/app/javascript/bundles/User/components/SubscriptionCancel.jsx
@@ -10,6 +10,7 @@ const SUBSCRIPTION_CANCEL_URL = userID => `/users/${userID}/subscription`
 
 function SubscriptionCancelView ({ user, subscription }) {
   const [open, setOpen] = useState(false)
+  const [active, toggleActive] = useState(subscription.active)
 
   const handleClickOpen = () => {
     setOpen(true)
@@ -20,9 +21,14 @@ function SubscriptionCancelView ({ user, subscription }) {
   }
 
   const handleSubscriptionCancellation = async () => {
-    await fetch(SUBSCRIPTION_CANCEL_URL(user.id), {
-      method: 'delete'
-    })
+    const isSubscriptionCancelled = await fetch(
+      SUBSCRIPTION_CANCEL_URL(user.id),
+      {
+        method: 'delete'
+      }
+    )
+
+    toggleActive(!subscription.active)
 
     handleClose()
   }
@@ -36,6 +42,7 @@ function SubscriptionCancelView ({ user, subscription }) {
       <Button color='secondary' onClick={handleClickOpen}>
         Cancel Subscription
       </Button>
+      {!active && 'No active subscription'}
       <Dialog
         open={open}
         onClose={handleClose}

diff --git a/spec/features/user/users_profile_spec.rb b/spec/features/user/users_profile_spec.rb
@@ -43,7 +43,7 @@
         click_button 'Cancel Subscription'
       end
 
-      sleep 2
+      expect(page).to have_content('No active subscription')
       subscription.reload
       expect(subscription.active).to eq(false)
     end