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

Follow FactoryBot rename #2315

Merged
merged 1 commit into from
Oct 24, 2017
Merged

Conversation

mamhoff
Copy link
Contributor

@mamhoff mamhoff commented Oct 23, 2017

FactoryGirl has been renamed to FactoryBot. This changes all occurences of
FactoryGirl to follow their new naming convention.

This also fixes our sample app builds.

FactoryGirl has been renamed to FactoryBot. This changes all occurences of
FactoryGirl to follow their new naming convention.

This also fixes our sample app builds.
available_to_admin: true) }
let!(:second_method) { FactoryGirl.create(:payment_method, available_to_users: true,
let!(:second_method) { FactoryBot.create(:payment_method, available_to_users: true,

Choose a reason for hiding this comment

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

Block body expression is on the same line as the block start.

@@ -596,9 +596,9 @@ def merge!(other_order, user = nil)
context "with more than one payment method" do
subject { order.available_payment_methods }

let!(:first_method) { FactoryGirl.create(:payment_method, available_to_users: true,
let!(:first_method) { FactoryBot.create(:payment_method, available_to_users: true,

Choose a reason for hiding this comment

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

Block body expression is on the same line as the block start.

Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

Thanks

@mamhoff mamhoff merged commit 19efd80 into solidusio:master Oct 24, 2017
@mamhoff mamhoff deleted the rename-factory-bot branch October 24, 2017 08:16
schmijos added a commit to renuo/solidus_simple_sale_price that referenced this pull request Apr 19, 2018
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