Skip to content

Add test coverage for ai-code-git-repo-recent-modified-files#137

Merged
tninja merged 1 commit intokang_featfrom
copilot/sub-pr-136
Jan 31, 2026
Merged

Add test coverage for ai-code-git-repo-recent-modified-files#137
tninja merged 1 commit intokang_featfrom
copilot/sub-pr-136

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 31, 2026

The ai-code-git-repo-recent-modified-files function lacked test coverage despite the repository maintaining comprehensive tests for other ai-code-git.el functions.

Changes

  • Added tests to test/test_ai-code-git.el covering:
    • Git repository context with various file structures
    • Non-Git directory context
    • Prefix argument behavior (inserting vs opening files)
    • Empty results handling

Tests follow the existing pattern in the test suite using temporary directories, mocked magit-toplevel, and cl-letf for stubbing functions.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Address feedback on command to open recently modified files Add test coverage for ai-code-git-repo-recent-modified-files Jan 31, 2026
Copilot AI requested a review from tninja January 31, 2026 18:29
@tninja tninja marked this pull request as ready for review January 31, 2026 19:30
@tninja tninja merged commit eb6dc1c into kang_feat Jan 31, 2026
1 check passed
tninja added a commit that referenced this pull request Jan 31, 2026
…ctory (#136)

* Add command to open recently modified files in Git repo or directory

* Add prefix argument support for inserting recent file name in buffer

* Update ai-code-prompt-mode file association to use expanded org path pattern

* Update ai-code-git.el

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update ai-code-git.el

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Initial plan (#137)

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants