cleanup

Author: isabellaenriquez

static/gsApp/views/amCheckout/checkoutOverviewV2.spec.tsx

@@ -100,7 +100,7 @@ describe('CheckoutOverviewV2', function () {
     );
 
     expect(screen.getByText('All Sentry Products')).toBeInTheDocument();
-    expect(screen.getByTestId('seer-reserved')).toBeInTheDocument(); // .toHaveTextContent('Seer $216/yr'); TODO(seer): uncomment this once fixtures are updated with pricing schedule
+    expect(screen.getByTestId('seer-reserved')).toHaveTextContent('Seer$216/yr');
     expect(screen.getByText('Total Annual Charges')).toBeInTheDocument();
     expect(screen.getByText('$312/yr')).toBeInTheDocument();
     expect(screen.getByTestId('additional-monthly-charge')).toHaveTextContent(
@@ -201,4 +201,34 @@ describe('CheckoutOverviewV2', function () {
 
     expect(screen.queryByTestId('seer-reserved')).not.toBeInTheDocument();
   });
+
+  it('does not show seer when not included in formData', function () {
+    const formData: CheckoutFormData = {
+      plan: 'am3_team_auf',
+      reserved: {
+        errors: 100000,
+        attachments: 25,
+        replays: 50,
+        spans: 10_000_000,
+        monitorSeats: 1,
+        profileDuration: 0,
+        profileDurationUI: 0,
+        uptime: 1,
+      },
+      onDemandMaxSpend: 5000,
+    };
+
+    render(
+      <CheckoutOverviewV2
+        activePlan={teamPlanAnnual}
+        billingConfig={billingConfig}
+        formData={formData}
+        onUpdate={jest.fn()}
+        organization={organization}
+        subscription={subscription}
+      />
+    );
+
+    expect(screen.queryByTestId('seer-reserved')).not.toBeInTheDocument();
+  });
 });

static/gsApp/views/amCheckout/checkoutOverviewV2.tsx

@@ -46,9 +46,6 @@ function CheckoutOverviewV2({activePlan, formData, onUpdate: _onUpdate}: Props)
     [formData.onDemandMaxSpend]
   );
 
-  // const hasSeerEnabled = !!formData.seerEnabled;
-  // const hasSeerFeature = organization.features.includes('seer-billing');
-
   const renderPlanDetails = () => {
     return (
       <PanelChild>
@@ -72,56 +69,6 @@ function CheckoutOverviewV2({activePlan, formData, onUpdate: _onUpdate}: Props)
     );
   };
 
-  // const renderAdditionalFeatureSummary = ({
-  //   featureKey,
-  //   featureEnabled,
-  //   featureAvailable,
-  //   title,
-  //   tooltipTitle,
-  //   priceCents,
-  // }: {
-  //   featureAvailable: boolean;
-  //   featureEnabled: boolean;
-  //   featureKey: string;
-  //   priceCents: number;
-  //   title: string;
-  //   tooltipTitle: string;
-  // }) => {
-  //   return (
-  //     featureAvailable &&
-  //     featureEnabled && (
-  //       <PanelChild data-test-id={`${featureKey}-summary`}>
-  //         <SpaceBetweenRow style={{alignItems: 'start'}}>
-  //           <Column>
-  //             <div style={{display: 'flex', alignItems: 'center', gap: space(1)}}>
-  //               <Title>
-  //                 {title}
-  //                 &nbsp;&nbsp;
-  //                 <QuestionTooltip size="xs" title={tooltipTitle} />
-  //               </Title>
-  //             </div>
-  //           </Column>
-  //           <Column minWidth="150px" alignItems="end">
-  //             <Title>{`+${utils.displayPrice({cents: priceCents})}/mo`}</Title>
-  //             <Description>Additional usage billed separately</Description>
-  //           </Column>
-  //         </SpaceBetweenRow>
-  //       </PanelChild>
-  //     )
-  //   );
-  // };
-
-  // const renderSeerSummary = () => {
-  //   return renderAdditionalFeatureSummary({
-  //     featureKey: 'seer',
-  //     featureEnabled: hasSeerEnabled,
-  //     featureAvailable: hasSeerFeature,
-  //     title: t('Sentry AI Agent'),
-  //     tooltipTitle: t('Additional Seer information.'),
-  //     priceCents: SEER_MONTHLY_PRICE_CENTS,
-  //   });
-  // };
-
   const renderPayAsYouGoBudget = (paygBudgetTotal: number) => {
     return (
       <PanelChild>

static/gsApp/views/amCheckout/steps/planSelect.tsx

@@ -238,88 +238,6 @@ function PlanSelect({
     );
   };
 
-  // const renderSeer = () => {
-  //   return renderAdditionalFeature({
-  //     featureKey: 'seer',
-  //     featureEnabled: seerIsEnabled,
-  //     featureAvailable: hasSeerFeature,
-  //     title: 'Add Seer, our AI Agent',
-  //     description: 'Insights and solutions to fix bugs faster',
-  //     icon: <SeerIcon size="lg" />,
-  //     priceCents: SEER_MONTHLY_PRICE_CENTS,
-  //     featureItems: ['Root Cause Analysis', 'Autofix PRs', 'AI Issue Priority'],
-  //     tooltipTitle: t('Additional Seer information.'),
-  //     isNew: true,
-  //     setFeatureEnabled: setSeerIsEnabled,
-  //   });
-  // };
-
-  // const renderAdditionalFeature = ({
-  //   featureKey,
-  //   featureEnabled,
-  //   featureAvailable,
-  //   title,
-  //   description,
-  //   icon,
-  //   priceCents,
-  //   featureItems,
-  //   tooltipTitle,
-  //   isNew = false,
-  //   setFeatureEnabled,
-  // }: {
-  //   description: string;
-  //   featureAvailable: boolean;
-  //   featureEnabled: boolean;
-  //   featureItems: string[];
-  //   featureKey: string;
-  //   icon: React.ReactNode;
-  //   priceCents: number;
-  //   setFeatureEnabled: (enabled: boolean) => void;
-  //   title: string;
-  //   tooltipTitle: string;
-  //   isNew?: boolean;
-  // }) => {
-  //   return (
-  //     featureAvailable && (
-  //       <StepFooter data-test-id={`footer-${featureKey}`}>
-  //         <FeatureContainer>
-  //           <FeatureHeader>
-  //             {icon}
-  //             {title}
-  //             {isNew && <FeatureBadge type="new" variant="badge" />}
-  //           </FeatureHeader>
-  //           <FeatureDescription>{description}</FeatureDescription>
-  //           <FeatureList>
-  //             {featureItems.map((item, index) => (
-  //               <FeatureItem key={index}>{item}</FeatureItem>
-  //             ))}
-  //           </FeatureList>
-  //           <ButtonWrapper>
-  //             <Button
-  //               priority={'default'}
-  //               icon={featureEnabled ? <IconCheckmark /> : <IconAdd />}
-  //               onClick={() => {
-  //                 // Update the UI state
-  //                 setFeatureEnabled(!featureEnabled);
-  //                 // Update the form data
-  //                 onUpdate({...formData, [`${featureKey}Enabled`]: !featureEnabled});
-  //               }}
-  //               data-test-id={`${featureKey}-toggle-button`}
-  //             >
-  //               {featureEnabled
-  //                 ? t('Added to plan')
-  //                 : t('%s/mo', utils.displayPrice({cents: priceCents}))}
-  //             </Button>
-  //           </ButtonWrapper>
-  //           <div>
-  //             Extra usage requires PAYG <QuestionTooltip size="xs" title={tooltipTitle} />
-  //           </div>
-  //         </FeatureContainer>
-  //       </StepFooter>
-  //     )
-  //   );
-  // };
-
   const renderFooter = () => {
     const bizPlanContent = getContentForPlan('business', checkoutTier);
     let missingFeatures: string[] = [];

static/gsApp/views/amCheckout/utils.tsx

@@ -249,8 +249,6 @@ export function getReservedPriceCents({
 
   Object.entries(selectedProducts ?? {}).forEach(([apiName, selectedProductData]) => {
     if (selectedProductData.enabled) {
-      // TODO: replace this to use the pricing schedule once that's serialized on availableReservedBudgetTypes
-      // This way, we can handle both annual and monthly prices
       const budgetTypeInfo =
         plan.availableReservedBudgetTypes[apiName as ReservedBudgetCategoryType];
       if (budgetTypeInfo) {