Skip to content

Commit db48a71

Browse files
ajay-sentryandrewshie-sentry
authored andcommitted
chore(billing): Clean up category-specific fields for PerCategoryOnDemandBudgetWithSpends (#96488)
This PR just removes a couple fields from an interface and subsequently from some tests. They weren't used at all in the UI otherwise. Closes https://linear.app/getsentry/issue/BIL-959/clean-up-category-specific-fields-for <!-- Sentry employees and contractors can delete or ignore the following. --> ### Legal Boilerplate Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.
1 parent d01c326 commit db48a71

File tree

8 files changed

+0
-43
lines changed

8 files changed

+0
-43
lines changed

static/gsApp/components/navBillingStatus.spec.tsx

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -204,9 +204,6 @@ describe('PrimaryNavigationQuotaExceeded', function () {
204204
},
205205
usedSpends: {},
206206
enabled: true,
207-
attachmentSpendUsed: 0,
208-
errorSpendUsed: 0,
209-
transactionSpendUsed: 0,
210207
};
211208
SubscriptionStore.set(organization.slug, subscription);
212209
render(<PrimaryNavigationQuotaExceeded organization={organization} />);

static/gsApp/types/index.tsx

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -242,10 +242,6 @@ export type PerCategoryOnDemandBudget = {
242242
};
243243

244244
type PerCategoryOnDemandBudgetWithSpends = PerCategoryOnDemandBudget & {
245-
attachmentSpendUsed: number;
246-
errorSpendUsed: number;
247-
transactionSpendUsed: number;
248-
// TODO(data categories): BIL-959
249245
usedSpends: Partial<Record<DataCategory, number>>;
250246
};
251247

static/gsApp/views/amCheckout/steps/onDemandBudgets.spec.tsx

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -320,9 +320,6 @@ describe('OnDemandBudgets AM Checkout', function () {
320320
onDemandBudgets: {
321321
enabled: false,
322322
budgetMode: OnDemandBudgetMode.PER_CATEGORY,
323-
attachmentSpendUsed: 0,
324-
errorSpendUsed: 0,
325-
transactionSpendUsed: 0,
326323
usedSpends: {errors: 0, transactions: 0, attachments: 0, replays: 0},
327324
budgets: {errors: 1000, transactions: 2000, attachments: 3000},
328325
},

static/gsApp/views/onDemandBudgets/onDemandBudgets.spec.tsx

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -235,9 +235,6 @@ describe('OnDemandBudgets', () => {
235235
replays: 0,
236236
monitorSeats: 0,
237237
},
238-
attachmentSpendUsed: 0,
239-
errorSpendUsed: 0,
240-
transactionSpendUsed: 0,
241238
usedSpends: {},
242239
},
243240
});
@@ -380,9 +377,6 @@ describe('OnDemandBudgets', () => {
380377
attachments: 3000,
381378
monitorSeats: 4000,
382379
},
383-
errorSpendUsed: 100,
384-
transactionSpendUsed: 200,
385-
attachmentSpendUsed: 300,
386380
usedSpends: {
387381
errors: 100,
388382
transactions: 200,
@@ -616,9 +610,6 @@ describe('OnDemandBudgets', () => {
616610
attachments: 3000,
617611
monitorSeats: 4000,
618612
},
619-
attachmentSpendUsed: 0,
620-
errorSpendUsed: 0,
621-
transactionSpendUsed: 0,
622613
usedSpends: {},
623614
},
624615
});

static/gsApp/views/onDemandBudgets/utils.spec.tsx

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -149,9 +149,6 @@ describe('parseOnDemandBudgetsFromSubscription', function () {
149149
profileDurationUI: 0,
150150
logBytes: 0,
151151
},
152-
attachmentSpendUsed: 0,
153-
errorSpendUsed: 0,
154-
transactionSpendUsed: 0,
155152
usedSpends: {
156153
errors: 0,
157154
transactions: 0,
@@ -321,9 +318,6 @@ describe('getTotalBudget', function () {
321318
enabled: true,
322319
budgetMode: OnDemandBudgetMode.PER_CATEGORY,
323320
budgets: {errors: 100, transactions: 200, attachments: 300, uptime: 400},
324-
attachmentSpendUsed: 0,
325-
errorSpendUsed: 0,
326-
transactionSpendUsed: 0,
327321
usedSpends: {errors: 0, transactions: 0, attachments: 0, replays: 0},
328322
},
329323
});

static/gsApp/views/subscriptionPage/onDemandSettings.spec.tsx

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -145,9 +145,6 @@ describe('edit on-demand budget', () => {
145145
attachments: 3000,
146146
monitorSeats: 4000,
147147
},
148-
errorSpendUsed: 100,
149-
transactionSpendUsed: 200,
150-
attachmentSpendUsed: 300,
151148
usedSpends: {
152149
errors: 100,
153150
transactions: 200,
@@ -271,9 +268,6 @@ describe('edit on-demand budget', () => {
271268
replays: 300,
272269
monitorSeats: 200,
273270
},
274-
errorSpendUsed: 76,
275-
transactionSpendUsed: 0,
276-
attachmentSpendUsed: 0,
277271
usedSpends: {
278272
errors: 76,
279273
transactions: 0,
@@ -365,9 +359,6 @@ describe('edit on-demand budget', () => {
365359
replays: 0,
366360
monitorSeats: 5000,
367361
},
368-
attachmentSpendUsed: 0,
369-
errorSpendUsed: 0,
370-
transactionSpendUsed: 0,
371362
usedSpends: {},
372363
},
373364
});
@@ -562,9 +553,6 @@ describe('edit on-demand budget', () => {
562553
replays: 0,
563554
monitorSeats: 5000,
564555
},
565-
attachmentSpendUsed: 0,
566-
errorSpendUsed: 0,
567-
transactionSpendUsed: 0,
568556
usedSpends: {},
569557
},
570558
});

static/gsApp/views/subscriptionPage/pendingChanges.spec.tsx

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -393,9 +393,6 @@ describe('Subscription > PendingChanges', function () {
393393
transactions: 0,
394394
attachments: 0,
395395
},
396-
attachmentSpendUsed: 0,
397-
errorSpendUsed: 0,
398-
transactionSpendUsed: 0,
399396
usedSpends: {},
400397
},
401398
pendingChanges: PendingChangesFixture({

static/gsApp/views/subscriptionPage/usageTotals.spec.tsx

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1498,9 +1498,6 @@ describe('Subscription > CombinedUsageTotals', function () {
14981498
[DataCategory.SEER_AUTOFIX]: 4,
14991499
[DataCategory.SEER_SCANNER]: 5,
15001500
},
1501-
attachmentSpendUsed: 0,
1502-
errorSpendUsed: 0,
1503-
transactionSpendUsed: 0,
15041501
usedSpends: {
15051502
[DataCategory.SEER_AUTOFIX]: 4,
15061503
[DataCategory.SEER_SCANNER]: 5,

0 commit comments

Comments
 (0)