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

Fix the CI #145

Merged
merged 6 commits into from
Mar 16, 2021
Merged

Fix the CI #145

merged 6 commits into from
Mar 16, 2021

Conversation

kennyadsl
Copy link
Member

Description

Solidus master now tracks 3.0 and we are testing this gem against it. Almost all the deprecated code was removed but there was still something.

@kennyadsl kennyadsl self-assigned this Feb 18, 2021
@kennyadsl
Copy link
Member Author

I still couldn't figure out what happens with specs here. My initial investigation showed that for some specs, the ensure_inventory_units (called from here) raise InsufficientStock, even if there's apparently nothing in the specs that could make it behave this way.

Also, we have the same exact tests in solidus_frontend and they work correctly.

🤔

@kennyadsl kennyadsl merged commit e5706e7 into master Mar 16, 2021
@kennyadsl kennyadsl deleted the kennyadsl/fix-ci branch March 16, 2021 14:01
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.

1 participant