Skip to content

test: add git config integration tests#232

Draft
patrick-stephens wants to merge 3 commits intomainfrom
161_add_git_config_integration_tests
Draft

test: add git config integration tests#232
patrick-stephens wants to merge 3 commits intomainfrom
161_add_git_config_integration_tests

Conversation

@patrick-stephens
Copy link
Contributor

@patrick-stephens patrick-stephens commented Mar 20, 2026

Resolves #161 by adding integration tests for the git_config plugin.


Summary by cubic

Add integration tests for the git_config plugin and pass the current PR SHA to test jobs so the plugin polls the right ref. Implements the coverage requested in #161.

  • New Tests

    • BATS test to validate git_config startup, repo polling, commit detection, and switching to the updated config.
    • Adds initial and updated Fluent Bit configs used by the test; updated config logs the current GIT_SHA.
    • Verifies custom plugins are discoverable: git_config, log_sampling, and dedupe.
  • CI

    • Sets GIT_SHA in test workflows to ${{ github.event.pull_request.head.sha || github.sha }}.
    • run-bats.sh reads and logs GIT_SHA to drive the tests.

Written for commit 9ae46e6. Summary will update on new commits.

Signed-off-by: Patrick Stephens <pat@telemetryforge.io>
Signed-off-by: Patrick Stephens <pat@telemetryforge.io>
Signed-off-by: Patrick Stephens <pat@telemetryforge.io>
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.

Add integration tests for git config plugin

1 participant