Fix: skip-variables-check option in self-contained templates (#5053)

* fix: skip-variables-check option in self-contained templates

* Update build workflow envs
This commit is contained in:
Ramana Reddy 2024-04-23 16:04:32 +05:30 committed by GitHub
parent 5dacb255ac
commit 61e9be530f
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 4 additions and 2 deletions

View File

@ -42,6 +42,7 @@ jobs:
- name: Test - name: Test
env: env:
GITHUB_TOKEN: "${{ secrets.GITHUB_TOKEN }}" GITHUB_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
PDCP_API_KEY: "${{ secrets.PDCP_API_KEY }}"
run: go test ./... run: go test ./...
- name: Integration Tests - name: Integration Tests
@ -49,6 +50,7 @@ jobs:
env: env:
GH_ACTION: true GH_ACTION: true
GITHUB_TOKEN: "${{ secrets.GITHUB_TOKEN }}" GITHUB_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
PDCP_API_KEY: "${{ secrets.PDCP_API_KEY }}"
run: | run: |
chmod +x run.sh chmod +x run.sh
bash run.sh ${{ matrix.os }} bash run.sh ${{ matrix.os }}

View File

@ -277,7 +277,7 @@ func (r *requestGenerator) makeSelfContainedRequest(ctx context.Context, data st
return nil, fmt.Errorf("malformed request supplied") return nil, fmt.Errorf("malformed request supplied")
} }
if err := expressions.ContainsUnresolvedVariables(parts[1]); err != nil { if err := expressions.ContainsUnresolvedVariables(parts[1]); err != nil && !r.request.SkipVariablesCheck {
return nil, ErrUnresolvedVars.Msgf(parts[1]) return nil, ErrUnresolvedVars.Msgf(parts[1])
} }
@ -296,7 +296,7 @@ func (r *requestGenerator) makeSelfContainedRequest(ctx context.Context, data st
} }
return r.generateRawRequest(ctx, data, parsed, values, payloads) return r.generateRawRequest(ctx, data, parsed, values, payloads)
} }
if err := expressions.ContainsUnresolvedVariables(data); err != nil { if err := expressions.ContainsUnresolvedVariables(data); err != nil && !r.request.SkipVariablesCheck {
// early exit: if there are any unresolved variables in `path` after evaluation // early exit: if there are any unresolved variables in `path` after evaluation
// then return early since this will definitely fail // then return early since this will definitely fail
return nil, ErrUnresolvedVars.Msgf(data) return nil, ErrUnresolvedVars.Msgf(data)