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
ANW-1940 port resources spec to capybara #3155
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@thimios here are some debuggers to be removed
c937cd7
to
3a4a6cb
Compare
3a4a6cb
to
f7ce3e4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is good to merge once the tests finally pass
3564bac
to
cc3a4d2
Compare
cc3a4d2
to
98ef523
Compare
98ef523
to
16a2e24
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@thimios Phew that was a big one, nice work! I noticed a couple points for follow up and added some of the solution pointers.
select_repository(@repository) | ||
end | ||
|
||
it 'can create a new digital object instance with a note to a resource' do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this test probably belongs with the resources_spec.rb file since it's testing the ability of a Resource record to create a digital object instance for itself.
The description might also read 'can create and link a new digital object instance with a note'.
316d8df
to
944dd26
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏼
Description
Related JIRA Ticket or GitHub Issue
https://archivesspace.atlassian.net/browse/ANW-1940
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: