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

Makes spec/services/hyrax/file_set_fixity_check_service_spec.rb Dassie-only. #6459

Merged
merged 2 commits into from
Dec 1, 2023

Conversation

bwatson78
Copy link
Contributor

Fixes

Fixes spec/services/hyrax/file_set_fixity_check_service_spec.rb.

Summary

Makes spec/services/hyrax/file_set_fixity_check_service_spec.rb Dassie-only.

Type of change (for release notes)

Add an appropriate notes-* label to the PR (or indicate here) that classifies this change.

  • notes-valkyrie Valkyrie Progress

@samvera/hyrax-code-reviewers

@bwatson78 bwatson78 added valkyrization notes-valkyrie Release Notes: Valkyrie specific labels Nov 20, 2023
@bwatson78 bwatson78 self-assigned this Nov 20, 2023
@no-reply
Copy link
Contributor

Doesn't this code need to work for koppie?

@bwatson78 bwatson78 force-pushed the updates_file_set_fixity_check_service_spec branch from 373d152 to 73f19f7 Compare November 20, 2023 19:19
@bwatson78
Copy link
Contributor Author

@no-reply I attempted to convert the tested FileSet to a :hyrax_file_set, but was blocked by errors stating that .files wasn't a method of Hyrax::FileSet. So I went back to the Service to see if anything pointed to this being ActiveFedora only, and saw that in the Service class, the expected param is an ActiveFedora object.

I tried to do due diligence and track down whether there was a separate Valkyrie process to perform Fixity, but couldn't find one. Would you happen to know where the Fixity processing starts for Valkyrie FileSets?

@bwatson78 bwatson78 force-pushed the updates_file_set_fixity_check_service_spec branch 3 times, most recently from bdc010a to ad6b7a4 Compare November 30, 2023 14:30
@bwatson78 bwatson78 force-pushed the updates_file_set_fixity_check_service_spec branch from 0d57624 to e78c619 Compare December 1, 2023 20:30
@dlpierce dlpierce merged commit e8b8058 into main Dec 1, 2023
4 checks passed
@dlpierce dlpierce deleted the updates_file_set_fixity_check_service_spec branch December 1, 2023 21:07
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.

3 participants