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

[Upstream] Split factories #1663

Merged
merged 1 commit into from
Oct 4, 2018

Conversation

javierm
Copy link

@javierm javierm commented Oct 3, 2018

References

  • Pull request consul#2838

Objectives

Split the factories.rb file into smaller logical files

Notes

  • There were some factories specific to this fork and I couldn't figure out where to add them. For now, I've created spec/factories/custom.rb.

end
end

trait :with_confirmed_hide do

Choose a reason for hiding this comment

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

Layout/IndentationConsistency: Inconsistent indentation detected. (https://github.com/bbatsov/ruby-style-guide#spaces-indentation)

hidden_at { Time.current }
end

trait :with_ignored_flag do

Choose a reason for hiding this comment

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

Layout/IndentationConsistency: Inconsistent indentation detected. (https://github.com/bbatsov/ruby-style-guide#spaces-indentation)

valuation_finished true
end

trait :hidden do

Choose a reason for hiding this comment

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

Layout/IndentationConsistency: Inconsistent indentation detected. (https://github.com/bbatsov/ruby-style-guide#spaces-indentation)

@javierm javierm force-pushed the upstream-refactor-factories-in-several-files branch from 2d96058 to af60611 Compare October 3, 2018 16:44
@javierm javierm changed the title [Upstream] Refactor factories in several files [Upstream] Split factories Oct 3, 2018
@javierm javierm merged commit da0cabd into master Oct 4, 2018
@javierm javierm deleted the upstream-refactor-factories-in-several-files branch October 4, 2018 11:05
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