FI-2836: Fix double requests from suite endpoints (#505)

* fix comparison * remove logging statement
inferno-framework · Jun 17, 2024 · 4b48919 · 4b48919
1 parent 0a95ece
commit 4b48919
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/lib/inferno/dsl/suite_endpoint.rb b/lib/inferno/dsl/suite_endpoint.rb
@@ -289,7 +289,6 @@ def add_persistence_callback # rubocop:disable Metrics/CyclomaticComplexity
           uri.query = env['rack.request.query_string'] if env['rack.request.query_string'].present?
           url = uri&.to_s
           verb = env['REQUEST_METHOD']
-          logger.info('get body')
           request_body = env['rack.input']
           request_body.rewind if env['rack.input'].respond_to? :rewind
           request_body = request_body.instance_of?(Puma::NullIO) ? nil : request_body.string

diff --git a/lib/inferno/utils/middleware/request_logger.rb b/lib/inferno/utils/middleware/request_logger.rb
@@ -34,7 +34,7 @@ def call(env)
 
           # rack.after_reply is handled by puma, which doesn't process requests
           # in unit tests, so we manually run them when in the test environment
-          env['rack.after_reply']&.each(&:call) if (ENV['APP_ENV'] = 'test')
+          env['rack.after_reply']&.each(&:call) if ENV['APP_ENV'] == 'test'
 
           response
         end