Skip to content

Commit

Permalink
Since we run plenty of dockerized tests without requiring an env var …
Browse files Browse the repository at this point in the history
…to (#7291)

be set, let's make the Radius tests behave that way too.
  • Loading branch information
ncabatoff authored and jefferai committed Aug 14, 2019
1 parent 843ab28 commit 28e8973
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions builtin/credential/radius/backend_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -144,11 +144,6 @@ func TestBackend_users(t *testing.T) {
}

func TestBackend_acceptance(t *testing.T) {
if os.Getenv(logicaltest.TestEnvVar) == "" {
t.Skip(fmt.Sprintf("Acceptance tests skipped unless env '%s' set", logicaltest.TestEnvVar))
return
}

b, err := Factory(context.Background(), &logical.BackendConfig{
Logger: nil,
System: &logical.StaticSystemView{
Expand Down Expand Up @@ -210,7 +205,6 @@ func TestBackend_acceptance(t *testing.T) {
logicaltest.Test(t, logicaltest.TestCase{
CredentialBackend: b,
PreCheck: testAccPreCheck(t, host, port),
AcceptanceTest: true,
Steps: []logicaltest.TestStep{
// Login with valid but unknown user will fail because unregistered_user_policies is empty
testConfigWrite(t, configDataAcceptanceNoAllowUnreg, false),
Expand Down

0 comments on commit 28e8973

Please # to comment.