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

Adapt line_items_spec to work in the rails 4 branch as well #4909

Merged
merged 1 commit into from
Mar 25, 2020

Conversation

luisramos0
Copy link
Contributor

What? Why?

Closes #4892

See commit message.

What should we test?

Green build.

Release notes

Changelog Category: Changed
Adapt test code to rails 4.

…ble on the adjustment is not enough to populate line_item.adjustments. Here we make this assignment explicit fixing the spec in rails 4
@luisramos0 luisramos0 self-assigned this Mar 3, 2020
@luisramos0 luisramos0 requested a review from sauloperez March 21, 2020 17:05
@sauloperez sauloperez merged commit 87fae15 into openfoodfoundation:master Mar 25, 2020
@luisramos0 luisramos0 deleted the line_items_spec branch March 25, 2020 10:53
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.

[Spree 2.1] Fix models/line_items_spec (2 broken specs)
4 participants