Merge pull request #103018 from feiskyer/test-cleanup

chore: clean up the tests that have external dependencies
This commit is contained in:
Kubernetes Prow Robot 2021-06-21 08:22:51 -07:00 committed by GitHub
commit 671251a8b9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -50,68 +50,6 @@ var (
}
)
func TestGetServicePrincipalTokenFromMSIWithUserAssignedID(t *testing.T) {
configs := []*AzureAuthConfig{
{
UseManagedIdentityExtension: true,
UserAssignedIdentityID: "UserAssignedIdentityID",
},
// The Azure service principal is ignored when
// UseManagedIdentityExtension is set to true
{
UseManagedIdentityExtension: true,
UserAssignedIdentityID: "UserAssignedIdentityID",
TenantID: "TenantID",
AADClientID: "AADClientID",
AADClientSecret: "AADClientSecret",
},
}
env := &azure.PublicCloud
for _, config := range configs {
token, err := GetServicePrincipalToken(config, env)
assert.NoError(t, err)
msiEndpoint, err := adal.GetMSIVMEndpoint()
assert.NoError(t, err)
spt, err := adal.NewServicePrincipalTokenFromMSIWithUserAssignedID(msiEndpoint,
env.ServiceManagementEndpoint, config.UserAssignedIdentityID)
assert.NoError(t, err)
assert.Equal(t, token, spt)
}
}
func TestGetServicePrincipalTokenFromMSI(t *testing.T) {
configs := []*AzureAuthConfig{
{
UseManagedIdentityExtension: true,
},
// The Azure service principal is ignored when
// UseManagedIdentityExtension is set to true
{
UseManagedIdentityExtension: true,
TenantID: "TenantID",
AADClientID: "AADClientID",
AADClientSecret: "AADClientSecret",
},
}
env := &azure.PublicCloud
for _, config := range configs {
token, err := GetServicePrincipalToken(config, env)
assert.NoError(t, err)
msiEndpoint, err := adal.GetMSIVMEndpoint()
assert.NoError(t, err)
spt, err := adal.NewServicePrincipalTokenFromMSI(msiEndpoint, env.ServiceManagementEndpoint)
assert.NoError(t, err)
assert.Equal(t, token, spt)
}
}
func TestGetServicePrincipalToken(t *testing.T) {
config := &AzureAuthConfig{
TenantID: "TenantID",