Skip to content

[10.0] Fix base_kanban_stage tests#762

Merged
lasley merged 1 commit into
OCA:10.0from
akretion:10-fix-base_kanban_stage
Mar 6, 2017
Merged

[10.0] Fix base_kanban_stage tests#762
lasley merged 1 commit into
OCA:10.0from
akretion:10-fix-base_kanban_stage

Conversation

@florian-dacosta

Copy link
Copy Markdown
Contributor

I think there is a problem with base_kanban_stage tests, as it never leaves the test mode :
https://github.com/OCA/server-tools/blob/10.0/base_kanban_stage/tests/test_base_kanban_abstract.py#L47

Tests in other module may fail because of that.

@lasley

lasley commented Mar 6, 2017

Copy link
Copy Markdown
Contributor

@obulkin - please take a look

@lasley lasley added this to the 10.0 milestone Mar 6, 2017

@obulkin obulkin left a comment

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.

👍 Looks good to me. enter_test_mode() makes some changes to the registry class that probably persist between test suites if they're not reversed by leave_test_mode(): https://github.com/odoo/odoo/blob/10.0/odoo/modules/registry.py#L435

@lasley

lasley commented Mar 6, 2017

Copy link
Copy Markdown
Contributor

Thanks @florian-dacosta

@lasley
lasley merged commit 628b11d into OCA:10.0 Mar 6, 2017
@lasley

lasley commented Mar 6, 2017

Copy link
Copy Markdown
Contributor

@florian-dacosta - Did you plan on backporting to v9?

@florian-dacosta

Copy link
Copy Markdown
Contributor Author

@lasley
No, actually I don't think the bug is present in version 9.

@lasley

lasley commented Mar 6, 2017

Copy link
Copy Markdown
Contributor

Awesome, thanks again @florian-dacosta!

SiesslPhillip pushed a commit to grueneerde/OCA-server-tools that referenced this pull request Nov 20, 2024
Syncing from upstream OCA/server-tools (8.0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants