Skip to content

feat: stop lib background tasks#301

Merged
r4mmer merged 6 commits intomasterfrom
feat/stop-lib-background-tasks
Oct 10, 2025
Merged

feat: stop lib background tasks#301
r4mmer merged 6 commits intomasterfrom
feat/stop-lib-background-tasks

Conversation

@r4mmer
Copy link
Copy Markdown
Member

@r4mmer r4mmer commented Oct 9, 2025

Motivation

The wallet-lib start a background task that prevents jest from finishing properly.
The lib (after v2.8.X) exports a method to stop this task so we should use it.

Acceptance Criteria

  • Bump the lib to 2.8.3
  • Call stopGLLBackgroundTask before all tests

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged
  • Make sure either the unit tests and/or the QA tests are capable of testing the new features
  • Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

@r4mmer r4mmer self-assigned this Oct 9, 2025
@r4mmer r4mmer moved this from Todo to In Progress (Done) in Hathor Network Oct 9, 2025
pedroferreira1
pedroferreira1 previously approved these changes Oct 10, 2025
Comment thread packages/wallet-service/tests/pushSendNotificationToDevice.test.ts
@@ -0,0 +1,3 @@
import { stopGLLBackgroundTask } from '@hathor/wallet-lib';
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Missing hathor header

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Done

@@ -0,0 +1,6 @@
import { stopGLLBackgroundTask } from '@hathor/wallet-lib';
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Missing hathor header

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Done

@pedroferreira1 pedroferreira1 moved this from In Progress (Done) to In Review (Done) in Hathor Network Oct 10, 2025
@r4mmer r4mmer moved this from In Review (Done) to In Progress (Done) in Hathor Network Oct 10, 2025
@github-project-automation github-project-automation Bot moved this from In Progress (Done) to In Review (WIP) in Hathor Network Oct 10, 2025
@r4mmer r4mmer merged commit b2fcd6b into master Oct 10, 2025
1 check passed
@github-project-automation github-project-automation Bot moved this from In Review (WIP) to Waiting to be deployed in Hathor Network Oct 10, 2025
@r4mmer r4mmer deleted the feat/stop-lib-background-tasks branch October 10, 2025 22:25
@r4mmer r4mmer moved this from Waiting to be deployed to Done in Hathor Network Oct 29, 2025
This was referenced Nov 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants