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

two small fixes #1

Closed
wants to merge 1 commit into from
Closed

two small fixes #1

wants to merge 1 commit into from

Conversation

stanislaw
Copy link

first: #role_list could not operate on model new_record with troles field equal to nil. I've added this simplest case.
You UserSetup had only Factory with :troles => [] which is not like real case with User.new => having troles field nil.

second: about thing of my last email: Adding .list (#add_roles) resolved email's issue and led me to the next break:
I enabled Common Write API specs and one of them is falling on:
5) Troles::Strategy::RefMany it should behave like Common API behaves like Common Write API#set_roles should set set roles to :user only
Failure/Error: user.set_roles(:admin)
NameError:
undefined local variable or method role_model' for #<Troles::Storage::RefMany:0x8705c30> Shared Example Group: "Common Write API" called from ./spec/troles/common/api_spec.rb:36 # ./lib/troles/storage/base_many.rb:14:infind_roles'
# ./lib/troles/storage/ref_many.rb:25:in set_roles' # ./lib/troles/common/api/write.rb:39:inset_roles'
# ./spec/troles/common/api/write_api_spec.rb:11:in `block (3 levels) in <top (required)>'
'''''
This is up to you.
The first thing I want to make work is simply adding roles ;)

@stanislaw stanislaw closed this Jun 4, 2011
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.

1 participant