Skip to content

chore(dev): add integration tests for the top command#24649

Merged
pront merged 10 commits intomasterfrom
pront-add-top-integration-tests
Feb 17, 2026
Merged

chore(dev): add integration tests for the top command#24649
pront merged 10 commits intomasterfrom
pront-add-top-integration-tests

Conversation

@pront
Copy link
Copy Markdown
Member

@pront pront commented Feb 12, 2026

Summary

Adds E2E tests for vector top.

Vector configuration

N/A

How did you test this PR?

This PR introduces tests.

Change Type

  • Bug fix
  • New feature
  • Non-functional (chore, refactoring, docs)
  • Performance

Is this a breaking change?

  • Yes
  • No

Does this PR include user facing changes?

  • Yes. Please add a changelog fragment based on our guidelines.
  • No. A maintainer will apply the no-changelog label to this PR.

References

  • Related: #

Notes

  • Please read our Vector contributor resources.
  • Do not hesitate to use @vectordotdev/vector to reach out to us regarding this PR.
  • Some CI checks run only after we manually approve them.
    • We recommend adding a pre-push hook, please see this template.
    • Alternatively, we recommend running the following locally before pushing to the remote branch:
      • make fmt
      • make check-clippy (if there are failures it's possible some of them can be fixed with make clippy-fix)
      • make test
  • After a review is requested, please avoid force pushes to help us review incrementally.
    • Feel free to push as many commits as you want. They will be squashed into one before merging.
    • For example, you can run git merge origin master and git push.
  • If this PR introduces changes Vector dependencies (modifies Cargo.lock), please
    run make build-licenses to regenerate the license inventory and commit the changes (if any). More details here.

@pront pront added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Feb 12, 2026
@pront pront changed the title chore(top): add top integration tests chore(dev): add integration tests for the top command Feb 12, 2026
@pront pront marked this pull request as ready for review February 17, 2026 15:54
@pront pront requested a review from a team as a code owner February 17, 2026 15:54
@datadog-vectordotdev
Copy link
Copy Markdown

datadog-vectordotdev bot commented Feb 17, 2026

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: ba51693 | Docs | Was this helpful? Give us feedback!

Comment thread tests/integration/top/harness.rs
Comment thread tests/integration/top/harness.rs
Comment thread tests/integration/top/harness.rs Outdated
Comment thread tests/integration/top/harness.rs Outdated
@pront pront enabled auto-merge February 17, 2026 16:54
@pront pront added this pull request to the merge queue Feb 17, 2026
Merged via the queue into master with commit a51820e Feb 17, 2026
51 of 52 checks passed
@pront pront deleted the pront-add-top-integration-tests branch February 17, 2026 19:54
@github-actions github-actions bot locked and limited conversation to collaborators Feb 17, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog Changes in this PR do not need user-facing explanations in the release changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants