Skip to content

Commit

Permalink
fix(dynamoping): adjust tests for alternator with enforced authentica…
Browse files Browse the repository at this point in the history
…tion

Ref #4036
  • Loading branch information
Michal-Leszczynski committed Sep 16, 2024
1 parent 450d9c1 commit c72e68d
Showing 1 changed file with 9 additions and 3 deletions.
12 changes: 9 additions & 3 deletions pkg/ping/dynamoping/dynamoping_integration_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -7,17 +7,20 @@ package dynamoping

import (
"context"
"github.com/scylladb/scylla-manager/v3/pkg/testutils/testconfig"
"testing"
"time"

"github.com/pkg/errors"
"github.com/scylladb/scylla-manager/v3/pkg/testutils/testconfig"

_ "github.com/scylladb/scylla-manager/v3/pkg/testutils"
)

func TestPingIntegration(t *testing.T) {
config := Config{
Addr: "http://" + testconfig.ManagedClusterHost() + ":8000",
Timeout: 250 * time.Millisecond,
Addr: "http://" + testconfig.ManagedClusterHost() + ":8000",
Timeout: 250 * time.Millisecond,
RequiresAuthentication: true,
}

t.Run("simple", func(t *testing.T) {
Expand All @@ -31,6 +34,9 @@ func TestPingIntegration(t *testing.T) {
t.Run("query", func(t *testing.T) {
d, err := QueryPing(context.Background(), config)
if err != nil {
if errors.Is(err, ErrAlternatorQueryPingNotSupported) {
t.Skip(ErrAlternatorQueryPingNotSupported)
}
t.Error(err)
}
t.Logf("QueryPing() = %s", d)
Expand Down

0 comments on commit c72e68d

Please sign in to comment.