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 Java test runner support to Che #619

Merged
merged 4 commits into from
Sep 11, 2020
Merged

Add Java test runner support to Che #619

merged 4 commits into from
Sep 11, 2020

Conversation

ericwill
Copy link
Contributor

Re-upload of #602 in order to get CI to pass.

Signed-off-by: Thomas Mäder <tmader@redhat.com>
Signed-off-by: Thomas Mäder <tmader@redhat.com>
Signed-off-by: Thomas Mäder <tmader@redhat.com>
Signed-off-by: Thomas Mäder <tmader@redhat.com>
@benoitf
Copy link
Contributor

benoitf commented Sep 11, 2020

you could have just pushed to previous PR :-)

I think there is still a problem when the java test runner is added

I saw that in the video of happy path test, there was 'Activating test runner...' at the bottom but I didn't investigate much

@che-bot
Copy link
Contributor

che-bot commented Sep 11, 2020

Happy path tests passed.

@benoitf
Copy link
Contributor

benoitf commented Sep 11, 2020

let's merge it :D

@ericwill
Copy link
Contributor Author

I saw that in the video of happy path test, there was 'Activating test runner...' at the bottom but I didn't investigate much

It's normal, Thomas made a demo video of it working fine.

@ericwill ericwill merged commit ef3341f into eclipse-che:master Sep 11, 2020
@che-bot che-bot added this to the 7.19 milestone Sep 11, 2020
monaka pushed a commit to PizzaFactory/che-plugin-registry that referenced this pull request Sep 18, 2020
* Add testrunner to Java related plugins

Signed-off-by: Thomas Mäder <tmader@redhat.com>

* Fix plugin name

Signed-off-by: Thomas Mäder <tmader@redhat.com>

* Deprecate previous version

Signed-off-by: Thomas Mäder <tmader@redhat.com>

* Remove extra newline

Signed-off-by: Thomas Mäder <tmader@redhat.com>

Co-authored-by: Thomas Mäder <tmader@redhat.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.

4 participants