Skip to content

load testing resources from classpath path #1823

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

Merged
merged 5 commits into from
Mar 9, 2023

Conversation

mwiede
Copy link
Contributor

@mwiede mwiede commented Feb 22, 2022

I was importing the test-jar of com.github.docker-java:docker-java to reuse the test classes. I found the problem, that this was not possible until the files used in testing are loading from classpath instead of source path.

@bsideup bsideup modified the milestones: 3.2.13, next Feb 22, 2022
@bsideup
Copy link
Member

bsideup commented Feb 22, 2022

Hi @mwiede! Sorry, it looks like tests were unstable. Could you please with with master (just fixed it there). Thank you!

@stale
Copy link

stale bot commented May 25, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot closed this Jun 28, 2022
@mwiede
Copy link
Contributor Author

mwiede commented Sep 13, 2022

@eddumelendez could you please consider reopen and merge this? Thanks

@eddumelendez eddumelendez reopened this Sep 13, 2022
@stale stale bot closed this Oct 15, 2022
@eddumelendez eddumelendez reopened this Oct 15, 2022
@eddumelendez eddumelendez removed this from the next milestone Nov 23, 2022
@eddumelendez eddumelendez requested a review from a team as a code owner March 8, 2023 23:51
@eddumelendez eddumelendez merged commit 7ce35f4 into docker-java:main Mar 9, 2023
@eddumelendez
Copy link
Member

Thanks for your contribution, @mwiede ! and sorry about the delay.

@eddumelendez eddumelendez added this to the next milestone Mar 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants