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

add tests for better coverage of pcdm objects #79

Merged
merged 1 commit into from
May 12, 2015
Merged

Conversation

elrayle
Copy link
Contributor

@elrayle elrayle commented May 12, 2015

This primarily extends pcdm object tests to make them consistent with testing for pcdm collection. This also added object hasMember object in the same way that collection hasMember collection.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 4ef4026 on improve_object_tests into * on master*.

awead added a commit that referenced this pull request May 12, 2015
add tests for better coverage of pcdm objects
@awead awead merged commit e73758c into master May 12, 2015
@awead awead deleted the improve_object_tests branch May 12, 2015 18:44
false
end

def ancestor? object
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add comments about what this method does.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this method be private?

@elrayle
Copy link
Contributor Author

elrayle commented May 12, 2015

The code in question implements object hasMember object. It is directly modeled off the same code for collection hasMember collection in collection_behavior.

https://github.com/projecthydra-labs/hydra-pcdm/blob/master/lib/hydra/pcdm/models/concerns/collection_behavior.rb

methods:
def collection_ancestor? collections
def ancestor? collection

If changes need to happen to make this function better, then they need to happen there as well.

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.

5 participants