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

Add factory_bot lint test to core #3647

Merged
merged 1 commit into from
Jun 4, 2020

Commits on Jun 3, 2020

  1. Add factory_bot lint test to core

    Adds a test that lints FactoryBot. Also makes changes to two tests to
    conform to the lint - store_credit_event_factory needs to have an
    action, even if the action is replaced with one of the sub factories.
    store_credit_reason_factory tripped the linter because of a uniqueness
    contraint for the name - should not come up in normal testing, but I
    added a sequencer to account for it in the linter anyway.
    
    The linter ignores just one factory - customer_return_without_return_items
    this factory is intentionally invalid (see:
    /core/spec/lib/spree/core/testing_support/factories/customer_return_factory_spec.rb#L37
    )
    seand7565 committed Jun 3, 2020
    Configuration menu
    Copy the full SHA
    92a5e3b View commit details
    Browse the repository at this point in the history