Skip to content

Index test fixtures using the gem path#1688

Merged
dl-maura merged 2 commits intomainfrom
indexing-rake-task
Nov 13, 2025
Merged

Index test fixtures using the gem path#1688
dl-maura merged 2 commits intomainfrom
indexing-rake-task

Conversation

@thatbudakguy
Copy link
Member

@thatbudakguy thatbudakguy commented Nov 12, 2025

This lets us pull in the fixture data from Geoblacklight's repo
without needing to fetch it remotely from GitHub, by referencing
the files that are bundled with the gem. These always get
installed in your local Geoblacklight, so they're already there
for you to use.

Works both when working on the Geoblacklight gem and working on
your local install.

Might fix #1668.

Makes #1676 unnecessary because we don't need to do remote
indexing of fixtures anymore.

Also adds a slight delay after solr is loaded in docker to fix issues
where indexing was rejected because the core wasn't ready.

@github-actions
Copy link

github-actions bot commented Nov 12, 2025

Demo app download link: https://github.com/geoblacklight/geoblacklight/actions/runs/19313081720/artifacts/4549160603

  1. Download demo app and unzip file
  2. Change into app directory
    • run docker compose pull
    • run docker compose up
  3. Open in browser

This lets us pull in the fixture data from Geoblacklight's repo
without needing to fetch it remotely from GitHub, by referencing
the files that are bundled with the gem. These always get
installed in your local Geoblacklight, so they're already there
for you to use.

Works both when working on the Geoblacklight gem and working on
your local install.

Might fix #1668.
In testing template.rb builds, I was seeing that newly created apps
would fail at the indexing test fixtures step with ECONNRESET.

Adding a slight delay for solr to finish loading the core config
solved it.
@thatbudakguy thatbudakguy force-pushed the indexing-rake-task branch 2 times, most recently from 712dda9 to 4216645 Compare November 12, 2025 21:55
@thatbudakguy thatbudakguy marked this pull request as ready for review November 12, 2025 21:55
@dl-maura dl-maura self-requested a review November 13, 2025 13:10
@dl-maura dl-maura merged commit 4f3a27f into main Nov 13, 2025
28 checks passed
@dl-maura dl-maura deleted the indexing-rake-task branch November 13, 2025 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Starting a project with Quick Start instructions fails to start

2 participants