From 6950c2b107cfa35254db7fdbdc232ae1c76512e0 Mon Sep 17 00:00:00 2001 From: Weizhen Wang Date: Thu, 18 May 2023 15:13:36 +0800 Subject: [PATCH] *: fix panic in the TestStateSyncerSimple (#43944) close pingcap/tidb#43943 --- ddl/syncer/state_syncer_test.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ddl/syncer/state_syncer_test.go b/ddl/syncer/state_syncer_test.go index 584de2889e..107e0d0e36 100644 --- a/ddl/syncer/state_syncer_test.go +++ b/ddl/syncer/state_syncer_test.go @@ -62,9 +62,10 @@ func TestStateSyncerSimple(t *testing.T) { WithLease(testLease), WithInfoCache(ic), ) - go func() { + var wg util.WaitGroupWrapper + wg.Run(func() { require.NoError(t, d.OwnerManager().CampaignOwner()) - }() + }) defer d.OwnerManager().Cancel() // TODO: We can remove it when we call it in newDDL. require.NoError(t, d.StateSyncer().Init(ctx)) @@ -77,7 +78,6 @@ func TestStateSyncerSimple(t *testing.T) { require.Equal(t, stateInfo, respState) require.False(t, d.StateSyncer().IsUpgradingState()) // for watchCh - var wg util.WaitGroupWrapper var checkErr string stateInfo.State = syncer.StateUpgrading stateInfoByte, err := stateInfo.Marshal()