Skip to content

Conversation

@max-nextcloud
Copy link
Collaborator

@max-nextcloud max-nextcloud commented Jul 27, 2025

fixes #7505

@codecov
Copy link

codecov bot commented Jul 27, 2025

Codecov Report

❌ Patch coverage is 20.00000% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 59.65%. Comparing base (9f12f21) to head (1a8f287).
⚠️ Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
src/views/RichWorkspace.vue 0.00% 16 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7501   +/-   ##
=======================================
  Coverage   59.64%   59.65%           
=======================================
  Files         495      495           
  Lines       37869    37866    -3     
  Branches     1090     1090           
=======================================
+ Hits        22588    22589    +1     
+ Misses      15174    15170    -4     
  Partials      107      107           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@max-nextcloud max-nextcloud force-pushed the chore/fix-cypress-tests branch 12 times, most recently from 196db02 to 188f706 Compare July 27, 2025 09:42
Awaiting the request for the file lead to race conditions:
* Enter directory with rich workspace, loading is triggered.
* Navigate to directory without rich workspace.
* Loading finishes - rich workspace was shown.

Also clean up some minor things:
* `this.creating` was not used at all.
* The Editor does not emit `error`.

Signed-off-by: Max <max@nextcloud.com>
@max-nextcloud max-nextcloud force-pushed the chore/fix-cypress-tests branch from 9a98dd1 to ce387af Compare July 27, 2025 13:41
The header is just hidden if enabled is false.
But for the workspace this leads to continuous requests updating the content.

Signed-off-by: Max <max@nextcloud.com>
@max-nextcloud max-nextcloud force-pushed the chore/fix-cypress-tests branch from ce387af to 1a8f287 Compare July 27, 2025 13:50
@max-nextcloud max-nextcloud changed the title debug: print hasRichWorkspace in rich workspace test fix(workspace): toggle right away and only render if needed Jul 27, 2025
@max-nextcloud max-nextcloud requested a review from skjnldsv July 27, 2025 13:52
@max-nextcloud max-nextcloud marked this pull request as ready for review July 27, 2025 13:52
@max-nextcloud max-nextcloud requested a review from mejo- as a code owner July 27, 2025 13:52
Copy link
Member

@mejo- mejo- left a comment

Choose a reason for hiding this comment

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

Nice finding! I didn't test, but I guess you did. Thanks for being so persistent here!

@mejo-
Copy link
Member

mejo- commented Jul 27, 2025

Probably also a fix worth backporting, no?

Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

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

Make sense!

@max-nextcloud
Copy link
Collaborator Author

/backport to stable31

@max-nextcloud max-nextcloud merged commit aff1125 into main Jul 27, 2025
67 of 69 checks passed
@max-nextcloud max-nextcloud deleted the chore/fix-cypress-tests branch July 27, 2025 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cypress workspace test is flaky when switching between views

4 participants