[ADD] [16.0] stock_picking_report_delivery_driver

Shide commented 5 months ago

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 :)

rafaelbn commented 5 months ago

/ocabot merge patch

OCA-git-bot commented 5 months ago

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 commented 5 months ago

@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 commented 5 months ago

/ocabot merge nobump

OCA-git-bot commented 5 months ago

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 commented 5 months ago

@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 commented 5 months ago

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 commented 5 months ago

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

rousseldenis commented 5 months ago

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

@Shide Where ?

rousseldenis commented 5 months ago

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

@Shide Where ?

Ok, I see. I can check

rousseldenis commented 5 months ago

/ocabot merge nobump

OCA-git-bot commented 5 months ago

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 commented 5 months ago

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

Shide commented 5 months ago

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

rousseldenis commented 5 months ago

@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

OCA / stock-logistics-reporting