refactor: overhaul test suite and application architecture #15

2025-07-24T16:34:14Z

eleith commented

2025-07-24 16:34:14 +00:00

This PR overhauls the testing infrastructure and application architecture for improved
testability and developer workflow.

Test Scripts:
- Adds a check-coverage.sh script with two modes:
  - --mode=packages: A package-level summary for CI.
  - --mode=functions: A detailed function-level report for local development.
- Updates Makefile with test-coverage and test-coverage-full targets.
Application Architecture:
- Moves main entry point to cmd/miniflux-digest.
- Introduces an internal/app package to manage dependency injection.
Testing Improvements:
- Increases total test coverage from 71.4% to 72.4%.
- Adds tests for MarkCategoryAsRead error handling.
- Centralizes all service and data mocks into the internal/testutil package.

This PR overhauls the testing infrastructure and application architecture for improved testability and developer workflow. * Test Scripts: * Adds a check-coverage.sh script with two modes: * --mode=packages: A package-level summary for CI. * --mode=functions: A detailed function-level report for local development. * Updates Makefile with test-coverage and test-coverage-full targets. * Application Architecture: * Moves main entry point to cmd/miniflux-digest. * Introduces an internal/app package to manage dependency injection. * Testing Improvements: * Increases total test coverage from 71.4% to 72.4%. * Adds tests for MarkCategoryAsRead error handling. * Centralizes all service and data mocks into the internal/testutil package.

eleith added 5 commits 2025-07-24 16:34:14 +00:00

feat(coverage): add flexible test coverage script 52a8959820

- Introduces `check-coverage.sh` to handle test coverage reporting
- Adds `--mode` flag to switch between package and function-level reports
- `make test-coverage` provides a package-level summary for CI
- `make test-coverage-full` provides a detailed function-level report for local development

refactor: move to cmd-based layout and dependency injection 17abae5055

- Moves the main application entry point to `cmd/miniflux-digest`
- Introduces an `internal/app` package to house the core application struct and service interfaces
- Implements dependency injection to make the application more modular and testable
- Updates the `Dockerfile` to build the new application structure

test(coverage): increase coverage for processor and app 2b52c50b23

- Adds a new test to `processor_test.go` to cover the error handling path for `MarkCategoryAsRead`
- Adds a new test file, `miniflux_client_wrapper_test.go`, to provide dedicated coverage for the `MinifluxClientWrapper`
- Increases total test coverage from 71.4% to 72.4%

refactor(tests): unify and centralize mocks 1ace8118cf

- Moves all service mocks (`MinifluxClient`, `ArchiveService`, `EmailService`) to the `internal/testutil` package
- Expands the mock data factories in `testutil` to include `NewMockCategory`, `NewMockEntry`, and `NewMockFeedIcon`
- Refactors all tests to use the new centralized mocks and data factories
- This reduces code duplication and improves the consistency and maintainability of the test suite

refactor: complete application-wide refactoring 3e3e9fb6be

- Deletes the old top-level config files
- Updates the remaining service files (`archive`, `category`, `email`, `processor`) to align with the new dependency injection structure
- Modifies the `preview-miniflux-email` script to use the new application factory

eleith merged commit 8d3deea6b6 into main

2025-07-24 16:38:44 +00:00

eleith deleted branch gemini/feature/test-refactoring

2025-07-24 16:38:44 +00:00

eleith referenced this issue from a commit

2025-07-24 16:38:45 +00:00

refactor: overhaul test suite and application architecture (#15)

eleith referenced this issue from a commit

2025-07-28 04:58:10 +00:00

refactor: overhaul test suite and application architecture (#15)

eleith referenced this issue from a commit

2025-12-10 22:24:46 +00:00