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 Happy path tests failed because unexpected alert opened during Java LS initialization #16403

Closed
2 of 23 tasks
dmytro-ndp opened this issue Mar 19, 2020 · 4 comments
Closed
2 of 23 tasks
Labels
area/qe kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.

Comments

@dmytro-ndp
Copy link
Contributor

dmytro-ndp commented Mar 19, 2020

Describe the bug

E2E Happy path tests failed being run against Single User Eclipse Che with TLS support on minikube 1.1.1 because unexpected alert with unknown text opened during Java LS initialization: https://codeready-workspaces-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/view/che-pr-tests/view/K8S/job/Che-Theia-PR-check-E2E-Happy-path-tests-against-K8S__reserved/1055/console

         ▼ Editor.waitTabFocused "PetClinicApplication.java"
          ▼ Ide.checkLsInitializationStart
          ▼ Ide.waitStatusBarContains "Activating Language Support for Java"
Known flakiness has occurred https://github.com/eclipse/che/issues/14944
          ▼ DriverHelper.reloadPage
          ▼ Ide.waitStatusBarContains "Activating Language Support for Java"
    1) Java LS initialization


  3 passing (2m)
  1 failing

  1) Language server validation
       Java LS initialization:
     UnexpectedAlertOpenError: unexpected alert open: {Alert text : }

Workspace devfile: https://raw.githubusercontent.com/chepullreq4/pr-check-files/master/che-theia/pr-676/happy-path-workspace.yaml

The alert appeared after refresh browser workaround to issue #14944 had been applied.

Che version

  • latest
  • nightly
  • other: please specify

Steps to reproduce

https://github.com/eclipse/che/blob/master/tests/e2e/tests/e2e_happy_path/HappyPath.spec.ts

Runtime

  • kubernetes (include output of kubectl version)
  • Openshift (include output of oc version)
  • minikube 1.1.1
  • minishift (include output of minishift version and oc version)
  • docker-desktop + K8S (include output of docker version and kubectl version)
  • other: (please specify)

Screenshots

screenshot-Java_LS_initialization (1)

Screencast

https://youtu.be/Is1KEz8nO5s

Installation method

  • chectl
  • che-operator
  • minishift-addon
  • I don't know

Environment

  • my computer
    • Windows
    • Linux
    • macOS
  • Cloud
    • Amazon
    • Azure
    • GCE
    • other (please specify)
  • other: please specify

Eclipse Che Logs

Additional context

@dmytro-ndp dmytro-ndp added the kind/bug Outline of a bug - must adhere to the bug report template. label Mar 19, 2020
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Mar 19, 2020
@Katka92
Copy link
Contributor

Katka92 commented Mar 20, 2020

It seems that due to workaround, the test is trying to refresh the page. At this moment, pop-up shows up asking for confirmation to leave the page. That should be fixed by setting preferences before the test run - which seems to be broken.

@dmytro-ndp dmytro-ndp added severity/blocker Causes system to crash and be non-recoverable or prevents Che developers from working on Che code. and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Mar 20, 2020
@dmytro-ndp
Copy link
Contributor Author

@dmytro-ndp dmytro-ndp added severity/P1 Has a major impact to usage or development of the system. and removed severity/blocker Causes system to crash and be non-recoverable or prevents Che developers from working on Che code. labels Mar 21, 2020
@dmytro-ndp
Copy link
Contributor Author

Not a blocker after #14944 has been fixed.

@Katka92 Katka92 removed their assignment Mar 23, 2020
@rhopp rhopp closed this as completed Apr 15, 2020
@rhopp
Copy link
Contributor

rhopp commented Apr 15, 2020

Resolved by setting preference to AskForConfirmationType.never.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/qe kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

No branches or pull requests

4 participants