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.

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

Add unit tests to hashicorpvault handler #5189

Merged
merged 7 commits into from
Nov 22, 2023

Conversation

dttung2905
Copy link
Contributor

@dttung2905 dttung2905 commented Nov 16, 2023

Provide a description of what has been changed

Checklist

Fixes #

Relates to #5195

@dttung2905 dttung2905 requested a review from a team as a code owner November 16, 2023 23:00
Copy link

Thank you for your contribution! 🙏 We will review your PR as soon as possible.

While you are waiting, make sure to:

Learn more about:

@dttung2905 dttung2905 force-pushed the feat-add-unittest-hashicorp-vault branch from d907cdf to 842d536 Compare November 20, 2023 22:15
@dttung2905 dttung2905 changed the title [WIP] Add tests to hashicorpvault handler Add unit tests to hashicorpvault handler Nov 20, 2023
Copy link
Member

@zroubalik zroubalik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good, thanks for the contribution.
Only a nit in Changelog

CHANGELOG.md Outdated Show resolved Hide resolved
@dttung2905 dttung2905 force-pushed the feat-add-unittest-hashicorp-vault branch from c0f9f9f to 1848738 Compare November 21, 2023 15:13
Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
@dttung2905 dttung2905 force-pushed the feat-add-unittest-hashicorp-vault branch from ac52256 to b13d716 Compare November 22, 2023 08:38
Copy link
Member

@zroubalik zroubalik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, great job!

@zroubalik
Copy link
Member

zroubalik commented Nov 22, 2023

/run-e2e hashicorp
Update: You can check the progress here

@zroubalik zroubalik enabled auto-merge (squash) November 22, 2023 15:21
@zroubalik zroubalik merged commit 6b8bc6a into kedacore:main Nov 22, 2023
19 checks passed
yoongon pushed a commit to yoongon/keda that referenced this pull request Nov 24, 2023
Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
Signed-off-by: Yoon Park <yoongon.park@gmail.com>
toniiiik pushed a commit to toniiiik/keda that referenced this pull request Jan 15, 2024
Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
Signed-off-by: anton.lysina <alysina@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants