Skip to content

test: add unit tests for app workers #489

Open

Description

@chappie-daemon

Summary

The app-level workers have very low or zero coverage. static-mirroring-worker.ts has 0% coverage entirely.

Files to cover

  • src/app/static-mirroring-worker.ts (0%)
  • src/app/app.ts (20% statements, 0% branches)
  • src/app/worker.ts (52% statements, 9% branches)
  • src/app/maintenance-worker.ts (43% statements, 13% branches)

Out of scope

Acceptance criteria

  • Each worker has a corresponding .spec.ts under test/unit/app/
  • Worker lifecycle (start, stop, error handling) covered
  • Dependencies stubbed (DB, Redis, services)
  • npm run cover:unit passes without regressions

Activity

  1. CKodidela commented on Apr 18, 2026

    @CKodidela
    Contributor

    @cameri I would like to work on this.

  2. chappie-daemon commented on Apr 18, 2026

    @chappie-daemon
    CollaboratorAuthor

    Thanks @CKodidela! Go ahead and open a PR — we'll review it when it's ready.

  3. cameri commented on Apr 18, 2026

    @cameri
    Owner

    I assigned you another ticket for test coverage. @CKodidela

  4. YashIIT0909 commented on Apr 18, 2026

    @YashIIT0909
    Contributor

    I would like to work on this issue !

  5. chappie-daemon commented on Apr 18, 2026

    @chappie-daemon
    CollaboratorAuthor

    Thanks @YashIIT0909! This one is already assigned to @CKodidela. Feel free to check the other open issues labeled test for opportunities, or keep an eye on this one in case it becomes available.

  6. Priyanshubhartistm commented on Apr 18, 2026

    @Priyanshubhartistm
    Collaborator

    hii @cameri @phoenix-server
    ig no body working on this issues, can i work or
    its already assign then no problm
    thanku

  7. chappie-daemon commented on Apr 18, 2026

    @chappie-daemon
    CollaboratorAuthor

    Hi @Priyanshubhartistm! This one is assigned to @CKodidela. Feel free to check the other open issues labeled test — there may be opportunities there!

  8. cameri commented on Apr 18, 2026

    @cameri
    Owner

    Sorry for the confusion. Seems like @phoenix-server and myself were not aligned here.

    I'm officially assigning this ticket to @YashIIT0909 .

  9. CKodidela commented on Apr 18, 2026

    @CKodidela
    Contributor

    I've started working on this, but its ok i will work on the other issue assigned to me

  10. cameri commented on Apr 18, 2026

    @cameri
    Owner

    Apologies @CKodidela . Since you had the other ticket assigned, I figured you could work on that one instead of this one.

  11. CKodidela commented on Apr 18, 2026

    @CKodidela
    Contributor

    No worries, its really nice of you to be fair with all and let every one have a fair chance to contribute

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

Metadata

Metadata

Assignees

Labels

good first issueGood for newcomershelp wantedExtra attention is neededtestUnit or integration test improvements

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions