many fixes

internal/store/accounts_test.go

@@ -28,3 +28,39 @@ func TestMigratedIdempotency(t *testing.T) {
 		t.Fatal("unknown key must be false")
 	}
 }
+
+// A re-run must start from zero counters, not accumulate on top of the previous
+// run's totals — otherwise a run that skips everything still shows the prior
+// run's Copied count. ResetAccountCounters is called at the start of runAccount.
+func TestResetAccountCounters(t *testing.T) {
+	s := testStore(t)
+	ctx := context.Background()
+	epSrc, _ := s.CreateEndpoint(ctx, Endpoint{RoleLabel: "src", Host: "a", Port: 993, TLSMode: "ssl"})
+	epDst, _ := s.CreateEndpoint(ctx, Endpoint{RoleLabel: "dst", Host: "b", Port: 993, TLSMode: "ssl"})
+	taskID, _ := s.CreateTask(ctx, Task{Name: "t", SrcEndpointID: epSrc, DstEndpointID: epDst})
+	accID, _ := s.CreateAccount(ctx, Account{TaskID: taskID, SrcLogin: "u", SrcPassEnc: "x", DstLogin: "u2", DstPassEnc: "y"})
+
+	// First run: everything copied.
+	if err := s.IncAccountCounters(ctx, accID, 100, 0, 1); err != nil {
+		t.Fatalf("inc: %v", err)
+	}
+
+	// Second run starts: counters reset to zero.
+	if err := s.ResetAccountCounters(ctx, accID); err != nil {
+		t.Fatalf("reset: %v", err)
+	}
+	// Second run skips everything.
+	if err := s.IncAccountCounters(ctx, accID, 0, 100, 0); err != nil {
+		t.Fatalf("inc2: %v", err)
+	}
+
+	accs, err := s.ListAccountsByTask(ctx, taskID)
+	if err != nil || len(accs) != 1 {
+		t.Fatalf("list: %v len=%d", err, len(accs))
+	}
+	a := accs[0]
+	if a.Copied != 0 || a.Skipped != 100 || a.Errors != 0 {
+		t.Fatalf("after reset+rerun: copied=%d skipped=%d errors=%d want 0/100/0",
+			a.Copied, a.Skipped, a.Errors)
+	}
+}