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 tests on electron package have multiple false failures. #11604

Open
farmaazon opened this issue Nov 20, 2024 · 4 comments · May be fixed by #11595
Open

E2E tests on electron package have multiple false failures. #11604

farmaazon opened this issue Nov 20, 2024 · 4 comments · May be fixed by #11595
Assignees
Labels
--bug Type: bug --regression Important: regression -ci -gui p-highest Should be completed ASAP

Comments

@farmaazon
Copy link
Contributor

There are many failures on develop branch, and also it is blocking many respectable PRs.

Also as part of this task:

  • Add test for uploading image
  • Rename tests, so the tests of GUI only will be integration tests, and electron package tests will be E2E tests.
@farmaazon farmaazon added p-highest Should be completed ASAP -ci --bug Type: bug --regression Important: regression labels Nov 20, 2024
@farmaazon farmaazon added this to the 2024-12 Release milestone Nov 20, 2024
@farmaazon farmaazon self-assigned this Nov 20, 2024
@farmaazon farmaazon moved this from ❓New to 🔧 Implementation in Issues Board Nov 20, 2024
@enso-bot
Copy link

enso-bot bot commented Nov 20, 2024

Adam Obuchowicz reports a new STANDUP for yesterday (2024-11-19):

Progress: Started debugging CI failures on electron package tests which were blocking Jaroslav. It turned out to be a huge problem, as it happens often, also on develop branch. Tested some hypotheses and started adding traces to these tests. It should be finished by 2024-11-22.

Next Day: Next day I will be working on the same task. Continue debugging

@farmaazon farmaazon added the -gui label Nov 20, 2024
@enso-bot
Copy link

enso-bot bot commented Nov 21, 2024

Adam Obuchowicz reports a new STANDUP for yesterday (2024-11-20):

Progress: Fixed the traces uploading (as it was broken). In one case the AgGrid is not loaded, in the second I discovered the log of language server is missing. Added further debugs. It should be finished by 2024-11-22.

Next Day: Next day I will be working on the same task. continue debugging

@enso-bot
Copy link

enso-bot bot commented Nov 21, 2024

Adam Obuchowicz reports a new STANDUP for today (2024-11-21):

Progress: Analyzed the debug logs. Seemingly, the project manager is launched properly and receives projectCreate and projectOpen calls - just the LS does not start. Asked Hubert for help. In the second case, it turned out importing AgGrid license hangs - probably for similar reason as icons. Rebased and checked if Sergei fixes also fix this one It should be finished by 2024-11-22.

Next Day: Next day I will be working on the same task. See if I'm right and if there are any other failures. Merge tests and add a new one. Hopefully start next task.

@enso-bot
Copy link

enso-bot bot commented Nov 22, 2024

Adam Obuchowicz reports a new STANDUP for today (2024-11-22):

Progress: The integration tests seem to be mostly working on my PR now. Add some fixes and clean-ups, and un-drafted the PR. Started adding a set of trivial changes - but some of them turned out to be not so trivial... It should be finished by 2024-11-22.

Next Day: Next free task

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
--bug Type: bug --regression Important: regression -ci -gui p-highest Should be completed ASAP
Projects
Status: 🔧 Implementation
Development

Successfully merging a pull request may close this issue.

1 participant