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.

Sign up for GitHub

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

Jump to bottom

Fix content_deposit_event_job_spec.rb #6452

Merged
merged 1 commit into from
Nov 17, 2023
Merged

Fix content_deposit_event_job_spec.rb #6452

merged 1 commit into from
Nov 17, 2023

Conversation

abelemlih
Copy link
Contributor

Fixes

Fix failing tests in content_deposit_event_job_spec.rb when Valkyrie is used

  • notes-valkyrie Valkyrie Progress

Changes proposed in this pull request:

  • Change work creation in the test file to be Valkyrie compatible

@samvera/hyrax-code-reviewers

@abelemlih abelemlih added valkyrization notes-valkyrie Release Notes: Valkyrie specific labels Nov 16, 2023
@abelemlih abelemlih self-assigned this Nov 16, 2023
@abelemlih abelemlih force-pushed the abel-fix_content_deposit_event_job_spec.rb branch 4 times, most recently from 4f24d6e to 2cf469f Compare November 17, 2023 16:40
@abelemlih abelemlih force-pushed the abel-fix_content_deposit_event_job_spec.rb branch from c22a610 to 49ab4a5 Compare November 17, 2023 17:51
@abelemlih abelemlih merged commit 0cbf673 into main Nov 17, 2023
4 checks passed
@abelemlih abelemlih deleted the abel-fix_content_deposit_event_job_spec.rb branch November 17, 2023 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dunn dunn dunn approved these changes

@dlpierce dlpierce Awaiting requested review from dlpierce

Assignees

@abelemlih abelemlih

Labels
notes-valkyrie Release Notes: Valkyrie specific valkyrization
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@abelemlih @dunn