-
Notifications
You must be signed in to change notification settings - Fork 4
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
Activate news and member factories #9
Conversation
News factory added
Create news.rb
Project members added
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.
- Please add new factories to tests.
- Please follow code style used in ryspec
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 that this will not work. Because user
is not defined on project
spec/factories/project.rb
Outdated
@@ -17,6 +18,9 @@ | |||
project.project_time_entry_activities = FactoryBot.create_list(:time_entry_activity, 1) if project.project_time_entry_activities.empty? | |||
|
|||
FactoryBot.create_list :issue, evaluator.number_of_issues, project: project | |||
evaluator.number_of_members.times do | |||
project.add_default_member(user) |
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.
Missing test for this line.
Member changes reverted, Redmine logic for creating new projects counts on setting environment that is not currently modeled by factories, so It's better to solve explicitly in specs. |
No description provided.