Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Valkyrizes spec/services/hyrax/statistics/depositors/summary_spec.rb. #6528

Merged
merged 2 commits into from
Dec 14, 2023

Conversation

bwatson78
Copy link
Contributor

Fixes

Fixes spec/services/hyrax/statistics/depositors/summary_spec.rb.

Summary

Valkyrizes spec/services/hyrax/statistics/depositors/summary_spec.rb.

Type of change (for release notes)

  • notes-valkyrie Valkyrie Progress

Detailed Description

Swaps between ActiveFedora and Valkyrie objects depending on environment. Tried using Valkyrie objects only, but the indexing was defaulting to Valkyrie::Indexing::NullIndexingAdapter within the Dassie environment, which is essentially pass-through, making it hard to manipulate with the before statement.

@samvera/hyrax-code-reviewers

@bwatson78 bwatson78 added valkyrization notes-valkyrie Release Notes: Valkyrie specific labels Dec 11, 2023
@bwatson78 bwatson78 self-assigned this Dec 11, 2023
@dlpierce
Copy link
Contributor

The :index_adapter spec tag didn't work to use a different indexing adapter?

@bwatson78 bwatson78 force-pushed the updates_statistics_depositors_summary_spec branch from 3bfe0b9 to 2cf4756 Compare December 12, 2023 21:01
@bwatson78
Copy link
Contributor Author

Oh, didn't realize that was an option. I'll look for examples.

@bwatson78 bwatson78 force-pushed the updates_statistics_depositors_summary_spec branch from 1be0dd1 to fbaeadb Compare December 13, 2023 19:18
@dlpierce dlpierce merged commit dccce1e into main Dec 14, 2023
4 checks passed
@dlpierce dlpierce deleted the updates_statistics_depositors_summary_spec branch December 14, 2023 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
notes-valkyrie Release Notes: Valkyrie specific valkyrization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants