get rid of baremetrics (#3957)

Author: adrianviquez

.env

@@ -1,3 +1,2 @@
 REACT_APP_SENTRY_DSN=$SENTRY_DSN
 REACT_APP_STRIPE_KEY=$STRIPE_KEY
-REACT_APP_BAREMETRICS_TOKEN=$BAREMETRICS_TOKEN

.env.development

@@ -4,4 +4,3 @@ REACT_APP_BASE_URL=https://stage-web.codecov.dev
 REACT_APP_MARKETING_BASE_URL=https://about.codecov.io
 # REACT_APP_STRIPE_KEY=
 # REACT_APP_LAUNCHDARKLY=
-# REACT_APP_BAREMETRICS_TOKEN=

src/pages/PlanPage/subRoutes/CancelPlanPage/subRoutes/DowngradePlan/CancelButton/CancelButton.jsx

@@ -1,25 +1,24 @@
+import { format, fromUnixTime } from 'date-fns'
 import PropType from 'prop-types'
 import { useState } from 'react'
 
 import Button from 'ui/Button'
 import Modal from 'ui/Modal'
 
 import { useCancel } from './hooks'
-import { cleanupBaremetrics, getEndPeriod } from './utils'
 
 const FALLBACK_PERIOD_TEXT = 'the end of the period'
 
-function CancelButton({
-  customerId,
-  isFreePlan,
-  upComingCancellation,
-  currentPeriodEnd,
-}) {
+function getEndPeriod(unixPeriodEnd) {
+  return (
+    unixPeriodEnd &&
+    format(fromUnixTime(unixPeriodEnd), 'MMMM do yyyy, h:m aaaa')
+  )
+}
+
+function CancelButton({ isFreePlan, upComingCancellation, currentPeriodEnd }) {
   const [isModalOpen, setIsModalOpen] = useState(false)
-  const { cancelPlan, baremetricsBlocked, queryIsLoading } = useCancel({
-    customerId,
-    isModalOpen,
-  })
+  const { cancelPlan, queryIsLoading } = useCancel()
 
   const isDisabled = [
     // disable button if
@@ -29,14 +28,7 @@ function CancelButton({
   ].some(Boolean)
   const periodEnd = getEndPeriod(currentPeriodEnd)
 
-  function completeCancellation() {
-    if (baremetricsBlocked) {
-      cancelPlan()
-    }
-  }
-
   function handleOnClose() {
-    cleanupBaremetrics()
     setIsModalOpen(false)
   }
 
@@ -84,12 +76,10 @@ function CancelButton({
               Cancel
             </Button>
             <Button
-              // This ID is needed to render the baremetrics form. DO NOT CHANGE
-              id="barecancel-trigger"
               variant="danger"
               hook="continue-cancellation-button"
               disabled={isDisabled}
-              onClick={completeCancellation}
+              onClick={cancelPlan}
             >
               Confirm Cancellation
             </Button>

src/pages/PlanPage/subRoutes/CancelPlanPage/subRoutes/DowngradePlan/CancelButton/CancelButton.test.jsx

@@ -7,7 +7,6 @@ import CancelButton from './CancelButton'
 
 const mocks = vi.hoisted(() => ({
   useParams: vi.fn(),
-  useBarecancel: vi.fn(),
   useCancelPlan: vi.fn(),
   useAddNotification: vi.fn(),
 }))
@@ -20,14 +19,6 @@ vi.mock('react-router-dom', async () => {
   }
 })
 
-vi.mock('./useBarecancel', async () => {
-  const actual = await vi.importActual('./useBarecancel')
-  return {
-    ...actual,
-    useBarecancel: mocks.useBarecancel,
-  }
-})
-
 vi.mock('services/account/useCancelPlan', async () => {
   const actual = await vi.importActual('services/account/useCancelPlan')
   return {
@@ -72,7 +63,7 @@ const wrapper = ({ children }) => (
 )
 
 describe('CancelButton', () => {
-  function setup(baremetricsBlocked = false) {
+  function setup() {
     const user = userEvent.setup()
     const mutate = vi.fn()
     const addNotification = vi.fn()
@@ -84,7 +75,6 @@ describe('CancelButton', () => {
       mutate,
       onError: vi.fn(),
     })
-    mocks.useBarecancel.mockReturnValue({ baremetricsBlocked })
 
     return { mutate, addNotification, user }
   }
@@ -95,9 +85,9 @@ describe('CancelButton', () => {
     it('renders button text', () => {
       render(
         <CancelButton
-          customerId="cus_1n4o328hn4"
+          isFreePlan={false}
           planCost="users-pr-inappy"
-          upComingCancelation={false}
+          upComingCancellation={false}
           currentPeriodEnd={1675361466}
         />,
         { wrapper }
@@ -114,9 +104,9 @@ describe('CancelButton', () => {
       const { user } = setup()
       render(
         <CancelButton
-          customerId="cus_1n4o328hn4"
+          isFreePlan={false}
           planCost="users-pr-inappy"
-          upComingCancelation={false}
+          upComingCancellation={false}
           currentPeriodEnd={1675361466}
         />,
         { wrapper }
@@ -132,9 +122,9 @@ describe('CancelButton', () => {
         const { user } = setup()
         render(
           <CancelButton
-            customerId="cus_1n4o328hn4"
+            isFreePlan={false}
             planCost="users-pr-inappy"
-            upComingCancelation={false}
+            upComingCancellation={false}
             currentPeriodEnd={1675361466}
           />,
           { wrapper }
@@ -154,9 +144,9 @@ describe('CancelButton', () => {
         const { user } = setup()
         render(
           <CancelButton
-            customerId="cus_1n4o328hn4"
+            isFreePlan={false}
             planCost="users-pr-inappy"
-            upComingCancelation={false}
+            upComingCancellation={false}
             currentPeriodEnd={1675361466}
           />,
           { wrapper }
@@ -170,39 +160,16 @@ describe('CancelButton', () => {
         ).not.toBeInTheDocument()
       })
     })
-
-    describe('when unmounted', () => {
-      it('removes the baremetrics script', async () => {
-        const { user } = setup()
-        const { unmount } = render(
-          <CancelButton
-            customerId="cus_1n4o328hn4"
-            planCost="users-pr-inappy"
-            upComingCancelation={false}
-            currentPeriodEnd={1675361466}
-          />,
-          { wrapper }
-        )
-
-        await user.click(screen.getByTestId('downgrade-button'))
-
-        unmount()
-
-        expect(
-          screen.queryByTestId('baremetrics-script')
-        ).not.toBeInTheDocument()
-      })
-    })
   })
 
   describe('when clicking submit', () => {
     it('calls the cancelPlan/mutate function', async () => {
       const { mutate, user } = setup(true)
       render(
         <CancelButton
-          customerId="cus_1n4o328hn4"
+          isFreePlan={false}
           planCost="users-pr-inappy"
-          upComingCancelation={false}
+          upComingCancellation={false}
           currentPeriodEnd={1675361466}
         />,
         { wrapper }
@@ -219,9 +186,9 @@ describe('CancelButton', () => {
         const { mutate, addNotification, user } = setup(true)
         render(
           <CancelButton
-            customerId="cus_1n4o328hn4"
+            isFreePlan={false}
             planCost="users-pr-inappy"
-            upComingCancelation={false}
+            upComingCancellation={false}
             currentPeriodEnd={1675361466}
           />,
           { wrapper }
@@ -244,9 +211,9 @@ describe('CancelButton', () => {
         const { mutate, user } = setup(true)
         render(
           <CancelButton
-            customerId="cus_1n4o328hn4"
+            isFreePlan={false}
             planCost="users-pr-inappy"
-            upComingCancelation={false}
+            upComingCancellation={false}
             currentPeriodEnd={1675361466}
           />,
           { wrapper }

src/pages/PlanPage/subRoutes/CancelPlanPage/subRoutes/DowngradePlan/CancelButton/hooks.js

@@ -3,18 +3,11 @@ import { useHistory, useParams } from 'react-router-dom'
 import { useCancelPlan } from 'services/account/useCancelPlan'
 import { useAddNotification } from 'services/toastNotification/context'
 
-import { useBarecancel } from './useBarecancel'
-
-export function useCancel({ customerId, isModalOpen }, options = {}) {
+export function useCancel(options = {}) {
   const addToast = useAddNotification()
   const { provider, owner } = useParams()
   const { push } = useHistory()
   const { mutate, isLoading } = useCancelPlan({ provider, owner })
-  const { baremetricsBlocked } = useBarecancel({
-    customerId,
-    callbackSend: cancelPlan,
-    isModalOpen,
-  })
 
   function sendUserToPlan() {
     push(`/plan/${provider}/${owner}`)
@@ -36,7 +29,6 @@ export function useCancel({ customerId, isModalOpen }, options = {}) {
 
   return {
     cancelPlan,
-    baremetricsBlocked,
     queryIsLoading: isLoading,
   }
 }