Update Rack::Builder.parse_file syntax

This now returns an instance of `PrometheusExporter::Middleware` directly, instead of inside an array
alphagov · Jul 30, 2024 · 5616fb1 · 5616fb1
1 parent 538b0f9
commit 5616fb1
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/lib/benchmark/all.rb b/lib/benchmark/all.rb
@@ -21,7 +21,7 @@ def initialize(number_of_runs)
     end
 
     def app
-      @app ||= Rack::Builder.parse_file("config.ru")[0]
+      @app ||= Rack::Builder.parse_file("config.ru")
     end
 
     def run!

diff --git a/spec/features/http_request_handling_spec.rb b/spec/features/http_request_handling_spec.rb
@@ -6,7 +6,7 @@
   subject { last_response }
 
   def app
-    Rack::Builder.parse_file("config.ru")[0]
+    Rack::Builder.parse_file("config.ru")
   end
 
   let(:department_of_health) do
@@ -896,7 +896,7 @@ def app
     # In dev (but not production), CommonLogger middleware is in use, which expects
     # us not to violate the Rack spec. Part of this is not allowing nil query strings,
     # which at time of writing optic14n does.
-    let(:app) { Rack::CommonLogger.new(Rack::Builder.parse_file("config.ru")[0]) }
+    let(:app) { Rack::CommonLogger.new(Rack::Builder.parse_file("config.ru")) }
 
     describe "Nil querystrings do not faze us" do
       before do

diff --git a/spec/performance/benchmark_spec.rb b/spec/performance/benchmark_spec.rb
@@ -5,7 +5,7 @@
   include Rack::Test::Methods
 
   def app
-    Rack::Builder.parse_file("config.ru")[0]
+    Rack::Builder.parse_file("config.ru")
   end
 
   let(:number_of_runs) { ENV["NUMBER_OF_RUNS"] || 1000 }