geoblacklight 5.3.0 → 5.4.0

data/spec/solr_config_spec.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+RSpec.describe "Solr configuration" do
+  def parse(name)
+    Nokogiri::XML(File.read(File.expand_path("../solr/conf/#{name}", __dir__)))
+  end
+
+  let(:schema) { parse("schema.xml") }
+  let(:solrconfig) { parse("solrconfig.xml") }
+
+  # Every *_ti and *_tmi name matches a dynamicField, so boosting a field that no
+  # copyField populates is legal, queryable, and always empty -- Solr never
+  # complains, the boost just silently contributes nothing to scoring. That has
+  # happened twice now: renaming a copyField destination from *_ti to *_tmi
+  # (multiValued sources need multiValued destinations) left the matching qf/pf
+  # entry behind. Deliberately ignores dynamicFields, since resolving through one
+  # is exactly the bug being guarded against.
+  def populated_fields
+    schema.xpath("//fields/field/@name").map(&:value) +
+      schema.xpath("//copyField/@dest").map(&:value)
+  end
+
+  # Reads the /text() children rather than the <str> node's own text, so the XML
+  # comment inside <str name="pf"> cannot leak words into the field list.
+  def boosted_fields(param)
+    solrconfig.xpath("//requestHandler[@name='/select']//str[@name='#{param}']/text()")
+      .map(&:text).join(" ")
+      .split.map { |entry| entry.split("^").first }
+  end
+
+  %w[qf pf].each do |param|
+    it "only boosts #{param} fields that something populates" do
+      dead = boosted_fields(param) - populated_fields
+
+      expect(dead).to be_empty,
+        "#{param} boosts fields no copyField or <field> populates: #{dead.join(", ")}"
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -36,7 +36,30 @@ require "selenium-webdriver"
 require "webmock/rspec"
 WebMock.allow_net_connect!(net_http_connect_on_start: true)
 
-Capybara.javascript_driver = :selenium_chrome_headless
+Capybara.register_driver :chrome_headless do |app|
+  # Set up Chrome options
+  options = Selenium::WebDriver::Chrome::Options.new
+  options.add_argument("--headless")
+  options.add_argument("--disable-gpu")
+  options.add_argument("--no-sandbox")
+  options.add_argument("--window-size=1280,1024")
+  # Return from #visit at DOMContentLoaded instead of waiting for the full
+  # window `load` event. GeoBlacklight pages pull basemap tiles and WMS/TMS
+  # layers from external hosts; in CI those requests can hang and prevent
+  # `load` from ever firing, causing Net::ReadTimeout on #visit.
+  options.page_load_strategy = :eager
+
+  # Allow longer TCP reads to prevent timeout in the case of loading fixture
+  # eee6150b-ce2f-4837-9d17-ce72a0c1c26f, as part of relations_spec.rb
+  client = Selenium::WebDriver::Remote::Http::Default.new
+  client.read_timeout = 120 # seconds
+
+  Capybara::Selenium::Driver.new(app, browser: :chrome, options: options, http_client: client)
+end
+
+Capybara.javascript_driver = :chrome_headless
+
+Capybara.default_max_wait_time = ENV["CI"] ? 10 : 2
 
 require "geoblacklight"
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: geoblacklight
 version: !ruby/object:Gem::Version
-  version: 5.3.0
+  version: 5.4.0
 platform: ruby
 authors:
 - Mike Graves
@@ -10,7 +10,7 @@ authors:
 - Jack Reed
 bindir: bin
 cert_chain: []
-date: 2026-05-28 00:00:00.000000000 Z
+date: 2026-09-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -248,14 +248,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '4.11'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '4.11'
 - !ruby/object:Gem::Dependency
   name: factory_bot_rails
   requirement: !ruby/object:Gem::Requirement
@@ -299,19 +299,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: standardrb
+  name: standard
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.1
+        version: 1.56.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.1
+        version: 1.56.0
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
@@ -614,6 +614,7 @@ files:
 - lib/geoblacklight.rb
 - lib/geoblacklight/bounding_box.rb
 - lib/geoblacklight/constants.rb
+- lib/geoblacklight/deprecated_configuration.rb
 - lib/geoblacklight/download.rb
 - lib/geoblacklight/download/csv_download.rb
 - lib/geoblacklight/download/geojson_download.rb
@@ -890,6 +891,7 @@ files:
 - spec/javascripts/geoblacklight_spec.js
 - spec/javascripts/metadata_download_button_spec.js
 - spec/lib/geoblacklight/bounding_box_spec.rb
+- spec/lib/geoblacklight/deprecated_configuration_spec.rb
 - spec/lib/geoblacklight/download/csv_download_spec.rb
 - spec/lib/geoblacklight/download/geojson_download_spec.rb
 - spec/lib/geoblacklight/download/geotiff_download_spec.rb
@@ -921,6 +923,7 @@ files:
 - spec/models/concerns/geoblacklight/suppressed_records_search_behavior_spec.rb
 - spec/presenters/geoblacklight/bbox_item_presenter_spec.rb
 - spec/routing/catalog_routing_spec.rb
+- spec/solr_config_spec.rb
 - spec/spec_helper.rb
 - spec/support/backport_test_helpers.rb
 - spec/support/features.rb