Skip to content

Commit 35ffb6c

Browse files
khaliqgantclaude
andcommitted
fix(mountfuse): implement WriteFilesBulk on fakeRemoteClient
The mountsync.RemoteClient interface gained WriteFilesBulk in this PR, but internal/mountfuse/fuse_test.go's fakeRemoteClient was not updated to satisfy the widened interface — Go Test CI broke across the whole module with "missing method WriteFilesBulk" for every fuse test. Add a stub that returns an empty BulkWriteResponse — mountfuse tests never exercise the bulk path. If a future test does, widen the stub with a files table + error toggle similar to writeFileErr. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 433e0bb commit 35ffb6c

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

internal/mountfuse/fuse_test.go

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,13 @@ func (f *fakeRemoteClient) WriteFile(_ context.Context, _, path, _, _, _ string)
5656
return mountsync.WriteResult{TargetRevision: "r_new"}, nil
5757
}
5858

59+
func (f *fakeRemoteClient) WriteFilesBulk(_ context.Context, _ string, _ []mountsync.BulkWriteFile) (mountsync.BulkWriteResponse, error) {
60+
// mountfuse tests never exercise the bulk-write path — the daemon uses
61+
// WriteFile directly. Return an empty response; if a future test ever
62+
// exercises bulk here, widen this stub with fields + error toggles.
63+
return mountsync.BulkWriteResponse{}, nil
64+
}
65+
5966
func (f *fakeRemoteClient) DeleteFile(_ context.Context, _, path, _ string) error {
6067
if f.deleteErr != nil {
6168
return f.deleteErr

0 commit comments

Comments
 (0)