geoblacklight 5.2.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"
 

data/vite.config.ts

@@ -28,8 +28,5 @@ export default defineConfig(() => {
         ],
       },
     },
-    test: {
-      environment: "jsdom",
-    },
   };
 });

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: geoblacklight
 version: !ruby/object:Gem::Version
-  version: 5.2.0
+  version: 5.4.0
 platform: ruby
 authors:
 - Mike Graves
@@ -10,7 +10,7 @@ authors:
 - Jack Reed
 bindir: bin
 cert_chain: []
-date: 2026-02-18 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
@@ -483,6 +483,8 @@ files:
 - app/components/blacklight/icons/uva_component.rb
 - app/components/blacklight/icons/web_services_component.rb
 - app/components/blacklight/icons/websites_component.rb
+- app/components/geoblacklight/accordion_component.html.erb
+- app/components/geoblacklight/accordion_component.rb
 - app/components/geoblacklight/arcgis_component.html.erb
 - app/components/geoblacklight/arcgis_component.rb
 - app/components/geoblacklight/attribute_table_component.html.erb
@@ -557,6 +559,7 @@ files:
 - app/javascript/geoblacklight/leaflet/constants.js
 - app/javascript/geoblacklight/leaflet/controls/geosearch.js
 - app/javascript/geoblacklight/leaflet/controls/layer_opacity.js
+- app/javascript/geoblacklight/leaflet/controls/sidebar.js
 - app/javascript/geoblacklight/leaflet/controls/sleep.js
 - app/javascript/geoblacklight/leaflet/inspection.js
 - app/javascript/geoblacklight/leaflet/layer_index_map.js
@@ -611,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
@@ -887,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
@@ -918,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
@@ -927,7 +933,6 @@ files:
 - spec/tasks/geoblacklight_spec.rb
 - spec/test_app_templates/lib/generators/test_app_generator.rb
 - template.rb
-- test/leaflet/layers.test.js
 - vite.config.ts
 homepage: http://github.com/geoblacklight/geoblacklight
 licenses:

data/test/leaflet/layers.test.js

@@ -1,17 +0,0 @@
-import { wmsLayer, wmtsLayer } from "../../app/javascript/leaflet/layers";
-import { describe, expect, it } from "vitest";
-
-describe("leaflet/layers", () => {
-  describe("wmsLayer", () => {
-    it("returns a WMS tile layer", () => {
-      const url = "http://example.com/wms";
-      const layerId = "layerId";
-      const opacity = 0.5;
-      const detectRetina = true;
-      const layer = wmsLayer(url, { layerId, opacity, detectRetina });
-      expect(layer.options.layers).toEqual(layerId);
-      expect(layer.options.opacity).toEqual(opacity);
-      expect(layer.options.detectRetina).toEqual(detectRetina);
-    });
-  });
-});