Skip to content

Add metrics integration testing (v1) - #1

Closed
maxday wants to merge 27 commits into
mainfrom
maxday/integration-test
Closed

maxday wants to merge 27 commits into
mainfrom
maxday/integration-test

Conversation

@maxday

@maxday maxday commented May 10, 2021 •

Copy link
Copy Markdown
Contributor

This PR :

  • bootstraps the integration tests setup for metrics
  • embed one integration test : metrics with DD_ENHANCED_METRICS=true

Coming in a next PR :

  • more tests about metrics
  • logic to loop over all test cases (instead of hardcoding the test function name)
  • check invocation status code

lym953 added a commit that referenced this pull request Jul 13, 2026
Invoke both functions a second time after the cold start and check the
tag in a separate time window, to confirm the tag isn't only set on
invocation #1 but persists across warm invocations.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
lym953 added a commit that referenced this pull request Jul 14, 2026
…dy processed

Handles both orderings of the Invoke event and platform.initStart. If
initStart has already resolved the durable tag by the time invocation #1
arrives, there's nothing to wait for, so emit the metric right away
instead of holding it back (which previously would only be flushed at
shutdown, since platform.initStart doesn't fire again).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant