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] stock_dropshipping: fix test classes #32792

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
3 participants
@len-odoo
Copy link
Contributor

commented Apr 18, 2019

Commit 52c89ad added a test in v11.
Commit 1e0a92c forward-ported that commit to v12, that already had a test.
As a result, the class TestDropship was defined twice.
It follows that the first test was not executed.

opw 1957701

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

[FIX] stock_dropshipping: fix test classes
Commit 52c89ad added a test in v11.
Commit 1e0a92c forward-ported that commit to v12, that already had a test.
As a result, the class TestDropship was defined twice.
It follows that the first test was not executed.

opw 1957701

@robodoo robodoo added the seen 🙂 label Apr 18, 2019

@C3POdoo C3POdoo added the OE label Apr 18, 2019

@robodoo robodoo added the CI 🤖 label Apr 18, 2019

@len-odoo

This comment has been minimized.

Copy link
Contributor Author

commented Apr 18, 2019

robodoo r+

@robodoo robodoo added the r+ 👌 label Apr 18, 2019

@robodoo robodoo closed this in 6dcf798 Apr 18, 2019

@robodoo robodoo added merged 🎉 and removed merging 👷 labels Apr 18, 2019

@robodoo

This comment has been minimized.

Copy link
Contributor

commented Apr 18, 2019

Merged, thanks!

@len-odoo len-odoo deleted the odoo-dev:12.0-opw1957701-dropt-len branch Apr 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.