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

Cypress tests are failing with current master #1072

Closed
max-nextcloud opened this issue Jan 11, 2024 · 3 comments
Closed

Cypress tests are failing with current master #1072

max-nextcloud opened this issue Jan 11, 2024 · 3 comments
Labels
1. to develop bug Something isn't working tests

Comments

@max-nextcloud
Copy link
Collaborator

Describe the bug
Cypress tests have been failing recently. In particular three tests in pages-links.spec.js:

  • Opens link with absolute path to text file in Nextcloud in viewer
  • Opens link with relative path to pdf in Nextcloud in viewer
  • Opens link with absolute path to image in Nextcloud in viewer

To Reproduce
Steps to reproduce the behavior:

  1. Visit a recent dependency PR such as fix(deps): update dependency @nextcloud/moment to ^1.3.1 #1071
  2. Check the failed cypress tests on master.
  3. Find the failure there

Expected behavior
Cypress tests should pass and dependency PRs should be merge automatically.

Screenshots

cypress-screenshot1
cypress-screenshot2
cypress-screenshot3

Server details:

  • Current Cypress master CI

Client details:

  • electron

Nextcloud log (data/nextcloud.log)

Nextcloud logs are empty (i.e. no errors) for the run in question.

@max-nextcloud max-nextcloud added bug Something isn't working tests 1. to develop labels Jan 11, 2024
@github-project-automation github-project-automation bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Office team Jan 11, 2024
@max-nextcloud
Copy link
Collaborator Author

max-nextcloud commented Jan 11, 2024

I tried to reproduce this locally with latest master, viewer and collectives but failed.

update: fails now that i am using the latest version of text.

@max-nextcloud
Copy link
Collaborator Author

I tried to bisect which commit in text introduced this but failed. Now that I got the test to fail i cannot get it to pass again. It even failed when i had the same text commit checked out as during the last passing CI run.

Since I had a local failure now i could debug it some. Looks like it the first failing test tries to open //test.png:
grafik

But the list of files after fetching the metadata only contains a file called /test.png so when the viewer tries to calculate the current index that is -1 and later the file to open is empty leading to exceptions.

@mejo- does that sound familiar to you?

@mejo-
Copy link
Member

mejo- commented Mar 6, 2024

This is fixed.

@mejo- mejo- closed this as completed Mar 6, 2024
@github-project-automation github-project-automation bot moved this from 🧭 Planning evaluation (don't pick) to ☑️ Done in 📝 Office team Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop bug Something isn't working tests
Projects
Archived in project
Development

No branches or pull requests

2 participants