From c1ebfb7e0440648428ea8304c4aedf87bb24db20 Mon Sep 17 00:00:00 2001 From: nielash Date: Tue, 22 Jul 2025 20:13:30 -0400 Subject: [PATCH] sync: fix testLoggerVsLsf checking wrong fs Before this change, two tests (TestServerSideCopyOverSelf and TestServerSideMoveOverSelf) were checking the wrong Fs in the call to testLoggerVsLsf. This fixes it by making sure we are testing the same two Fs's we synced. --- fs/sync/sync_test.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/fs/sync/sync_test.go b/fs/sync/sync_test.go index 8853d7203..ae78f1c8a 100644 --- a/fs/sync/sync_test.go +++ b/fs/sync/sync_test.go @@ -658,7 +658,7 @@ func TestServerSideCopyOverSelf(t *testing.T) { ctx = predictDstFromLogger(ctx) err = CopyDir(ctx, FremoteCopy, r.Fremote, false) require.NoError(t, err) - testLoggerVsLsf(ctx, r.Fremote, r.Flocal, operations.GetLoggerOpt(ctx).JSON, t) + testLoggerVsLsf(ctx, FremoteCopy, r.Fremote, operations.GetLoggerOpt(ctx).JSON, t) fstest.CheckItems(t, FremoteCopy, file1) file2 := r.WriteObject(ctx, "sub dir/hello world", "hello world again", t2) @@ -667,7 +667,7 @@ func TestServerSideCopyOverSelf(t *testing.T) { ctx = predictDstFromLogger(ctx) err = CopyDir(ctx, FremoteCopy, r.Fremote, false) require.NoError(t, err) - testLoggerVsLsf(ctx, r.Fremote, r.Flocal, operations.GetLoggerOpt(ctx).JSON, t) + testLoggerVsLsf(ctx, FremoteCopy, r.Fremote, operations.GetLoggerOpt(ctx).JSON, t) fstest.CheckItems(t, FremoteCopy, file2) } @@ -703,7 +703,7 @@ func TestServerSideMoveOverSelf(t *testing.T) { ctx = predictDstFromLogger(ctx) err = CopyDir(ctx, FremoteCopy, r.Fremote, false) require.NoError(t, err) - testLoggerVsLsf(ctx, r.Fremote, r.Flocal, operations.GetLoggerOpt(ctx).JSON, t) + testLoggerVsLsf(ctx, FremoteCopy, r.Fremote, operations.GetLoggerOpt(ctx).JSON, t) fstest.CheckItems(t, FremoteCopy, file1) file2 := r.WriteObject(ctx, "sub dir/hello world", "hello world again", t2)