From de5362f32b9c783002295d7553317d3a5fbbaa71 Mon Sep 17 00:00:00 2001 From: kitcommerce <262714811+kitcommerce@users.noreply.github.com> Date: Fri, 20 Feb 2026 10:12:55 -0500 Subject: [PATCH] WA-OPS-653: Fix existing Rubocop offenses on next --- Rakefile | 2 +- core/lib/workarea/configuration.rb | 4 ++-- core/lib/workarea/ext/freedom_patches/bson_document.rb | 2 +- core/test/lib/workarea/elasticsearch/document_test.rb | 2 +- 4 files changed, 5 insertions(+), 5 deletions(-) diff --git a/Rakefile b/Rakefile index 3ab9134b73..f1625da563 100644 --- a/Rakefile +++ b/Rakefile @@ -77,7 +77,7 @@ task "performance_test_ci" do GEMS.each do |gem| $: << "#{gem}/test" - paths << "#{gem}/test/performance/**/*_test.rb" + paths << "#{gem}/test/performance/**/*_test.rb" end Rails::TestUnit::Runner.rake_run(paths) diff --git a/core/lib/workarea/configuration.rb b/core/lib/workarea/configuration.rb index d41b5175a7..b92a1cf85a 100644 --- a/core/lib/workarea/configuration.rb +++ b/core/lib/workarea/configuration.rb @@ -951,8 +951,8 @@ def self.setup_defaults # migration. config.localized_active_fields = true - # Options passed to the Selenium driver's capabilities - # Chrome 75+ defaults to W3C protocol. The old `w3c: false` forced the +# Options passed to the Selenium driver's capabilities +# Chrome 75+ defaults to W3C protocol. The old `w3c: false` forced the # deprecated JSON Wire Protocol, which modern Chrome/chromedriver rejects. config.headless_chrome_options = {} diff --git a/core/lib/workarea/ext/freedom_patches/bson_document.rb b/core/lib/workarea/ext/freedom_patches/bson_document.rb index 6c830eabeb..8e1db78361 100644 --- a/core/lib/workarea/ext/freedom_patches/bson_document.rb +++ b/core/lib/workarea/ext/freedom_patches/bson_document.rb @@ -7,7 +7,7 @@ class BSON::Document # Save the original method if it exists if method_defined?(:deep_symbolize_keys!) alias_method :original_bson_deep_symbolize_keys!, :deep_symbolize_keys! - + # Override to convert to Hash first, then symbolize def deep_symbolize_keys! # Convert to regular Hash to avoid BSON deprecation diff --git a/core/test/lib/workarea/elasticsearch/document_test.rb b/core/test/lib/workarea/elasticsearch/document_test.rb index 5d94d39384..dcd862f572 100644 --- a/core/test/lib/workarea/elasticsearch/document_test.rb +++ b/core/test/lib/workarea/elasticsearch/document_test.rb @@ -111,7 +111,7 @@ def test_count assert_equal(1, Foo.count) assert_equal(1, Foo.current_index.count({})) - + # Test count with query filter assert_equal(1, Foo.current_index.count({ query: { term: { id: '1' } } })) assert_equal(0, Foo.current_index.count({ query: { term: { id: '999' } } }))