mirror of
https://github.com/hanzoai/tasks.git
synced 2026-10-04 02:04:03 +00:00
David ReissandGitHub
c7c27e5ab1
## What changed? Use unique.Handle for dynamicconfig.Keys instead of strings. ## Why? Force keys to always be lowercase so we don't have to convert at lookup time, and also speed up lookups by interning. ## How did you test it? - [x] covered by existing tests
84 lines
2.2 KiB
Go
84 lines
2.2 KiB
Go
package dynamicconfig_test
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
"go.temporal.io/server/common/dynamicconfig"
|
|
)
|
|
|
|
func TestMemoryClient(t *testing.T) {
|
|
c := dynamicconfig.NewMemoryClient()
|
|
k := dynamicconfig.MakeKey("key")
|
|
|
|
// plain override
|
|
assert.Nil(t, c.GetValue(k))
|
|
remove := c.OverrideValue(k, 123)
|
|
assert.Equal(t, []dynamicconfig.ConstrainedValue{{Value: 123}}, c.GetValue(k))
|
|
remove()
|
|
assert.Nil(t, c.GetValue(k))
|
|
|
|
// two levels, pop in correct order
|
|
remove1 := c.OverrideValue(k, 123)
|
|
remove2 := c.OverrideValue(k, 456)
|
|
assert.Equal(t, []dynamicconfig.ConstrainedValue{{Value: 456}}, c.GetValue(k))
|
|
remove2()
|
|
assert.Equal(t, []dynamicconfig.ConstrainedValue{{Value: 123}}, c.GetValue(k))
|
|
remove1()
|
|
assert.Nil(t, c.GetValue(k))
|
|
|
|
// three levels, pop in wrong order
|
|
remove1 = c.OverrideValue(k, 123)
|
|
remove2 = c.OverrideValue(k, 456)
|
|
remove3 := c.OverrideValue(k, 789)
|
|
assert.Equal(t, []dynamicconfig.ConstrainedValue{{Value: 789}}, c.GetValue(k))
|
|
remove2()
|
|
assert.Equal(t, []dynamicconfig.ConstrainedValue{{Value: 789}}, c.GetValue(k))
|
|
remove3()
|
|
assert.Equal(t, []dynamicconfig.ConstrainedValue{{Value: 123}}, c.GetValue(k))
|
|
remove1()
|
|
remove3() // no-op
|
|
remove2() // no-op
|
|
assert.Nil(t, c.GetValue(k))
|
|
}
|
|
|
|
func TestMemoryClientSubscriptions(t *testing.T) {
|
|
c := dynamicconfig.NewMemoryClient()
|
|
k := dynamicconfig.MakeKey("key")
|
|
|
|
calls := 0
|
|
|
|
c.Subscribe(func(changed map[dynamicconfig.Key][]dynamicconfig.ConstrainedValue) {
|
|
calls++
|
|
assert.Contains(t, changed, k)
|
|
switch calls {
|
|
case 1:
|
|
assert.Equal(t, []dynamicconfig.ConstrainedValue{{Value: 123}}, changed[k])
|
|
case 2:
|
|
assert.Nil(t, changed[k])
|
|
case 3:
|
|
assert.Equal(t, []dynamicconfig.ConstrainedValue{{Value: 456}}, changed[k])
|
|
case 4:
|
|
assert.Equal(t, []dynamicconfig.ConstrainedValue{{Value: 789}}, changed[k])
|
|
case 5:
|
|
assert.Equal(t, []dynamicconfig.ConstrainedValue{{Value: 456}}, changed[k])
|
|
case 6:
|
|
assert.Nil(t, changed[k])
|
|
}
|
|
})
|
|
|
|
remove := c.OverrideValue(k, 123)
|
|
assert.Equal(t, 1, calls)
|
|
remove()
|
|
assert.Equal(t, 2, calls)
|
|
|
|
remove1 := c.OverrideValue(k, 456)
|
|
assert.Equal(t, 3, calls)
|
|
remove2 := c.OverrideValue(k, 789)
|
|
assert.Equal(t, 4, calls)
|
|
remove2()
|
|
assert.Equal(t, 5, calls)
|
|
remove1()
|
|
assert.Equal(t, 6, calls)
|
|
}
|