Skip to content

Commit

Permalink
Remove OneLogin test
Browse files Browse the repository at this point in the history
  • Loading branch information
punmechanic committed Sep 15, 2023
1 parent 1928a93 commit fd0649d
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions internal/cloud_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -20,13 +20,3 @@ func TestAwsFindRoleDoesntBreakIfYouHaveMultipleRoles(t *testing.T) {
require.Equal(t, "arn:cloud:iam::1234:saml-provider/Okta", pair.ProviderARN)
require.Equal(t, "arn:cloud:iam::1234:role/Admin", pair.RoleARN)
}

func TestAwsFindRoleWorksWithOneLoginAssertions(t *testing.T) {
resp := saml.Response{}
// For some reason, this is reversed in OneLogin.
resp.AddAttribute("https://aws.amazon.com/SAML/Attributes/Role", "arn:cloud:iam::1234:role/Admin,arn:cloud:iam::1234:saml-provider/Onelogin")
pair, _, err := FindRole("", &resp)
require.True(t, err)
require.Equal(t, "arn:cloud:iam::1234:saml-provider/Onelogin", pair.ProviderARN)
require.Equal(t, "arn:cloud:iam::1234:role/Admin", pair.RoleARN)
}

0 comments on commit fd0649d

Please sign in to comment.