refactor(test): replace replicated render logic with real function calls

[dlyongemallo]

No description provided.

[Copilot]

Pull request overview

Refactors functional rendering specs to stop duplicating production rendering logic and instead exercise the real internal render helpers exposed via each render module’s _test table.

Changes:

• panel_render_spec.lua: replaces replicated grouped-folder-count counting/rendering with calls to render_folder_count() and validates rendered output.
• file_history_render_spec.lua: removes locally mirrored implementations of render_stat_bar / render_file_stats / date formatting and calls the real _test helpers/formatters.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
lua/diffview/tests/functional/panel_render_spec.lua	Updates folder-count tests to invoke render_folder_count() instead of reimplementing grouped counting/rendering logic.
lua/diffview/tests/functional/file_history_render_spec.lua	Switches file-history render tests to use render._test exports rather than duplicated local implementations/constants.

Comments suppressed due to low confidence (1)

lua/diffview/tests/functional/panel_render_spec.lua:227

• Same issue here: flat:find("1") / flat:find("2") is too weak and can produce false positives. Prefer checking the segments by the git-status highlight groups (DiffviewStatusAdded / DiffviewStatusDeleted) and/or asserting the exact count+icon strings, so the test fails if ordering or status grouping regresses.

        local flat = comp:flat_text()
        -- Should contain both status groups.
        assert.truthy(flat:find("1"), "expected count for A status")
        assert.truthy(flat:find("2"), "expected count for D status")

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.