[FIX] delivery_driver: wrong model name

[yajo]

Regression from #774:

  File "/opt/odoo/auto/addons/delivery_driver/migrations/16.0.1.0.1/post-migration.py", line 15, in migrate
    env["carrier.driver"]
  File "/opt/odoo/custom/src/odoo/odoo/api.py", line 549, in __getitem__
    return self.registry[model_name](self, (), ())
  File "/opt/odoo/custom/src/odoo/odoo/modules/registry.py", line 190, in __getitem__
    return self.models[model_name]
KeyError: 'carrier.driver'
Error: 'carrier.driver'

@moduon MT-5232

[OCA-git-bot]

Hi @EmilioPascual, @rafaelbn,
some modules you are maintaining are being modified, check this out!

[rousseldenis]

Tests are failing because of that sh... CTTExpress module

[yajo]

It seems one of those cases where a unit test has been confused with a monitoring tool. Now what? This issue is pretty critical...

[pedrobaeza]

The test in CTT is not intended for monitoring, but for assuring that the API doesn't change and the module stops to work. Mocking that will lead to fail in that case but green CIs. Anyway, it's true that the demo environment against the tests is done is a bit unstable, but it's usually question of minutes.

You can meanwhile add the PR in your deployment, as it's not critical, as it doesn't affect any new installation nor anyone not using the module.

[EmilioPascual]

/ocabot merge

[OCA-git-bot]

Hi @EmilioPascual. Your command failed:

Required option bumpversion_mode for command merge. Possible values : major, minor, patch, nobump.

Ocabot commands

• ocabot merge major|minor|patch|nobump
• ocabot rebase
• ocabot migration {MODULE_NAME}

More information

• ocabot documentation
• OCA guidelines, specially the "Version Numbers" section.

[rousseldenis]

It seems one of those cases where a unit test has been confused with a monitoring tool. Now what? This issue is pretty critical...

In ten years, we'll laugh about that around a drink 😃

But, as already said, that one deserves a separate repo to not bother other people testing flows.

[EmilioPascual]

/ocabot merge minor

[OCA-git-bot]

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-783-by-EmilioPascual-bump-minor, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 5180e6f. Thanks a lot for contributing to OCA. ❤️

[yajo]

I think we need a guideline about flaky tests, so here's my proposal: OCA/odoo-community.org#158