Skip to content

Commit 4702f41

Browse files
authored
Refine round-trip auth fix per validation feedback
1 parent 05fa963 commit 4702f41

2 files changed

Lines changed: 4 additions & 5 deletions

File tree

.github/workflows/integration-test.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,6 @@ jobs:
125125
env:
126126
AZURE_CLIENT_ID: ${{ secrets.AZURE_CLIENT_ID }}
127127
AZURE_TENANT_ID: ${{ secrets.AZURE_TENANT_ID }}
128-
AZURE_SUBSCRIPTION_ID: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
129128
run: |
130129
$skipTeardown = '${{ inputs.skip_teardown }}' -eq 'true'
131130
$logLevel = '${{ inputs.log_level }}'

tests/integration/all-resource-types/run-roundtrip-test.ps1

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -136,17 +136,17 @@ function Get-ApiopsLogLevel([string]$ScriptLogLevel) {
136136
}
137137

138138
function Get-ApiopsAuthArgs {
139-
$args = @()
139+
$authArgs = @()
140140

141141
if (-not [string]::IsNullOrWhiteSpace($env:AZURE_CLIENT_ID)) {
142-
$args += @('--client-id', $env:AZURE_CLIENT_ID)
142+
$authArgs += @('--client-id', $env:AZURE_CLIENT_ID)
143143
}
144144

145145
if (-not [string]::IsNullOrWhiteSpace($env:AZURE_TENANT_ID)) {
146-
$args += @('--tenant-id', $env:AZURE_TENANT_ID)
146+
$authArgs += @('--tenant-id', $env:AZURE_TENANT_ID)
147147
}
148148

149-
return $args
149+
return $authArgs
150150
}
151151

152152
function Invoke-Teardown {

0 commit comments

Comments
 (0)