Skip to content

Commit

Permalink
fixup: dont assume parsing in test
Browse files Browse the repository at this point in the history
Signed-off-by: Todd Baert <todd.baert@dynatrace.com>
  • Loading branch information
toddbaert committed Jul 5, 2024
1 parent 251d65b commit 8a7e186
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions flagd/pkg/service/flag-sync/sync-multiplexer_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,8 @@ import (
"github.com/stretchr/testify/assert"
)

const emptyConfigString = "{\"flags\":{}}"

func TestRegistration(t *testing.T) {
// given
mux, err := NewMux(getSimpleFlagStore())
Expand Down Expand Up @@ -56,7 +58,7 @@ func TestRegistration(t *testing.T) {
source: "C",
connection: make(chan payload, 1),
flagStringValidator: func(flagString string, testSource string, testName string) {
assert.Equal(t, flagString, string(emptyConfigBytes))
assert.Equal(t, flagString, emptyConfigString)
},
expectError: false,
},
Expand Down Expand Up @@ -196,5 +198,5 @@ func TestGetAllFlags(t *testing.T) {
return
}

assert.Equal(t, flags, string(emptyConfigBytes))
assert.Equal(t, flags, emptyConfigString)
}

0 comments on commit 8a7e186

Please sign in to comment.