Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Subtask] Add Integration Test of OAuth2TokenProvider of client-python #4208

Closed
noidname01 opened this issue Jul 18, 2024 · 2 comments · Fixed by #4663
Closed

[Subtask] Add Integration Test of OAuth2TokenProvider of client-python #4208

noidname01 opened this issue Jul 18, 2024 · 2 comments · Fixed by #4663
Assignees
@noidname01
Labels
0.7.0 Release v0.7.0 subtask Subtasks of umbrella issue

Comments

@noidname01
Copy link
Collaborator

Describe the subtask

  • add Integration Test of OAuth2TokenProvider of client-python

Parent issue

#3755
@noidname01 noidname01 added the subtask Subtasks of umbrella issue label Jul 18, 2024
@noidname01 noidname01 mentioned this issue Jul 18, 2024
Merged
2 tasks
jerryshao pushed a commit that referenced this issue Jul 19, 2024
@noidname01
[#3755] improvement(client-python): Support OAuth2TokenProvider for P…
213bcc9 
…ython client (#4011)

### What changes were proposed in this pull request?

* Add `OAuth2TokenProvider` and `DefaultOAuth2TokenProvider` in
`client-python`
* There are some components and tests missing because it would be a big
code change if they were also done in this PR, they will be added in the
following PRs
	- [ ] Error Handling: #4173 
	- [ ] Integration Test: #4208 
* Modify test file structure, and found issue #4136, solve it by reset
environment variable.

### Why are the changes needed?

Fix: #3755, #4136

### Does this PR introduce _any_ user-facing change?

No

### How was this patch tested?

Add UT and tested by `./gradlew clients:client-python:unittest`

---------

Co-authored-by: TimWang <[email protected]>
@noidname01 noidname01 mentioned this issue Jul 30, 2024
Closed
@xunliu
Copy link
Member

xunliu commented Aug 5, 2024

hi @noidname01 This issue has any update?

@noidname01
Copy link
Collaborator Author

@xunliu Sorry for the inactive at this issue. After some survey, this issue involve relatively large code change than others in order to implement the OAuth example in documentation with Python, so it took me longer time to plan.

@noidname01 noidname01 mentioned this issue Aug 23, 2024
Merged
@jerryshao jerryshao added the 0.7.0 Release v0.7.0 label Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@noidname01 noidname01

Labels
0.7.0 Release v0.7.0 subtask Subtasks of umbrella issue
Projects
None yet
Milestone
No milestone
3 participants
@jerryshao @xunliu @noidname01