Skip to content

[FIX] views.edit: import MockRequest based on Odoo version - #7

Merged
jdkirkwood merged 1 commit into
masterfrom
master-import_mockrequest-kij
Nov 11, 2025
Merged

[FIX] views.edit: import MockRequest based on Odoo version#7
jdkirkwood merged 1 commit into
masterfrom
master-import_mockrequest-kij

Conversation

@jdkirkwood

@jdkirkwood jdkirkwood commented Nov 10, 2025

Copy link
Copy Markdown
Contributor

Description

In 19.0, MockRequest is moved to http_routing.tests.common.

Ensure that config["test_enable"] is set before import MockRequest to avoid traceback message in logs - no longer any need for mute_logger().

Reported by: YOMA 👍

All Submissions:

  • My commit respects the Odoo commit guideline
  • My commit message respects the commit template
  • I have used pre-commit [other files/methods are raising errors - ignore for now]
  • The PR contains only my modification and no other external commit

Sh/Runbot:

  • The commits pass test and the branch is green
  • Unit tests have been implemented / standard ones rewritten
  • The Staging is ISO-Prod and will contain only this dev

Upgrade:

  • The data affected (if any) by the changes has been migrated

Maintenance reminders:

  • Always bump the version of the manifest on the affected modules.
  • Notify the developer responsible for the initial development task (when this is relevant).

In `19.0`, `MockRequest` is moved to `http_routing.tests.common`.
Ensure that `config["test_enable"]` is set before `import` to avoid traceback message in logs - no longer any need for `mute_logger()`.
@jdkirkwood
jdkirkwood requested a review from a team November 10, 2025 19:21
@isko-odoo isko-odoo self-assigned this Nov 11, 2025
@jdkirkwood
jdkirkwood merged commit 4f9bb99 into master Nov 11, 2025
1 check failed
@jdkirkwood
jdkirkwood deleted the master-import_mockrequest-kij branch November 11, 2025 14:02
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.

2 participants