fix(billing): use status allowlist for plan resolution in usage endpoint

- Replace past_due-only check with active/trialing allowlist
- All other statuses (canceled, unpaid, incomplete, etc.) now map to free
- Add test.each for all non-active statuses + trialing test

Author: PierreBrisorgueil

modules/billing/controllers/billing.controller.js

@@ -67,7 +67,8 @@ const getUsage = async (req, res) => {
   try {
     // Determine current plan via service layer
     const subscription = await BillingService.getSubscription(req.organization._id);
-    const plan = (!subscription || subscription.status === 'past_due') ? 'free' : (subscription.plan || 'free');
+    const activeStatuses = ['active', 'trialing'];
+    const plan = (!subscription || !activeStatuses.includes(subscription.status)) ? 'free' : (subscription.plan || 'free');
 
     // Get usage counters (includes month field)
     const usage = await BillingUsageService.get(req.organization._id.toString());

modules/billing/tests/billing.usage.endpoint.unit.tests.js

@@ -97,8 +97,15 @@ describe('Billing usage endpoint unit tests:', () => {
     }));
   });
 
-  test('should return free plan when subscription is past_due', async () => {
-    mockBillingService.getSubscription.mockResolvedValue({ plan: 'starter', status: 'past_due' });
+  test.each([
+    'past_due',
+    'canceled',
+    'unpaid',
+    'incomplete',
+    'incomplete_expired',
+    'paused',
+  ])('should return free plan when subscription status is %s', async (status) => {
+    mockBillingService.getSubscription.mockResolvedValue({ plan: 'starter', status });
     mockBillingUsageService.get.mockResolvedValue({ month: '2026-03', counters: { 'documents.create': 2 } });
 
     const req = { organization: { _id: orgId } };
@@ -113,6 +120,22 @@ describe('Billing usage endpoint unit tests:', () => {
     }));
   });
 
+  test('should return paid plan when subscription status is trialing', async () => {
+    mockBillingService.getSubscription.mockResolvedValue({ plan: 'starter', status: 'trialing' });
+    mockBillingUsageService.get.mockResolvedValue({ month: '2026-03', counters: {} });
+
+    const req = { organization: { _id: orgId } };
+    await billingController.getUsage(req, res);
+
+    expect(res.status).toHaveBeenCalledWith(200);
+    expect(res.json).toHaveBeenCalledWith(expect.objectContaining({
+      data: expect.objectContaining({
+        plan: 'starter',
+        limits: { 'documents.create': 20, 'requests.execute': 2000 },
+      }),
+    }));
+  });
+
   test('should return empty limits when plan has no quota config', async () => {
     mockBillingService.getSubscription.mockResolvedValue({ plan: 'enterprise', status: 'active' });
     mockBillingUsageService.get.mockResolvedValue({ month: '2026-03', counters: {} });