From 0430836ca2c8ace538ceb733ddd72c635631e24c Mon Sep 17 00:00:00 2001 From: Dhruwang Date: Wed, 20 May 2026 17:14:15 +0530 Subject: [PATCH] test: drop assertOrganizationAIConfigured capability assertion MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Per review feedback — exclude the test-side regression assertion for the capability argument. Production-code change in the previous commit (ENG-1001) is the actual fix. Co-Authored-By: Claude Opus 4.7 (1M context) --- apps/web/modules/ee/analysis/charts/actions.test.ts | 4 ---- 1 file changed, 4 deletions(-) diff --git a/apps/web/modules/ee/analysis/charts/actions.test.ts b/apps/web/modules/ee/analysis/charts/actions.test.ts index eb0e193780..7bdbcc04c9 100644 --- a/apps/web/modules/ee/analysis/charts/actions.test.ts +++ b/apps/web/modules/ee/analysis/charts/actions.test.ts @@ -206,9 +206,5 @@ describe("chart Cube actions", () => { userId: "user-1", source: "charts.generateAIChartAction", }); - // AI chart generation should gate on "smartTools" — it only sends schema context - // and the prompt to the LLM, no response PII. Using "dataAnalysis" would require - // users to opt into the PII-consent toggle for a non-PII feature. - expect(mocks.assertOrganizationAIConfigured).toHaveBeenCalledWith("organization-1", "smartTools"); }); });