From 7aafbaa169799f929923cbe8c8dcdbeafce114a1 Mon Sep 17 00:00:00 2001 From: Jose Colella Date: Wed, 18 Mar 2026 04:54:25 -0700 Subject: [PATCH] fix(ofrep): fix flaky rate limit test with date-based Retry-After The test used `Time.now + 1` for the Retry-After header, which could expire before the assertion ran on slow CI runners (Ruby 3.4). Changed to `Time.now + 60` to avoid timing sensitivity. Co-Authored-By: Claude Opus 4.6 Signed-off-by: Jose Colella --- .../spec/openfeature/ofrep/provider/client_spec.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/providers/openfeature-ofrep-provider/spec/openfeature/ofrep/provider/client_spec.rb b/providers/openfeature-ofrep-provider/spec/openfeature/ofrep/provider/client_spec.rb index 0a94314..649f3b5 100644 --- a/providers/openfeature-ofrep-provider/spec/openfeature/ofrep/provider/client_spec.rb +++ b/providers/openfeature-ofrep-provider/spec/openfeature/ofrep/provider/client_spec.rb @@ -173,7 +173,7 @@ it "blocks subsequent calls when rate limited with Retry-After header (date)" do stub_request(:post, "http://localhost:8080/ofrep/v1/evaluate/flags/my_flag") - .to_return(status: 429, headers: {"Retry-After" => (Time.now + 1).httpdate}) + .to_return(status: 429, headers: {"Retry-After" => (Time.now + 60).httpdate}) expect { client.evaluate(flag_key: "my_flag", evaluation_context: default_evaluation_context)