fixup! Add test

OCA · Oct 10, 2023 · b0e5817 · b0e5817
1 parent d500750
commit b0e5817
Show file tree

Hide file tree

Showing 5 changed files with 4 additions and 6 deletions.
diff --git a/pingen/__init__.py b/pingen/__init__.py
@@ -1,5 +1,4 @@
 # Author: Guewen Baconnier
-# Copyright 2012-2017 Camptocamp SA
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 from . import controllers
 from . import models
diff --git a/pingen/controllers/main.py b/pingen/controllers/main.py
@@ -17,8 +17,6 @@
 
 class PingenController(http.Controller):
     def _verify_signature(self, request_content):
-        if config["test_enable"]:
-            return True
         webhook_signature = http.request.httprequest.headers.get("Signature")
         companies = http.request.env["res.company"].sudo().search([])
         for company in companies:

diff --git a/pingen/models/__init__.py b/pingen/models/__init__.py
@@ -1,4 +1,3 @@
-# Copyright 2018 Camptocamp SA
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 from . import ir_attachment
 from . import pingen

diff --git a/pingen/tests/__init__.py b/pingen/tests/__init__.py
@@ -1,4 +1,3 @@
-# Copyright 2023 Camptocamp SA
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html)
 
 from . import test_pingen
diff --git a/pingen/tests/test_pingen.py b/pingen/tests/test_pingen.py
@@ -197,7 +197,10 @@ def test_pingen_push_document(self):
                 pingen_document.pingen_uuid
             )
             mock_request.httprequest.method = "POST"
-            self.pc.letter_issues()
+            # avoid checking signature when calling webhooks
+            with patch("odoo.addons.pingen.controllers.main.PingenController._verify_signature") as mocked_verify_sign:
+                mocked_verify_sign.return_value = True
+                self.pc.letter_issues()
 
         self.assertEqual(pingen_document.state, "pingen_error")
         self.assertEqual(pingen_document.last_error_message, "Validation failed")