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 rspecs to tagging spec #740

Merged
merged 2 commits into from
Mar 2, 2016

Conversation

pcupueran
Copy link
Contributor

I completed the test that were pending in the tagging_spec file. This helps to maintain the code

@seuros
Copy link
Collaborator

seuros commented Mar 2, 2016

can you squash your commits ?

@pcupueran
Copy link
Contributor Author

Hi Seuros, Ok I will squash them

Add rspec for owned_by scope

Add rspec for owned_by scope

a tagging should belong to a specific user

Add rspec test for owned by scope

a tagging should belong to a specific user

Add rspec test for by_context tagging scope

a tagging should be found by context

Add rspec test for by_contexts tagging scope

taggings can be found by contexts

Add rspec test for not_owned tagging scope

not_owned scope should found the taggings that do not have owner

Refactor owned_by rspec test for tagging_rspec

Refactor by_contexts rspec test for tagging_rspec
Update changelog

Update changelog
seuros added a commit that referenced this pull request Mar 2, 2016
@seuros seuros merged commit dde41a4 into mbleigh:master Mar 2, 2016
@seuros
Copy link
Collaborator

seuros commented Mar 2, 2016

Awesome 💚
Thank you @pcupueran

@dmgarland
Copy link

Well done Patty!

@SimonDKnight
Copy link

Nice addition Patty!

@pcupueran
Copy link
Contributor Author

Thank you!

tekniklr pushed a commit to tekniklr/acts-as-taggable-on that referenced this pull request Mar 19, 2021
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.

4 participants