[ADD] [16.0] stock_picking_report_delivery_driver

[Shide]

Adds Delivery Driver to Stock Picking reports

Delivery Slip report

Picking Operations report

Delivery Slip without Carrier but Driver

MT-6115 @moduon @rafaelbn @yajo @EmilioPascual @fcvalgar please review if you want :)

[fcvalgar]

Functional review.

LGTM, Thank you @Shide

Picking Operations

Delivery Slip

Only driver

Only Carrier

[EmilioPascual]

Nice Job @Shide. Just a single change

[rafaelbn]

👍🏼 Thank you!

[rafaelbn]

/ocabot merge patch

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-321-by-rafaelbn-bump-patch, awaiting test results.

[OCA-git-bot]

@rafaelbn your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-321-by-rafaelbn-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

[rafaelbn]

/ocabot merge nobump

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-321-by-rafaelbn-bump-nobump, awaiting test results.

[OCA-git-bot]

@rafaelbn your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-321-by-rafaelbn-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

[rafaelbn]

Hello @Shide ! :-)

Test are failing, this is the message:

2024-06-02 20:58:05,334 288 ERROR odoo odoo.addons.stock_average_daily_sale.tests.test_average_daily_sale: FAIL: TestAverageSale.test_average_sale
Traceback (most recent call last):
  File "/__w/stock-logistics-reporting/stock-logistics-reporting/stock_average_daily_sale/tests/test_average_daily_sale.py", line 52, in test_average_sale
    self.assertRecordValues(
  File "/opt/odoo/odoo/tests/common.py", line 610, in assertRecordValues
    self.fail('\n'.join(errors))
AssertionError: The records and expected_values do not match.
Wrong number of records to compare: 0 records versus 1 expected values.

==== Missing record ====
{'average_qty_by_sale': 10.0,
 'nbr_sales': 1.0,
 'qty_in_stock': 40.0,
 'recommended_qty': 20.0,
 'warehouse_id': 1}
 
2024-06-02 20:58:05,337 288 INFO odoo odoo.addons.stock_average_daily_sale.tests.test_average_daily_sale: Starting TestAverageSale.test_average_sale_multiple ... 
2024-06-02 20:58:05,631 288 INFO odoo odoo.addons.stock_average_daily_sale.tests.test_average_daily_sale: ====================================================================== 
2024-06-02 20:58:05,631 288 ERROR odoo odoo.addons.stock_average_daily_sale.tests.test_average_daily_sale: FAIL: TestAverageSale.test_average_sale_multiple
Traceback (most recent call last):
  File "/__w/stock-logistics-reporting/stock-logistics-reporting/stock_average_daily_sale/tests/test_average_daily_sale.py", line 129, in test_average_sale_multiple
    self.assertRecordValues(
  File "/opt/odoo/odoo/tests/common.py", line 610, in assertRecordValues
    self.fail('\n'.join(errors))
AssertionError: The records and expected_values do not match.
Wrong number of records to compare: 0 records versus 1 expected values.

==== Missing record ====
{'nbr_sales': 3.0, 'qty_in_stock': 19.0, 'warehouse_id': 1}

Could you please take a look? 😄 ❤️

Thank you!

[Shide]

@rousseldenis @lmignon Could you please check the failing tests in stock_average_daily_sale module?

[rousseldenis]

@rousseldenis @lmignon Could you please check the failing tests in stock_average_daily_sale module?

@Shide Where ?

[rousseldenis]

@rousseldenis @lmignon Could you please check the failing tests in stock_average_daily_sale module?

@Shide Where ?

Ok, I see. I can check

[rousseldenis]

/ocabot merge nobump

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-321-by-rousseldenis-bump-nobump, awaiting test results.

[OCA-git-bot]

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

[Shide]

@rousseldenis what was happening? What do you do to let tests pass?

[rousseldenis]

@rousseldenis what was happening? What do you do to let tests pass?

Apparently, a temporary failure. Wasn't able to reproduce problem locally, I re run the merge