Skip to content

Commit 270c0ae

Browse files
authored
Bugfix/lint workflow (CloudPirates-io#367)
* Fix invalid caching in github workflow * fix invalid testing script * Fix unit tests yet again * hofixes * Disavle unit test for clusterpirate * Fix if statement not found * Fix if statement not found --------- Signed-off-by: Finn Rades <64548817+zOnlyKroks@users.noreply.github.com>
1 parent 954de0f commit 270c0ae

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

.github/workflows/pull-request.yaml

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,8 @@ jobs:
7777
7878
- name: Run chart testing (lint & unittest)
7979
if: steps.list-changed.outputs.changed == 'true'
80-
run: ct lint --target-branch ${{ github.event.repository.default_branch }} --validate-maintainers=false --additional-commands "sh -c 'if [ ! -f {{ .Path }}/.disable-unittest ]; then helm unittest {{ .Path }}; else echo \"Skipping unittest for {{ .Path }} (.disable-unittest found)\"; fi'"
80+
run: |
81+
ct lint --target-branch ${{ github.event.repository.default_branch }} --validate-maintainers=false --additional-commands 'helm unittest {{ .Path }}'
8182
8283
integration-test:
8384
runs-on: ubuntu-latest
@@ -134,6 +135,12 @@ jobs:
134135
CHART_NAME=${chart_directory#charts/}
135136
echo "Testing chart: $CHART_NAME"
136137
138+
# Check if integration test is disabled for this chart
139+
if [ -f "${chart_directory}/.disable-unittest" ]; then
140+
echo "⏩ Skipping integration test for $CHART_NAME (.disable-unittest found)"
141+
continue
142+
fi
143+
137144
# Run test script without cluster creation (kind-action already created it)
138145
# and without cleanup (let GitHub Actions handle it)
139146
./test-charts.sh "$CHART_NAME" --no-cleanup

0 commit comments

Comments
 (0)