-
Notifications
You must be signed in to change notification settings - Fork 116
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
gateway: add api to get authenticated user organizations #299
gateway: add api to get authenticated user organizations #299
Conversation
7cdad59
to
5411a9a
Compare
Hi Simone, I've just rollback the API's name from /user/{userref}/orgs to /users/{userref}/orgs in configstore and fixed some lint issues on setup_test.go. It is ready to be reviewed. thanks |
a5e0e90
to
d8f42b8
Compare
@tulliobotti64 Please also rename the commit to |
e03912c
to
c01b943
Compare
6ba293f
to
7dfc12c
Compare
7dfc12c
to
ec4eab8
Compare
ec4eab8
to
d0d219c
Compare
60f075a
to
5c737fd
Compare
i've run both tests on my pc and I got no error. Can you try to restart Agola's run ? thanks |
c5c3982
to
bc7d9e1
Compare
bc7d9e1
to
9e32d65
Compare
tests/setup_test.go
Outdated
} | ||
|
||
if diff := cmp.Diff(expectedOrgs, orgs); diff != "" { | ||
t.Fatalf("user orgs mismatch (-expected +got):\n%s", diff) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For consistency use (-want +got)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done, please review 👍
tests/setup_test.go
Outdated
@@ -40,6 +40,7 @@ import ( | |||
gwapitypes "agola.io/agola/services/gateway/api/types" | |||
gwclient "agola.io/agola/services/gateway/client" | |||
rstypes "agola.io/agola/services/runservice/types" | |||
"github.com/google/go-cmp/cmp" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Move this below with external packages
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
gateway: add api to get authenticated user organizations gateway: add api to get authenticated user organizations
9e32d65
to
7160194
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tulliobotti64 Thanks! Merging.
…by_organization gateway: add api to get authenticated user organizations
…by_organization gateway: add api to get authenticated user organizations
Added in the Gateway the API /user/{userref}/orgs
Added some integration tests. Unit tests were done already
To fix the issue #292