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

Introduce jest test #5

Merged
merged 18 commits into from
Jan 10, 2024
Merged

Introduce jest test #5

merged 18 commits into from
Jan 10, 2024

Conversation

nonumpa
Copy link
Member

@nonumpa nonumpa commented Nov 30, 2023

  1. Test hocuspocus elasticsearch and snapshot extension
    • prepare test DB
    • use hocuspocus server/provider to the test extension
  2. Use github action to test on push and pull request

@nonumpa nonumpa requested a review from MrOrz November 30, 2023 05:34
nonumpa and others added 5 commits November 30, 2023 20:45
Co-authored-by: Johnson Liang <[email protected]>
- Fix hocuspocus open handle by destroying server in each test
- Fix (WARN) Define `ts-jest` config under `globals` is deprecated
@nonumpa nonumpa marked this pull request as ready for review December 21, 2023 14:42
@nonumpa nonumpa requested a review from MrOrz December 27, 2023 08:12
- Remove wait 1s statement by using `syncedNewHocuspocusProvider` which resolve and return provider after onSynced
- Update 'log error' test to async
- Update comments
@MrOrz MrOrz merged commit 7b91454 into master Jan 10, 2024
2 checks passed
@MrOrz MrOrz deleted the test branch January 10, 2024 11:18
nonumpa added a commit that referenced this pull request Jan 23, 2024
…site

It's because we upgraded hocuspocus/server to 2.7.1 in package-lock [by accident](#5).
And [newer version](ueberdosis/hocuspocus#635) adds a new MessageType which old client does not know
nonumpa added a commit that referenced this pull request Jan 23, 2024
…site

It's because we upgraded hocuspocus/server to 2.7.1 in package-lock [by accident](#5).
And newer version adds a new MessageType which old client does not know
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.

2 participants