7e4ea16149
The bulk of this commit is moving the LeaderRoutineManager from the agent/consul package into its own package: lib/gort. It also got a renaming and its Start method now requires a context. Requiring that context required updating a whole bunch of other places in the code.
102 lines
2.6 KiB
Go
102 lines
2.6 KiB
Go
package routine
|
|
|
|
import (
|
|
"context"
|
|
"sync/atomic"
|
|
"testing"
|
|
|
|
"github.com/hashicorp/consul/sdk/testutil"
|
|
"github.com/hashicorp/consul/sdk/testutil/retry"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
func TestManager(t *testing.T) {
|
|
t.Parallel()
|
|
var runs uint32
|
|
var running uint32
|
|
mgr := NewManager(testutil.Logger(t))
|
|
|
|
run := func(ctx context.Context) error {
|
|
atomic.StoreUint32(&running, 1)
|
|
defer atomic.StoreUint32(&running, 0)
|
|
atomic.AddUint32(&runs, 1)
|
|
<-ctx.Done()
|
|
return nil
|
|
}
|
|
|
|
// IsRunning on unregistered service should be false
|
|
require.False(t, mgr.IsRunning("not-found"))
|
|
|
|
// start
|
|
require.NoError(t, mgr.Start(context.Background(), "run", run))
|
|
require.True(t, mgr.IsRunning("run"))
|
|
retry.Run(t, func(r *retry.R) {
|
|
require.Equal(r, uint32(1), atomic.LoadUint32(&runs))
|
|
require.Equal(r, uint32(1), atomic.LoadUint32(&running))
|
|
})
|
|
doneCh := mgr.Stop("run")
|
|
require.NotNil(t, doneCh)
|
|
<-doneCh
|
|
|
|
// ensure the background go routine was actually cancelled
|
|
retry.Run(t, func(r *retry.R) {
|
|
require.Equal(r, uint32(1), atomic.LoadUint32(&runs))
|
|
require.Equal(r, uint32(0), atomic.LoadUint32(&running))
|
|
})
|
|
|
|
// restart and stop
|
|
require.NoError(t, mgr.Start(context.Background(), "run", run))
|
|
retry.Run(t, func(r *retry.R) {
|
|
require.Equal(r, uint32(2), atomic.LoadUint32(&runs))
|
|
require.Equal(r, uint32(1), atomic.LoadUint32(&running))
|
|
})
|
|
|
|
doneCh = mgr.Stop("run")
|
|
require.NotNil(t, doneCh)
|
|
<-doneCh
|
|
|
|
retry.Run(t, func(r *retry.R) {
|
|
require.Equal(r, uint32(0), atomic.LoadUint32(&running))
|
|
})
|
|
|
|
// start with a context
|
|
ctx, cancel := context.WithCancel(context.Background())
|
|
require.NoError(t, mgr.Start(ctx, "run", run))
|
|
cancel()
|
|
|
|
// The function should exit of its own accord due to the parent
|
|
// context being canceled
|
|
retry.Run(t, func(r *retry.R) {
|
|
require.Equal(r, uint32(3), atomic.LoadUint32(&runs))
|
|
require.Equal(r, uint32(0), atomic.LoadUint32(&running))
|
|
// the task should automatically set itself to not running if
|
|
// it exits early
|
|
require.False(r, mgr.IsRunning("run"))
|
|
})
|
|
}
|
|
|
|
func TestManager_StopAll(t *testing.T) {
|
|
t.Parallel()
|
|
var runs uint32
|
|
var running uint32
|
|
mgr := NewManager(testutil.Logger(t))
|
|
|
|
run := func(ctx context.Context) error {
|
|
atomic.StoreUint32(&running, 1)
|
|
defer atomic.StoreUint32(&running, 0)
|
|
atomic.AddUint32(&runs, 1)
|
|
<-ctx.Done()
|
|
return nil
|
|
}
|
|
|
|
require.NoError(t, mgr.Start(context.Background(), "run1", run))
|
|
require.NoError(t, mgr.Start(context.Background(), "run2", run))
|
|
|
|
mgr.StopAll()
|
|
|
|
retry.Run(t, func(r *retry.R) {
|
|
require.False(r, mgr.IsRunning("run1"))
|
|
require.False(r, mgr.IsRunning("run2"))
|
|
})
|
|
}
|