Skip to content

Commit

Permalink
fix: uncomment test
Browse files Browse the repository at this point in the history
Signed-off-by: Guilhem Barthés <[email protected]>
  • Loading branch information
guilhem-barthes committed Feb 8, 2024
1 parent b174f05 commit 1cc9174
Showing 1 changed file with 10 additions and 12 deletions.
22 changes: 10 additions & 12 deletions e2e/datasample_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -34,21 +34,19 @@ func TestRegisterDataSample(t *testing.T) {
e2erequire.ProtoEqual(t, registeredSample, eventSample)
}

// func TestQueryDatasamplesUnfiltered(t *testing.T) {
// appClient := factory.NewTestClient()

// appClient.RegisterDataManager(client.DefaultDataManagerOptions())
// appClient.RegisterDataSample(client.DefaultDataSampleOptions().WithKeyRef("ds1"))
// appClient.RegisterDataSample(client.DefaultDataSampleOptions().WithKeyRef("ds2"))
// appClient.RegisterDataSample(client.DefaultDataSampleOptions().WithKeyRef("ds3"))
func TestQueryDatasamplesUnfiltered(t *testing.T) {
appClient := factory.NewTestClient()

// resp := appClient.QueryDataSamples("", 1000, nil)
appClient.RegisterDataManager(client.DefaultDataManagerOptions())
appClient.RegisterDataSample(client.DefaultDataSampleOptions().WithKeyRef("ds1"))
appClient.RegisterDataSample(client.DefaultDataSampleOptions().WithKeyRef("ds2"))
appClient.RegisterDataSample(client.DefaultDataSampleOptions().WithKeyRef("ds3"))

// print
resp := appClient.QueryDataSamples("", 1000, nil)

// require.Greater(t, len(resp.DataSamples), 3, "QueryDataSamples response should contain at least 3 datasamples")
// e2erequire.ContainsKeys(t, true, appClient, resp.DataSamples, "ds1", "ds2", "ds3")
// }
require.Greater(t, len(resp.DataSamples), 3, "QueryDataSamples response should contain at least 3 datasamples")
e2erequire.ContainsKeys(t, true, appClient, resp.DataSamples, "ds1", "ds2", "ds3")
}

func TestQueryDatasamplesFiltered(t *testing.T) {
appClient := factory.NewTestClient()
Expand Down

0 comments on commit 1cc9174

Please sign in to comment.