[FIX] snailmail: better tests & delivery address now visible on report

Tests now check for the 'error' field in Pingen repsonse
snailmail_external_layout.js now targets the right div

closes odoo/odoo#76246

Task: 2588145 & 2583718
Signed-off-by: Florian Daloze (fda) <fda@odoo.com>

addons/snailmail/static/src/js/snailmail_external_layout.js

 // Change address font-size if needed
 document.addEventListener('DOMContentLoaded', function (evt) {
-    var recipientAddress = document.getElementsByClassName('address row')[0].getElementsByTagName('address')[0];
+    var recipientAddress = document.querySelector('.address.row div[name="address"]').getElementsByTagName('address')[0];
     var height = parseFloat(window.getComputedStyle(recipientAddress, null).getPropertyValue('height'));
     var fontSize = parseFloat(window.getComputedStyle(recipientAddress, null).getPropertyValue('font-size'));
     recipientAddress.style.fontSize = (85/height) * fontSize + 'px';

addons/snailmail_account/tests/test_pingen_send.py

@@ -62,14 +62,14 @@ class TestPingenSend(AccountingTestCase):
         }
 
         response = requests.post(self.pingen_url, data=self.data, files=files)
-        if 400 <= response.status_code <= 599:
+        if 400 <= response.status_code <= 599 or response.json()['error']:
             msg = "%(code)s %(side)s Error: %(reason)s for url: %(url)s\n%(body)s" % {
                 'code': response.status_code,
                 'side': r"%s",
                 'reason': response.reason,
                 'url': self.pingen_url,
                 'body': response.text}
-            if response.status_code <= 499:
+            if response.status_code <= 499 or response.json()['error']:
                 raise requests.HTTPError(msg % "Client")
             else:
                 _logger.warning(msg % "Server")