Skip to content

Add stop/start methods to CleaningService for testing #10

@dennisthemenacing

Description

@dennisthemenacing

The starts its cleanup monitor immediately on construction, which makes unit testing awkward — the cleaner runs concurrently during tests even when mocked intervals are used.

Proposed addition:

Add and methods to :

  • — clears the timer and stops the monitor loop
  • — restarts the monitor (re-runs )

This lets tests pause the cleaner during assertions and resume it afterward. The constructor already calls if intervals are valid, and already exists in — it just needs a corresponding .

Files affected:

  • — add method
  • — update the interface
  • — add integration tests for stop/start behavior

This is a small, self-contained change with an existing method () as a reference.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions