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

inject and import dto mappers when used for an entity in tests #123

Merged

Conversation

atomfrede
Copy link
Member

inject mapstruct mappers as they are used in the tests

closes #105

@atomfrede atomfrede added the bug 🐞 Something isn't working label Jul 20, 2020
@agilob
Copy link
Contributor

agilob commented Jul 21, 2020

Thanks for picking this up, I was planning to do it next weekend :)

I would like to point out that importing and injecting isn't end of the problem here, the dto in tests aren't used

memberDTO should be used in the client.exchange method

image

@atomfrede
Copy link
Member Author

It's this one right? #104

Just wanted to keep them separate. This pr fixes just the compilation error when using dtos.

@agilob
Copy link
Contributor

agilob commented Jul 22, 2020

hah, yes, I didnt remember I've opened issue for that 👍

@atomfrede atomfrede merged commit eedd5c2 into jhipster:master Jul 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mapper beans aren't autowired in ResourceIT tests
2 participants