-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Improve Account resource test #23117
Conversation
My brother @dinu0000 is learning programming so I suggested he work on some JHipster stuff :P |
generators/server/templates/src/test/java/package/web/rest/AccountResourceIT.java.ejs
Show resolved
Hide resolved
generators/server/templates/src/test/java/package/web/rest/AccountResourceIT_oauth2.java.ejs
Outdated
Show resolved
Hide resolved
...erver/templates/src/test/java/package/web/rest/AccountResourceIT_skipUserManagement.java.ejs
Outdated
Show resolved
Hide resolved
@dinu0000 my bad. Seems like these needs to be integration Tests as DB is needed when running the tests with |
While testing this I noticed something weird. For a reactive app with Session based auth, when you hit |
No idea. Would need to dig deeper. |
fix #21217
Please make sure the below checklist is followed for Pull Requests.
When you are still working on the PR, consider converting it to Draft (below reviewers) and adding
skip-ci
label, you can still see CI build result at your branch.