Some Kanban tests are flaky #1877

New Issue

Closed

opened 2021-11-24 11:02:13 +00:00 by dpschen · 8 comments

dpschen commented

2021-11-24 11:02:13 +00:00

Member

I see these tests fail from time to time and have to restart the CI to make them pass.

I see these tests fail from time to time and have to restart the CI to make them pass.

Screenshot 2021-11-24 at 11.59.41.png

214 KiB

Screenshot 2021-11-24 at 11.59.41.png

dpschen added the

label 2021-11-24 11:02:13 +00:00

konrad commented

2021-11-24 13:31:16 +00:00

Owner

That's kind of a known issue with cypress. I think there's a workaround command in there somewhere.

That's kind of a known issue with cypress. I think there's a workaround command in there somewhere.

dpschen commented

2021-11-26 13:13:24 +00:00

Author

Member

You mean the getSettled command?

You mean the `getSettled` command?

konrad commented

2021-11-26 13:24:45 +00:00

Owner

No I mean the This element is detached from the dom issue

No I mean the `This element is detached from the dom` issue

dpschen commented

2021-11-26 13:26:16 +00:00

Author

Member

Ahh and what do you mean then by: [...] in there somewhere

Ahh and what do you mean then by: <cite>[...] in there somewhere</cite>

konrad commented

2021-11-26 13:33:09 +00:00

Owner

I think there's a command or some other workaround in the cypress commands (on my phone right now, can't really search)

I've ranted on twitter about this before and got a few suggestions out of it: https://twitter.com/kolaente/status/1399254966343176192

Maybe those are worth a re-visit.

https://kolaente.dev/vikunja/frontend/src/branch/main/cypress/support/commands.js#L10

I think there's a command or some other workaround in the cypress commands (on my phone right now, can't really search) I've ranted on twitter about this before and got a few suggestions out of it: https://twitter.com/kolaente/status/1399254966343176192 Maybe those are worth a re-visit. https://kolaente.dev/vikunja/frontend/src/branch/main/cypress/support/commands.js#L10

dpschen commented

2021-11-26 13:35:28 +00:00

Author

Member

I also saw that the new create-vue comes with cypress support built in. Maybe there's something we can learn.. Will check that.

I also saw that the new [create-vue](https://github.com/vuejs/create-vue) comes with cypress support built in. Maybe there's something we can learn.. Will check that.

dpschen commented

2021-11-30 20:30:07 +00:00

Author

Member

I think I could make these flaky tests working via getSettled in vikunja/frontend#816

That solves the issue for me. Tests need to work and not to be perfect =)

I think I could make these flaky tests working via `getSettled` in https://kolaente.dev/vikunja/frontend/pulls/816 That solves the issue for me. Tests need to work and not to be perfect =)

konrad commented

2021-11-30 20:31:06 +00:00

Owner

Tests need to work and not to be perfect =)

Oh definitely :)

> Tests need to work and not to be perfect =) Oh definitely :)

dpschen closed this issue

2022-11-22 11:45:51 +00:00

konrad referenced this issue from a commit

2024-02-07 14:00:33 +00:00

chore(deps): update dependency vue-tsc to v0.34.11 (#1877)

Sign in to join this conversation.

No Label

needs reproduction

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: vikunja/vikunja#1877

No description provided.