Skip to content

chore: add unittests to helm chart #192

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jul 3, 2025
Merged

Conversation

Steffen911
Copy link
Contributor

@Steffen911 Steffen911 commented Jul 3, 2025

Important

Add GitHub Actions workflow and unit tests for Helm chart with setup instructions in README.

  • GitHub Actions:
    • Adds .github/workflows/test.yaml to run Helm unit tests on pull_request and push to main.
    • Uses actions/checkout@v4, azure/setup-helm@v3, and EnricoMi/publish-unit-test-result-action@v2.
    • Installs helm-unittest plugin and runs tests on charts/langfuse/.
  • README:
    • Adds testing section with setup and local test running instructions.
  • Tests:
    • Adds basic_test.yaml and minimal-installation_test.yaml in charts/langfuse/tests/.
    • basic_test.yaml: Tests rendering with default values, app labels, and image specification.
    • minimal-installation_test.yaml: Tests rendering of basic resources, secret references, and database connections.

This description was created by Ellipsis for f86b990. You can customize this summary. It will automatically update as commits are pushed.

Copy link

github-actions bot commented Jul 3, 2025

Helm Unit Test Results

6 tests   6 ✅  0s ⏱️
2 suites  0 💤
1 files    0 ❌

Results for commit 62c7484.

♻️ This comment has been updated with latest results.

Steffen911 and others added 2 commits July 3, 2025 14:13
…n permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@Steffen911 Steffen911 merged commit a8a8087 into main Jul 3, 2025
5 checks passed
@Steffen911 Steffen911 deleted the steffen/add-unittests branch July 3, 2025 12:15
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jul 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant