-
Notifications
You must be signed in to change notification settings - Fork 143
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
client.Group.ListAssignedApplicationsForGroup returns an application properly, but gives error for nonexistent index #292
Labels
bug
Something isn't working
Comments
@pputman12 thanks for giving me a head start into debugging this. I will look into it. |
Just checking @pputman12, did you close this ticket inadvertently? I'm going to write some code to see if I can repo this in my dev env. |
I didn't close it inadvertently it was intentionally, I was mistaken about
the problem.
…On Wed, Apr 20, 2022 at 12:22 PM Mike Mondragon ***@***.***> wrote:
Just checking @pputman12 <https://github.com/pputman12>, did you close
this ticket inadvertently? I'm going to write some code to see if I can
repo this in my dev env.
—
Reply to this email directly, view it on GitHub
<#292 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGNEWVK7YMG4WWQACVPG6DVGA4ONANCNFSM5T2WYGXA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
ok, no problem! I have new integration test to add to the next release 😄 |
I was mistaken because I made a new integration test for something I'm
doing in okta, and it caused this error. But the problem wasn't with this
module, somehow when I run this new test, it triggers an error with an
empty slice returning from a similar method in a different test. If I run
each test separately, I can't reproduce it, but can reproduce it every time
if I run each test together, I'll write up something on it, but my initial
report was confusing me as it came from a similar method in another test
but only happened after I made the new test.
…On Wed, Apr 20, 2022 at 12:49 PM Mike Mondragon ***@***.***> wrote:
ok, no problem! I have new integration test to add to the next release 😄
—
Reply to this email directly, view it on GitHub
<#292 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAGNEWWVYVNEKVPOPVUK4KDVGA7RPANCNFSM5T2WYGXA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
MikeMondragon-okta
pushed a commit
that referenced
this issue
Apr 20, 2022
This was referenced Apr 20, 2022
monde
added a commit
that referenced
this issue
Jun 2, 2022
Test for Group ListAssignedApplications. Discussed in #292
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug?
When trying to call ListAssignedApplicationsForGroup, I get an error:
I'm writing a test that creates an okta group and app, with a rule that should assign them, so I'm testing if it exists, and it does when printing it out:
I've isolated the problem down to ListAssignedApplicationsForGroup but not sure why its complaining about index out of range.
What is expected to happen?
its expected to just run the test successfully, since the application exists and is assigned to the group
What is the actual behavior?
It does pull down the application, and I can see it searches for it and finds it, but I get an error saying index out of range
Reproduction Steps?
Try to call this method with these parameters:
ctx, client, err := oktaTest.NewConnection(
oktaCon.testOrg,
oktaCon.testUrl,
oktaCon.testApiToken,
)
if err != nil {
fmt.Printf("Could not connect to okta %v", err)
}
oktaGroupApps will have the proper application, but it will still error out at the end. It panics and recovers somehow?
Additional Information?
No response
Golang Version
1.18 and 1.18.1
SDK Version
latest
OS version
Darwin Patricks-MacBook-Pro.local 21.4.0 Darwin Kernel Version 21.4.0: Fri Mar 18 00:46:32 PDT 2022; root:xnu-8020.101.4~15/RELEASE_ARM64_T6000 arm64
The text was updated successfully, but these errors were encountered: