Skip to content

[19.0][MIG] purchase_order_line_original_date: Migration to 19.0#2850

Open
Hudson5577 wants to merge 10 commits intoOCA:19.0from
Hudson5577:19.0-mig-purchase_order_line_original_date
Open

[19.0][MIG] purchase_order_line_original_date: Migration to 19.0#2850
Hudson5577 wants to merge 10 commits intoOCA:19.0from
Hudson5577:19.0-mig-purchase_order_line_original_date

Conversation

@Hudson5577
Copy link
Copy Markdown

Standard migration

@Hudson5577 Hudson5577 changed the title 19.0-mig-purchase_order_line_original_date [19.0][MIG] purchase_order_line_original_date: Migration to 19.0 Oct 16, 2025
@HviorForgeFlow
Copy link
Copy Markdown
Member

/ocabot migration purchase_order_line_original_date

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Nov 4, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Nov 4, 2025
78 tasks
Copy link
Copy Markdown
Contributor

@BhaveshHeliconia BhaveshHeliconia left a comment

Choose a reason for hiding this comment

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

Consider using BaseCommon instead of TransactionCase. In this case, there’s no need to redefine res.partner is already available via BaseCommon. This would help simplify and optimize the test code.

from odoo.tests import common


class TestPoLineOriginalDate(common.TransactionCase):
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
class TestPoLineOriginalDate(common.TransactionCase):
class TestPoLineOriginalDate(Basecommon):

super().setUpClass()
cls.po_model = cls.env["purchase.order"]

cls.vendor_1 = cls.env["res.partner"].create(
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
cls.vendor_1 = cls.env["res.partner"].create(

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.

7 participants