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

[E2E] Add test for Java Vertx workspace. #14558

Merged
merged 4 commits into from
Sep 19, 2019

Conversation

Katka92
Copy link
Contributor

@Katka92 Katka92 commented Sep 16, 2019

What does this PR do?

Add test for Java Vertx default worskpace with example project.
Test will validate:

  • Workspace creation via UI
  • Language server validation
    • Expand project and open file in editor
    • Java LS initialization
    • Suggestion invoking
    • Error highlighting
    • Autocomplete
    • Codenavigation
  • Validation of project build
  • Close the terminal tasks
  • Stop workspace
  • Delete workspace

Test can be run via npm run test-java-vertx.

EDIT: The purpose of this test is to test default example stacks with their example project that user can create just by selecting it in UI.

Signed-off-by: kkanova <kkanova@redhat.com>
@che-bot che-bot added the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Sep 16, 2019
@che-bot
Copy link
Contributor

che-bot commented Sep 16, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

  • build details
  • "che-server" docker image: maxura/che-server:14558

@che-bot
Copy link
Contributor

che-bot commented Sep 16, 2019

E2E tests of Eclipse Che Multiuser on OCP has failed:

@Katka92
Copy link
Contributor Author

Katka92 commented Sep 16, 2019

ci-test

@Katka92
Copy link
Contributor Author

Katka92 commented Sep 16, 2019

ci-build

@che-bot
Copy link
Contributor

che-bot commented Sep 16, 2019

E2E tests of Eclipse Che Multiuser on OCP has failed:

@che-bot
Copy link
Contributor

che-bot commented Sep 18, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

  • build details
  • "che-server" docker image: maxura/che-server:14558

@che-bot
Copy link
Contributor

che-bot commented Sep 18, 2019

E2E tests of Eclipse Che Multiuser on OCP has been successful:

@che-bot
Copy link
Contributor

che-bot commented Sep 18, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

  • build details
  • "che-server" docker image: maxura/che-server:14558

@che-bot
Copy link
Contributor

che-bot commented Sep 18, 2019

E2E tests of Eclipse Che Multiuser on OCP has failed:

@Katka92
Copy link
Contributor Author

Katka92 commented Sep 19, 2019

ci-test

@che-bot
Copy link
Contributor

che-bot commented Sep 19, 2019

E2E tests of Eclipse Che Multiuser on OCP has been successful:

@dmytro-ndp
Copy link
Contributor

@Katka92: If there is an issue which this PR fixes?

@dmytro-ndp
Copy link
Contributor

@Katka92: do we need special devfile to create test workspace?

@Katka92
Copy link
Contributor Author

Katka92 commented Sep 19, 2019

@dmytro-ndp There is an issue in Rh-Che [1] and we think that this logic should be placed in upstream as there is nothing Rh-Che specific in a test.
The purpose of this test is to test default example stacks with their example project, so there is no special devfile used. We plan to extend our test suite to be able to test majority of workspaces that user can create just by selecting them in UI.

[1] redhat-developer/rh-che#1567

@dmytro-ndp
Copy link
Contributor

@Katka92: thanks for the answer! IMHO, the answer could be added to description of PR to explain what it is for.
I have just also realized that file mocha-java-vertx.opts which is required to run the test suite is absent. Is it expected?

@Katka92
Copy link
Contributor Author

Katka92 commented Sep 19, 2019

@dmytro-ndp Good catch! I forgot to add this new file, thank you very much! I've also updated PR description.

Copy link
Contributor

@dmytro-ndp dmytro-ndp left a comment

Choose a reason for hiding this comment

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

Looks good.

@che-bot
Copy link
Contributor

che-bot commented Sep 19, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

  • build details
  • "che-server" docker image: maxura/che-server:14558

@che-bot
Copy link
Contributor

che-bot commented Sep 19, 2019

E2E tests of Eclipse Che Multiuser on OCP has been successful:

@Katka92 Katka92 merged commit 496cdb8 into eclipse-che:master Sep 19, 2019
@Katka92 Katka92 deleted the e2e_add_javavertx_test branch March 3, 2020 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants