@oneuptime/common 12.0.17 → 12.0.18

package/Server/Services/BillingService.ts

@@ -2074,6 +2074,14 @@ export class BillingService extends BaseService {
       return "price_1U0iWJANuQdJ93r7iVAXwhdP";
     }
 
+    if (productType === ProductType.SecurityEvents) {
+      if (this.isTestEnvironment()) {
+        return "price_1U7efaANuQdJ93r74hFVOgdS";
+      }
+
+      return "price_1U7edTANuQdJ93r7hR9jpBfv";
+    }
+
     throw new BadDataException(
       "Plan with productType " + productType + " not found",
     );

package/Tests/App/Dashboard/PayAsYouGoNotices.test.tsx

@@ -120,6 +120,23 @@ describe("Pay as you go notices", () => {
       expect(card).toHaveTextContent("Starting at");
     });
 
+    it("names security events among the pillars billed at the telemetry rate", () => {
+      /*
+       * Security events ingest on the same key and meter at the same
+       * $0.10/GB as the other telemetry pillars. This sentence is the body of
+       * the consent checkbox a user has to tick to create a key, so a pillar
+       * missing from it is a pillar the user is charged for without ever
+       * having been told.
+       */
+      render(<TelemetryPayAsYouGoCard />);
+
+      expect(
+        screen.getByTestId("telemetry-pay-as-you-go-card"),
+      ).toHaveTextContent(
+        "Logs, traces, metrics, profiles and security events are billed at",
+      );
+    });
+
     it("says the charge stops when the key is deleted, so the notice is not just a threat", () => {
       render(<TelemetryPayAsYouGoCard />);
 

package/Tests/Server/Services/BillingService.test.ts

@@ -882,6 +882,9 @@ describe("BillingService", () => {
         expect(service.hasMeteredPlanPriceId(ProductType.SessionReplay)).toBe(
           true,
         );
+        expect(service.hasMeteredPlanPriceId(ProductType.SecurityEvents)).toBe(
+          true,
+        );
       });
 
       it("should return false for a product type with no Stripe price", () => {
@@ -901,6 +904,23 @@ describe("BillingService", () => {
         expect(LogDataIngestMeteredPlan.hasPriceId()).toBe(true);
         expect(SessionReplayDataIngestMeteredPlan.hasPriceId()).toBe(true);
       });
+
+      /*
+       * Security Events shipped its ingest, metering and staging without a
+       * Stripe price, so usage accrued in TelemetryUsageBilling and was never
+       * invoiced - silently, because the callers treat a missing price as
+       * "skip this plan". Enumerate the enum rather than listing types by
+       * hand so the next pillar added cannot repeat that.
+       */
+      it("should have a Stripe price for every ProductType", () => {
+        const unpriced: Array<ProductType> = Object.values(ProductType).filter(
+          (productType: ProductType) => {
+            return !service.hasMeteredPlanPriceId(productType);
+          },
+        );
+
+        expect(unpriced).toEqual([]);
+      });
     });
 
     describe("isPromoCodeValid", () => {