[PR #2124] [CLOSED] tests: Add a background worker test #2002

New issue

Closed

opened 2026-02-25 23:31:29 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 23:31:29 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/floccusaddon/floccus/pull/2124
Author: @marcelklehr
Created: 12/14/2025
Status: ❌ Closed

Base: develop ← Head: test/background-worker

📝 Commits (10+)

3e71cc1 tests: Add a background worker test
e457896 fix: Fix offscreen ping
93a1963 fix: Remove offscreen madness as it doesn't work
ec6bd93 test: Force sync in background worker test to reveal inconcistencies
dd2c429 fix: Try to prolong the bg worker lifetime with snakeoil
b78de77 fix: Persist state more frequently when running in a bg worker
fe0f08f fix: Make Controller proxy wait for response of syncAccount
2c9244a tests: Run benchmarks on chrome as well
0d1964f fix: Remove unnecessary code
72f5fa0 fix(Account#progressCallback): Prevent errors getting mappings after sync has finished

📊 Changes

22 files changed (+716 additions, -217 deletions)

View changed files

📝 .github/workflows/build.yml (+10 -1)
📝 .github/workflows/tests.yml (+14 -0)
➕ doiuse-report.baseline.txt (+16 -0)
📝 gulpfile.js (+2 -9)
➖ html/offscreen.html (+0 -2)
📝 manifest.chrome.json (+1 -1)
📝 package-lock.json (+294 -0)
📝 package.json (+9 -6)
➖ src/entries/offscreen.js (+0 -24)
📝 src/lib/Account.ts (+3 -28)
📝 src/lib/Controller.ts (+13 -2)
📝 src/lib/LocalTabs.ts (+22 -27)
📝 src/lib/Scanner.ts (+14 -1)
📝 src/lib/Tree.ts (+2 -1)
➖ src/lib/offscreen.ts (+0 -15)
📝 src/lib/strategies/Default.ts (+28 -27)
📝 src/lib/strategies/Unidirectional.ts (+4 -4)
➕ src/lib/yieldToEventLoop.ts (+3 -0)
📝 src/test/test.js (+156 -2)
📝 src/ui/NativeApp.vue (+11 -9)

...and 2 more files

📄 Description

No description provided

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/floccusaddon/floccus/pull/2124 **Author:** [@marcelklehr](https://github.com/marcelklehr) **Created:** 12/14/2025 **Status:** ❌ Closed **Base:** `develop` ← **Head:** `test/background-worker` --- ### 📝 Commits (10+) - [`3e71cc1`](https://github.com/floccusaddon/floccus/commit/3e71cc1f5f9415d5987a1e2c3b43afd70355d719) tests: Add a background worker test - [`e457896`](https://github.com/floccusaddon/floccus/commit/e45789667eb6755116608565d0cc44a3fbd9db86) fix: Fix offscreen ping - [`93a1963`](https://github.com/floccusaddon/floccus/commit/93a1963a8f857cba1a7a1e01f1f22f052021c816) fix: Remove offscreen madness as it doesn't work - [`ec6bd93`](https://github.com/floccusaddon/floccus/commit/ec6bd9323700d143550b292cc78e9f70c66ea4a4) test: Force sync in background worker test to reveal inconcistencies - [`dd2c429`](https://github.com/floccusaddon/floccus/commit/dd2c4292d913ff9598687e87bc7c99235c4c01b4) fix: Try to prolong the bg worker lifetime with snakeoil - [`b78de77`](https://github.com/floccusaddon/floccus/commit/b78de77a5607e3f4519bb00811d328f2c8fbb78b) fix: Persist state more frequently when running in a bg worker - [`fe0f08f`](https://github.com/floccusaddon/floccus/commit/fe0f08fc4f6069fc84c36191192ca90f6a26efec) fix: Make Controller proxy wait for response of syncAccount - [`2c9244a`](https://github.com/floccusaddon/floccus/commit/2c9244aee6ea6eaf6966a97c423396e95ae58848) tests: Run benchmarks on chrome as well - [`0d1964f`](https://github.com/floccusaddon/floccus/commit/0d1964f8d9b6f33166a06e83c85505a64d6f0d2c) fix: Remove unnecessary code - [`72f5fa0`](https://github.com/floccusaddon/floccus/commit/72f5fa08f843e572ce86d0ebc8e57a30ecc5641d) fix(Account#progressCallback): Prevent errors getting mappings after sync has finished ### 📊 Changes **22 files changed** (+716 additions, -217 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/build.yml` (+10 -1) 📝 `.github/workflows/tests.yml` (+14 -0) ➕ `doiuse-report.baseline.txt` (+16 -0) 📝 `gulpfile.js` (+2 -9) ➖ `html/offscreen.html` (+0 -2) 📝 `manifest.chrome.json` (+1 -1) 📝 `package-lock.json` (+294 -0) 📝 `package.json` (+9 -6) ➖ `src/entries/offscreen.js` (+0 -24) 📝 `src/lib/Account.ts` (+3 -28) 📝 `src/lib/Controller.ts` (+13 -2) 📝 `src/lib/LocalTabs.ts` (+22 -27) 📝 `src/lib/Scanner.ts` (+14 -1) 📝 `src/lib/Tree.ts` (+2 -1) ➖ `src/lib/offscreen.ts` (+0 -15) 📝 `src/lib/strategies/Default.ts` (+28 -27) 📝 `src/lib/strategies/Unidirectional.ts` (+4 -4) ➕ `src/lib/yieldToEventLoop.ts` (+3 -0) 📝 `src/test/test.js` (+156 -2) 📝 `src/ui/NativeApp.vue` (+11 -9) _...and 2 more files_ </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-02-25 23:31:29 +03:00

closed this issue
added the
pull-request
label

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/floccus#2002

No description provided.

Rows
Columns