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

address failing document_spec #518

Draft
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

duncdrum
Copy link
Contributor

@duncdrum duncdrum commented Sep 15, 2021

renaming documents is broken beyond testsuite fixes. We should not let the green in this one blind us that create, save, delete and renaming documents is bugged. The testsuite as is after this is merged, runs what can be run, and contains inlined functions and comments about what should be run, and why it currently can't. These all need to be addressed one way or the other, not by hacking away at the testsuite, but by code fixes.

close #400

see #412 - #414, #424, #430, #433, #513, #516 - #527

@cla-bot cla-bot bot added the cla-signed label Sep 15, 2021
@duncdrum duncdrum changed the title Knock knock ci address failing document_spec Sep 15, 2021
@cmccrorie cmccrorie changed the title address failing document_spec address failing document_spec Sep 16, 2021
@duncdrum duncdrum marked this pull request as ready for review September 23, 2021 14:46
@duncdrum duncdrum marked this pull request as draft September 23, 2021 15:06
@duncdrum
Copy link
Contributor Author

something is still not right, nothing gets rendered on CI, whereas this is what electron headless looks on my system.
Screenshot 2021-09-23 at 17 06 27

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[cypress] beforeEach and before hooks need refactoring
1 participant