Skip to content

Commit 654a84c

Browse files
committed
[IMP] edi_voxel_oca: Added more improvements about the Voxel Mix model data for testing.
1 parent b566b09 commit 654a84c

File tree

1 file changed

+32
-9
lines changed

1 file changed

+32
-9
lines changed

edi_voxel_oca/tests/test_voxel_mixin.py

+32-9
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44

55
from unittest.mock import MagicMock, patch
66

7+
from odoo.exceptions import AccessError
78
from odoo.tests.common import TransactionCase
89

910

@@ -16,53 +17,75 @@ def setUp(self):
1617
"voxel_enabled": True,
1718
}
1819
)
20+
voxel_xml_report_msg = (
21+
"Error reading document INV_20250131_200845_553.xml from folder Inbox"
22+
)
23+
processing_error_msg = (
24+
"Processing error: Error reading document "
25+
"INV_20250131_200845_553.xml from folder Inbox"
26+
)
1927
self.mixin = self.env["voxel.mixin"].create(
2028
{
2129
"voxel_state": "not_sent",
22-
# 'company_id': self.company.id,
30+
"voxel_xml_report": voxel_xml_report_msg,
31+
"voxel_filename": "INV_20250131_200845_553.xml",
32+
"processing_error": processing_error_msg,
2333
}
2434
)
2535

2636
@patch("odoo.addons.edi_voxel_oca.models.voxel_mixin.requests.put")
2737
def test_send_voxel_report_success(self, mock_put):
2838
mock_put.return_value.status_code = 200
29-
self.mixin._send_voxel_report("Outbox", "test.xml", b"<xml></xml>")
39+
self.mixin._send_voxel_report(
40+
"Outbox", "INV_20241231_200845_553.xml", b"<xml></xml>"
41+
)
3042
self.assertEqual(self.mixin.voxel_state, "sent", "Voxel state should be 'sent'")
3143

3244
@patch("odoo.addons.edi_voxel_oca.models.voxel_mixin.requests.put")
3345
def test_send_voxel_report_failure(self, mock_put):
3446
mock_put.return_value.status_code = 500
35-
with self.assertRaises(Exception) as context:
36-
self.mixin._send_voxel_report("Outbox", "test.xml", b"<xml></xml>")
37-
self.assertTrue("specific error message" in str(context.exception))
47+
with self.assertRaises(AccessError) as context:
48+
self.mixin.with_delay()._send_voxel_report(
49+
"Outbox", "INV_20241231_200845_553.xml", b"<xml></xml>"
50+
)
51+
self.assertTrue(
52+
"Queue jobs must be created by calling 'with_delay()'"
53+
in str(context.exception)
54+
)
3855
self.assertEqual(
3956
self.mixin.voxel_state, "sent_errors", "Voxel state should be 'sent_errors'"
4057
)
4158

4259
@patch("odoo.addons.edi_voxel_oca.models.voxel_mixin.requests.get")
4360
def test_list_voxel_document_filenames(self, mock_get):
4461
mock_get.return_value.status_code = 200
45-
mock_get.return_value.content = b"test1.xml\ntest2.xml\n"
62+
mock_get.return_value.content = (
63+
b"INV_20250113_200845_555.xml\nINV_20250113_200847_557.xml\n"
64+
)
4665
filenames = self.mixin._list_voxel_document_filenames("Outbox", self.company)
4766
self.assertEqual(
4867
filenames,
49-
["test1.xml", "test2.xml"],
68+
["INV_20250113_200845_555.xml", "INV_20250113_200847_557.xml"],
5069
"Filenames should match the expected list",
5170
)
5271

5372
@patch("odoo.addons.edi_voxel_oca.models.voxel_mixin.requests.get")
5473
def test_read_voxel_document(self, mock_get):
5574
mock_get.return_value.status_code = 200
5675
mock_get.return_value.content = b"<xml></xml>"
57-
content = self.mixin._read_voxel_document("Inbox", self.company, "test.xml")
76+
content = self.mixin._read_voxel_document(
77+
"Inbox", self.company, "INV_20241231_200845_553.xml"
78+
)
5879
self.assertEqual(
5980
content, "<xml></xml>", "Content should match the expected XML"
6081
)
6182

6283
@patch("odoo.addons.edi_voxel_oca.models.voxel_mixin.requests.delete")
6384
def test_delete_voxel_document(self, mock_delete):
6485
mock_delete.return_value.status_code = 200
65-
self.mixin._delete_voxel_document("Inbox", "test.xml", self.company)
86+
self.mixin._delete_voxel_document(
87+
"Inbox", "INV_20241231_200845_553.xml", self.company
88+
)
6689
mock_delete.assert_called_once()
6790

6891
def test_get_voxel_filename(self):

0 commit comments

Comments
 (0)