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

Activate news and member factories #9

Merged
merged 9 commits into from
Apr 5, 2019
Merged

Conversation

Joas1988
Copy link
Contributor

@Joas1988 Joas1988 commented Apr 2, 2019

No description provided.

Copy link
Contributor

@luk4s luk4s left a 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

Copy link
Contributor

@luk4s luk4s left a 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

@@ -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)
Copy link
Contributor

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.

@Joas1988
Copy link
Contributor Author

Joas1988 commented Apr 4, 2019

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.

@ondra-m ondra-m merged commit 92a0529 into easysoftware:devel Apr 5, 2019
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.

3 participants