refactor(homeview): extract formatDesc into testable method #165

New issue

Closed

opened 2026-07-07 05:22:02 +00:00 by the.auditor · 0 comments

the.auditor commented

2026-07-07 05:22:02 +00:00

Owner

formatDesc is a closure inside HomeView.renderActivities(), which makes it impossible to unit test the activity description formatting logic. This was highlighted by PR #164 which added issue event formatting — no tests could be written for the new logic.

Suggestion: extract formatDesc into a standalone method on HomeView (or a package-level function) so the formatting logic can be tested with table-driven tests.

Ref: PR #164

`formatDesc` is a closure inside `HomeView.renderActivities()`, which makes it impossible to unit test the activity description formatting logic. This was highlighted by PR #164 which added issue event formatting — no tests could be written for the new logic. Suggestion: extract `formatDesc` into a standalone method on `HomeView` (or a package-level function) so the formatting logic can be tested with table-driven tests. Ref: PR #164

the.auditor added the

Kind/Enhancement

Priority

Low

labels

2026-07-07 05:22:08 +00:00