feat(worktrees): allow CleanupFinishedAsync to filter by list
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -200,4 +200,54 @@ public class WorktreeMaintenanceServiceTests : IDisposable
|
||||
var remaining = await new WorktreeRepository(checkCtx).GetAllAsync();
|
||||
Assert.Empty(remaining);
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public async Task CleanupFinished_With_ListId_Only_Removes_That_Lists_Rows()
|
||||
{
|
||||
if (!GitAvailable) { Assert.True(true, "git not available -- skipping"); return; }
|
||||
|
||||
var repo = NewRepo();
|
||||
var git = new GitService();
|
||||
var db = NewDb();
|
||||
|
||||
var (listA, taskA) = MakeEntities(repo.RepoDir);
|
||||
var (listB, taskB) = MakeEntities(repo.RepoDir);
|
||||
|
||||
var wtA = await CreateWorktreeAsync(git, repo.RepoDir, taskA.Id);
|
||||
var wtB = await CreateWorktreeAsync(git, repo.RepoDir, taskB.Id);
|
||||
|
||||
using (var ctx = db.CreateContext())
|
||||
{
|
||||
await new ListRepository(ctx).AddAsync(listA);
|
||||
await new ListRepository(ctx).AddAsync(listB);
|
||||
var taskRepo = new TaskRepository(ctx);
|
||||
await taskRepo.AddAsync(taskA);
|
||||
await taskRepo.AddAsync(taskB);
|
||||
var wtRepo = new WorktreeRepository(ctx);
|
||||
await wtRepo.AddAsync(new WorktreeEntity
|
||||
{
|
||||
TaskId = taskA.Id, Path = wtA, BranchName = $"test/{taskA.Id}",
|
||||
BaseCommit = repo.BaseCommit, State = WorktreeState.Merged, CreatedAt = DateTime.UtcNow,
|
||||
});
|
||||
await wtRepo.AddAsync(new WorktreeEntity
|
||||
{
|
||||
TaskId = taskB.Id, Path = wtB, BranchName = $"test/{taskB.Id}",
|
||||
BaseCommit = repo.BaseCommit, State = WorktreeState.Merged, CreatedAt = DateTime.UtcNow,
|
||||
});
|
||||
}
|
||||
|
||||
var svc = new WorktreeMaintenanceService(
|
||||
db.CreateFactory(), git, NullLogger<WorktreeMaintenanceService>.Instance);
|
||||
|
||||
var result = await svc.CleanupFinishedAsync(listA.Id, CancellationToken.None);
|
||||
|
||||
Assert.Equal(1, result.Removed);
|
||||
Assert.False(Directory.Exists(wtA));
|
||||
Assert.True(Directory.Exists(wtB));
|
||||
|
||||
using var checkCtx = db.CreateContext();
|
||||
var remaining = await new WorktreeRepository(checkCtx).GetAllAsync();
|
||||
Assert.Single(remaining);
|
||||
Assert.Equal(taskB.Id, remaining[0].TaskId);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user