Files
insights/products/metrics/frontend/metricsIngestionLogic.test.ts
zeekay 10e6fec4c0 debrand: the brand token is INSIGHTS_, and a wrong one stops the container
The rule table mapped HOG_ -> FN_ ahead of any POSTHOG rule, so POSTHOG_FOO
split into POST + HOG_ and came out POSTFN_FOO. Nothing sets that. The live
pods set INSIGHTS_SKIP_MIGRATION_CHECKS, which bin/migrate-check reads on the
container entrypoint, so the misspelling does not degrade a feature -- it runs
migrate --check against a database whose history this tree does not share and
the container never starts.

Both replacements are measured against the shipped tree rather than chosen:
every HOG_ name with a counterpart there is spelled INSIGHTS_ (HOG_QL ->
INSIGHTS_QL, HOG_FUNCTION_TEMPLATES -> INSIGHTS_FUNCTION_TEMPLATES, 12 of 12
determinate cases), and FN_ appears in it nowhere. Upstream carries 95 HOG_
tokens and zero FN_ ones, so repairing FN_ back out is exactly the inverse of
the bad rule and cannot collide with an upstream name.

The rule is fixed for the next convergence, and the tree it already produced is
repaired in the same commit -- the codemod is idempotent against its own output,
so it cannot undo this itself.
2026-08-01 17:42:28 -07:00

134 lines
5.5 KiB
TypeScript

import { expectLogic } from 'kea-test-utils'
import { teamLogic } from 'scenes/teamLogic'
import { ProductIntentContext, ProductKey } from '~/queries/schema/schema-general'
import { initKeaTests } from '~/test/init'
import { AccessControlLevel, AccessControlResourceType, AppContext } from '~/types'
import { metricsHasMetricsRetrieve } from './generated/api'
import { metricsIngestionLogic } from './metricsIngestionLogic'
jest.mock('./generated/api', () => ({
metricsHasMetricsRetrieve: jest.fn(),
}))
describe('metricsIngestionLogic', () => {
let logic: ReturnType<typeof metricsIngestionLogic.build>
// teamLogic is mounted via connect once `logic` mounts, so its actionCreators are available here.
const firstIngestIntent = (): any =>
teamLogic.actionCreators.addProductIntent({
product_type: ProductKey.METRICS,
intent_context: ProductIntentContext.METRICS_FIRST_INGESTED,
})
beforeEach(() => {
localStorage.clear()
window.INSIGHTS_APP_CONTEXT = {
...window.INSIGHTS_APP_CONTEXT,
resource_access_control: {
...window.INSIGHTS_APP_CONTEXT?.resource_access_control,
[AccessControlResourceType.Metrics]: AccessControlLevel.Viewer,
},
} as AppContext
initKeaTests()
jest.mocked(metricsHasMetricsRetrieve).mockReset()
})
afterEach(() => {
logic?.unmount()
})
// Distinguishes a team completing OTel setup mid-session (intent) from a team that simply
// already had metrics (no intent) — firing on any `true` would mark every long-standing
// team as newly intending and corrupt the activation funnel.
it('records a product intent only on an observed no-metrics -> has-metrics transition, once', async () => {
jest.mocked(metricsHasMetricsRetrieve).mockResolvedValue({ hasMetrics: false } as any)
logic = metricsIngestionLogic()
logic.mount()
await expectLogic(logic)
.toDispatchActions(['loadTeamHasMetricsSuccess'])
.toNotHaveDispatchedActions([firstIngestIntent()])
jest.mocked(metricsHasMetricsRetrieve).mockResolvedValue({ hasMetrics: true } as any)
await expectLogic(logic, () => {
logic.actions.loadTeamHasMetrics()
}).toDispatchActions([firstIngestIntent()])
await expectLogic(logic, () => {
logic.actions.loadTeamHasMetrics()
}).toNotHaveDispatchedActions([firstIngestIntent()])
})
it('a team that already has metrics on first check records no intent', async () => {
jest.mocked(metricsHasMetricsRetrieve).mockResolvedValue({ hasMetrics: true } as any)
logic = metricsIngestionLogic()
logic.mount()
await expectLogic(logic)
.toDispatchActions(['loadTeamHasMetricsSuccess'])
.toNotHaveDispatchedActions([firstIngestIntent()])
})
// Copying the agent snippet is the strongest pre-ingest signal a team is taking
// the scrape path — it must register a distinct intent (not the generic docs one)
// so activation can attribute the scrape funnel, once per variant per session.
it('records a scrape-path intent when an agent snippet is copied, once per variant', async () => {
jest.mocked(metricsHasMetricsRetrieve).mockResolvedValue({ hasMetrics: false } as any)
logic = metricsIngestionLogic()
logic.mount()
const scrapeIntent = (): any =>
teamLogic.actionCreators.addProductIntent({
product_type: ProductKey.METRICS,
intent_context: ProductIntentContext.METRICS_SCRAPE_AGENT_SNIPPET_COPIED,
metadata: { variant: 'docker' },
})
await expectLogic(logic, () => {
logic.actions.reportScrapeSnippetCopied('docker')
}).toDispatchActions([scrapeIntent()])
// The same variant again is not a new intent...
await expectLogic(logic, () => {
logic.actions.reportScrapeSnippetCopied('docker')
}).toNotHaveDispatchedActions([scrapeIntent()])
// ...but the other variant is.
await expectLogic(logic, () => {
logic.actions.reportScrapeSnippetCopied('kubernetes')
}).toDispatchActions([
teamLogic.actionCreators.addProductIntent({
product_type: ProductKey.METRICS,
intent_context: ProductIntentContext.METRICS_SCRAPE_AGENT_SNIPPET_COPIED,
metadata: { variant: 'kubernetes' },
}),
])
})
it('does not check ingestion without metrics viewer access', async () => {
window.INSIGHTS_APP_CONTEXT = {
...window.INSIGHTS_APP_CONTEXT,
resource_access_control: {
...window.INSIGHTS_APP_CONTEXT?.resource_access_control,
[AccessControlResourceType.Metrics]: AccessControlLevel.None,
},
} as AppContext
logic = metricsIngestionLogic()
logic.mount()
await expectLogic(logic).delay(10)
expect(metricsHasMetricsRetrieve).not.toHaveBeenCalled()
// A denied check must stay "unknown", not become "no metrics" - false would
// show the setup prompt to a user on a team that may have plenty of metrics.
await expectLogic(logic, () => {
logic.actions.loadTeamHasMetrics()
}).toFinishAllListeners()
expect(metricsHasMetricsRetrieve).not.toHaveBeenCalled()
expect(logic.values.hasMetrics).toBeUndefined()
})
})