fix(configuration): illogical refresh interval default (#6319)

When using the file provider with watch enabled, the refresh interval should just be set to always default as the cost is minimal.

Signed-off-by: James Elliott <james-d-elliott@users.noreply.github.com>

1 files changed, 7 insertions, 7 deletions

diff --git a/internal/handlers/handler_authz_builder_test.go b/internal/handlers/handler_authz_builder_test.go
index e7b145d36..a2d6609e0 100644
--- a/internal/handlers/handler_authz_builder_test.go
+++ b/internal/handlers/handler_authz_builder_test.go
@@ -14,31 +14,31 @@ func TestAuthzBuilder_WithConfig(t *testing.T) {
 
 	builder.WithConfig(&schema.Configuration{
 		AuthenticationBackend: schema.AuthenticationBackend{
-			RefreshInterval: "always",
+			RefreshInterval: schema.NewRefreshIntervalDurationAlways(),
 		},
 	})
 
-	assert.Equal(t, time.Second*0, builder.config.RefreshInterval)
+	assert.Equal(t, schema.NewRefreshIntervalDurationAlways(), builder.config.RefreshInterval)
 
 	builder.WithConfig(&schema.Configuration{
 		AuthenticationBackend: schema.AuthenticationBackend{
-			RefreshInterval: "disable",
+			RefreshInterval: schema.NewRefreshIntervalDurationNever(),
 		},
 	})
 
-	assert.Equal(t, time.Second*-1, builder.config.RefreshInterval)
+	assert.Equal(t, schema.NewRefreshIntervalDurationNever(), builder.config.RefreshInterval)
 
 	builder.WithConfig(&schema.Configuration{
 		AuthenticationBackend: schema.AuthenticationBackend{
-			RefreshInterval: "1m",
+			RefreshInterval: schema.NewRefreshIntervalDuration(time.Minute),
 		},
 	})
 
-	assert.Equal(t, time.Minute, builder.config.RefreshInterval)
+	assert.Equal(t, schema.NewRefreshIntervalDuration(time.Minute), builder.config.RefreshInterval)
 
 	builder.WithConfig(nil)
 
-	assert.Equal(t, time.Minute, builder.config.RefreshInterval)
+	assert.Equal(t, schema.NewRefreshIntervalDuration(time.Minute), builder.config.RefreshInterval)
 }
 
 func TestAuthzBuilder_WithEndpointConfig(t *testing.T) {