diff --git a/markdown/TEST-REPORT.md b/markdown/TEST-REPORT.md index c667ccb1..d674f534 100644 --- a/markdown/TEST-REPORT.md +++ b/markdown/TEST-REPORT.md @@ -1,20 +1,21 @@ # πŸ§ͺ Test Report -*Generated on 2025-08-12 14:28:17 CEST* +*Generated on 2025-08-12 14:30:40 CEST* ## 🧾 General Info -- **duration**: 4.76562237739563 +- **duration**: 4.833057641983032 - **root**: /workspace/tligui_y/slic - **environment**: {} ## πŸ“‹ Summary -- **Failed**: 6 +- **Failed**: 3 +- **Passed**: 3 - **Total**: 6 - **Collected**: 6 ## πŸ”Ž Tests
-❌ Failed (6) +❌ Failed (3) -
πŸ“„ test_utils_elog.py @@ -28,7 +29,7 @@ **duration:** ```python - 0.0003777281381189823 + 0.0005297278985381126 ``` **outcome:** @@ -42,7 +43,7 @@ **duration:** ```python - 0.009992022067308426 + 0.008404782973229885 ``` **outcome:** @@ -93,7 +94,7 @@ tests/test_utils_elog.py:45: _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ - self = + self = message = 'Hello from local test', msg_id = None, reply = False attributes = {'Author': 'robot', 'Category': 'General', 'Encoding': 'HTML', 'Subject': 'Test simple', ...} attachments = [], suppress_email_notification = False, encoding = 'HTML' @@ -319,343 +320,7 @@ **duration:** ```python - 0.0003286711871623993 - ``` - - **outcome:** - - ```python - passed - ``` -
- ↳ Function: test_get_default_elog_instance_with_direct_password_and_real_check - -
- ❌ Test 2 - - **_*πŸ“Œ Setup phase*_** - - **duration:** - - ```python - 0.0001472141593694687 - ``` - - **outcome:** - - ```python - passed - ``` - - **_*πŸ“Œ Call phase*_** - - **duration:** - - ```python - 0.0002653147093951702 - ``` - - **outcome:** - - ```python - failed - ``` - - **crash:** - - ```python - path: /workspace/tligui_y/slic/slic/utils/elog.py - lineno: 39 - message: NameError: name 'elog' is not defined - ``` - - **traceback:** - - ```python - - path: tests/test_utils_elog.py - lineno: 58 - message: None - - path: slic/utils/elog.py - lineno: 39 - message: NameError - ``` - - **longrepr:** - - ```python - def test_get_default_elog_instance_with_direct_password_and_real_check(): - url = "http://localhost:8080/demo" - user = "robot" - password = "testpassword" - - > elog_instance, returned_user = get_default_elog_instance(url, user=user, password=password) - - tests/test_utils_elog.py:58: - _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ - - url = 'http://localhost:8080/demo' - kwargs = {'password': 'testpassword', 'user': 'robot'}, user = 'robot' - - def get_default_elog_instance(url, **kwargs): - kwargs.setdefault("user", getuser()) - user = kwargs["user"] - - if "password" not in kwargs: - try: - home = Path.home() - fn = home / ".elog_psi" - with fn.open() as f: - pw = f.read().strip() - except Exception: - print(f"Enter elog password for user: {user}") - pw = getpass() - kwargs["password"] = pw - - > return elog.open(url, **kwargs), user - E NameError: name 'elog' is not defined - - slic/utils/elog.py:39: NameError - ``` - - **_*πŸ“Œ Teardown phase*_** - - **duration:** - - ```python - 0.000166354700922966 - ``` - - **outcome:** - - ```python - passed - ``` -
- ↳ Function: test_get_default_elog_instance_asks_password_and_opens - -
- ❌ Test 3 - - **_*πŸ“Œ Setup phase*_** - - **duration:** - - ```python - 0.00012920983135700226 - ``` - - **outcome:** - - ```python - passed - ``` - - **_*πŸ“Œ Call phase*_** - - **duration:** - - ```python - 0.001176837831735611 - ``` - - **outcome:** - - ```python - failed - ``` - - **crash:** - - ```python - path: /workspace/tligui_y/slic/slic/utils/elog.py - lineno: 39 - message: NameError: name 'elog' is not defined - ``` - - **traceback:** - - ```python - - path: tests/test_utils_elog.py - lineno: 75 - message: None - - path: slic/utils/elog.py - lineno: 39 - message: NameError - ``` - - **longrepr:** - - ```python - mock_home = - mock_getpass = - - @patch("slic.utils.elog.getpass") - @patch("slic.utils.elog.Path.home") - def test_get_default_elog_instance_asks_password_and_opens(mock_home, mock_getpass): - mock_home.return_value = Path("/does/not/exist") # Fausse home β†’ lecture Γ©choue - mock_getpass.return_value = "testpassword" - - url = "http://localhost:8080/demo" - user = "robot" - - > elog_instance, returned_user = get_default_elog_instance(url, user=user) - - tests/test_utils_elog.py:75: - _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ - - url = 'http://localhost:8080/demo' - kwargs = {'password': 'testpassword', 'user': 'robot'}, user = 'robot' - home = PosixPath('/does/not/exist'), fn = PosixPath('/does/not/exist/.elog_psi') - pw = 'testpassword' - - def get_default_elog_instance(url, **kwargs): - kwargs.setdefault("user", getuser()) - user = kwargs["user"] - - if "password" not in kwargs: - try: - home = Path.home() - fn = home / ".elog_psi" - with fn.open() as f: - pw = f.read().strip() - except Exception: - print(f"Enter elog password for user: {user}") - pw = getpass() - kwargs["password"] = pw - - > return elog.open(url, **kwargs), user - E NameError: name 'elog' is not defined - - slic/utils/elog.py:39: NameError - ``` - - **_*πŸ“Œ Teardown phase*_** - - **duration:** - - ```python - 0.0001467810943722725 - ``` - - **outcome:** - - ```python - passed - ``` -
- ↳ Function: test_get_default_elog_with_path_home - -
- ❌ Test 4 - - **_*πŸ“Œ Setup phase*_** - - **duration:** - - ```python - 0.00011813687160611153 - ``` - - **outcome:** - - ```python - passed - ``` - - **_*πŸ“Œ Call phase*_** - - **duration:** - - ```python - 0.0015169298276305199 - ``` - - **outcome:** - - ```python - failed - ``` - - **crash:** - - ```python - path: /workspace/tligui_y/slic/slic/utils/elog.py - lineno: 39 - message: NameError: name 'elog' is not defined - ``` - - **traceback:** - - ```python - - path: tests/test_utils_elog.py - lineno: 102 - message: None - - path: slic/utils/elog.py - lineno: 39 - message: NameError - ``` - - **longrepr:** - - ```python - mock_home = - mock_getuser = - mock_getpass = - - @patch("slic.utils.elog.getpass") - @patch("slic.utils.elog.getuser") - @patch("slic.utils.elog.Path.home") - def test_get_default_elog_with_path_home(mock_home, mock_getuser, mock_getpass): - fake_user = "robot" - fake_pw = "testpassword" - mock_getuser.return_value = fake_user - mock_getpass.return_value = fake_pw # fallback safety - - tmp_home = Path("/tmp/fake_home_for_robot") - tmp_home.mkdir(parents=True, exist_ok=True) - pw_file = tmp_home / ".elog_psi" - pw_file.write_text(fake_pw) - mock_home.return_value = tmp_home - - url = "http://localhost:8080/demo" - - try: - > elog_instance, returned_user = get_default_elog_instance(url) - - tests/test_utils_elog.py:102: - _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ - - url = 'http://localhost:8080/demo' - kwargs = {'password': 'testpassword', 'user': 'robot'}, user = 'robot' - home = PosixPath('/tmp/fake_home_for_robot') - fn = PosixPath('/tmp/fake_home_for_robot/.elog_psi') - f = <_io.TextIOWrapper name='/tmp/fake_home_for_robot/.elog_psi' mode='r' encoding='UTF-8'> - pw = 'testpassword' - - def get_default_elog_instance(url, **kwargs): - kwargs.setdefault("user", getuser()) - user = kwargs["user"] - - if "password" not in kwargs: - try: - home = Path.home() - fn = home / ".elog_psi" - with fn.open() as f: - pw = f.read().strip() - except Exception: - print(f"Enter elog password for user: {user}") - pw = getpass() - kwargs["password"] = pw - - > return elog.open(url, **kwargs), user - E NameError: name 'elog' is not defined - - slic/utils/elog.py:39: NameError - ``` - - **_*πŸ“Œ Teardown phase*_** - - **duration:** - - ```python - 0.0001523476094007492 + 0.00036900071427226067 ``` **outcome:** @@ -673,7 +338,7 @@ **duration:** ```python - 0.0001242789439857006 + 0.00012351106852293015 ``` **outcome:** @@ -687,7 +352,7 @@ **duration:** ```python - 0.00016640778630971909 + 0.005688583943992853 ``` **outcome:** @@ -699,64 +364,214 @@ **crash:** ```python - path: /workspace/tligui_y/slic/slic/utils/elog.py - lineno: 39 - message: NameError: name 'elog' is not defined + path: /workspace/tligui_y/slic/slic/utils/logbook.py + lineno: 276 + message: slic.utils.logbook_exceptions.LogbookInvalidMessageID: Invalid message ID: None ``` **traceback:** ```python - path: tests/test_utils_elog.py - lineno: 116 + lineno: 122 message: None - - path: tests/test_utils_elog.py - lineno: 51 - message: in get_test_elog - path: slic/utils/elog.py - lineno: 11 - message: in __init__ - - path: slic/utils/elog.py - lineno: 39 - message: NameError + lineno: 17 + message: in post + - path: slic/utils/logbook.py + lineno: 276 + message: LogbookInvalidMessageID ``` **longrepr:** ```python def test_post(): - > elog = get_test_elog() + elog = get_test_elog() + + title = "AUTHOR_OVERRIDE_TEST" + text = "This is a message" + author = "robot" + + > elog.post(text, attributes = {"Author": author}) - tests/test_utils_elog.py:116: + tests/test_utils_elog.py:122: _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ - tests/test_utils_elog.py:51: in get_test_elog - return Elog("http://localhost:8080/demo", user="robot", password="testpassword") - slic/utils/elog.py:11: in __init__ - self._log, self.user = get_default_elog_instance(url, **kwargs) + slic/utils/elog.py:17: in post + return self._log.post(*args, **kwargs) _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ - url = 'http://localhost:8080/demo' - kwargs = {'password': 'testpassword', 'user': 'robot'}, user = 'robot' + self = + message = 'This is a message', msg_id = None, reply = False + attributes = {'Author': 'robot', 'When': 1755001837, 'cmd': 'Submit', 'exp': 'demo', ...} + attachments = [], suppress_email_notification = False, encoding = None + timeout = None, kwargs = {'Author': 'robot'} + new_attachment_list = [('Text', ('', b'This is a message'))] + objects_to_close = [] + attributes_to_edit = {'Author': b'robot', 'When': 1755001837, 'cmd': b'Submit', 'exp': b'demo', ...} - def get_default_elog_instance(url, **kwargs): - kwargs.setdefault("user", getuser()) - user = kwargs["user"] + def post(self, message, msg_id=None, reply=False, attributes=None, attachments=None, + suppress_email_notification=False, encoding=None, timeout=None, **kwargs): + """ + Posts message to the logbook. If msg_id is not specified new message will be created, otherwise existing + message will be edited, or a reply (if reply=True) to it will be created. This method returns the msg_id + of the newly created message. + """ - if "password" not in kwargs: - try: - home = Path.home() - fn = home / ".elog_psi" - with fn.open() as f: - pw = f.read().strip() - except Exception: - print(f"Enter elog password for user: {user}") - pw = getpass() - kwargs["password"] = pw + print("STARTING POST") + # Ajout des impressions pour dΓ©boguer + print(f"Message to post: {message}") + print(f"msg_id: {msg_id}") + print(f"Attributes: {attributes}") + print(f"Attachments: {attachments}") + print(f"Encoding: {encoding}") + print(f"Timeout: {timeout}") + print(f"Additional kwargs: {kwargs}") - > return elog.open(url, **kwargs), user - E NameError: name 'elog' is not defined + attributes = attributes or {} + attributes = {**attributes, **kwargs} # kwargs as attributes with higher priority + print(f"Updated attributes: {attributes}") + + attachments = attachments or [] + print(f"Attachments list: {attachments}") + + if encoding is not None: + if encoding not in ['plain', 'HTML', 'ELCode']: + raise LogbookMessageRejected('Invalid message encoding. Valid options: plain, HTML, ELCode.') + attributes['Encoding'] = encoding + + if suppress_email_notification: + attributes["suppress"] = 1 + + # Prepare attachments + if attachments: + new_attachment_list, objects_to_close = self._prepare_attachments(attachments) + print(f"New attachments prepared: {new_attachment_list}") + else: + objects_to_close = [] + new_attachment_list = [] + + attributes_to_edit = dict() + + if msg_id: + print(f"Editing message with msg_id: {msg_id}") + if reply: + print(f"Replying to message with msg_id: {msg_id}") + self._check_if_message_on_server(msg_id) + attributes['reply_to'] = str(msg_id) + else: + print("Editing existing message.") + attributes['edit_id'] = str(msg_id) + attributes['skiplock'] = '1' + msg_to_edit, attributes_to_edit, existing_attachments_list = self.read(msg_id) + + # Merge new attributes + for attribute, data in attributes.items(): + if data is not None: + attributes_to_edit[attribute] = data + + print(f"Attributes after merging: {attributes_to_edit}") + + # Process existing attachments + i = 0 + existing_attachments_filename_list = [] + for attachment in existing_attachments_list: + attributes_to_edit[f'attachment{i}'] = os.path.basename(attachment) + existing_attachments_filename_list.append(os.path.basename(attachment)[14:]) + i += 1 + + print(f"Existing attachments: {existing_attachments_filename_list}") + + duplicate_attachment_list = [] + for new_attachment in new_attachment_list: + new_attachment_filename = new_attachment[1][0] + print(f"Checking new attachment: {new_attachment_filename}") + if new_attachment_filename in existing_attachments_filename_list: + # Same attachment exists on the server, compare content + new_attachment_content = new_attachment[1][1].read() + new_attachment[1][1].seek(0) + attachment_index = existing_attachments_filename_list.index(new_attachment_filename) + existing_attachment_content = self.download_attachment( + url=existing_attachments_list[attachment_index], + timeout=timeout + ) + if new_attachment_content == existing_attachment_content: + print(f"Duplicate attachment detected: {new_attachment_filename}") + duplicate_attachment_list.append(new_attachment) + else: + print(f"Attachment content has changed: {new_attachment_filename}") + self.delete_attachment(msg_id, attributes=attributes_to_edit, + attachment_id=attachment_index, + timeout=timeout, text=msg_to_edit) + existing_attachments_filename_list.pop(attachment_index) + existing_attachments_list.pop(attachment_index) + + print(f"Duplicate attachments to remove: {duplicate_attachment_list}") + + # Remove duplicates + for attach in duplicate_attachment_list: + new_attachment_list.remove(attach) + + print(f"Final new attachments list: {new_attachment_list}") + else: + # Creating a new message, add timestamp if not present + if 'When' not in attributes: + attributes['When'] = int(datetime.now().timestamp()) + + # Final check on attributes + if not attributes_to_edit: + attributes_to_edit = attributes + + print(f"Final attributes to send to the server: {attributes_to_edit}") + + # Remove reserved attributes + _remove_reserved_attributes(attributes_to_edit) + + new_attachment_list.append(('Text', ('', message.encode('iso-8859-1')))) + print(f"Final attachment list including message text: {new_attachment_list}") + + # Add base message attributes + self._add_base_msg_attributes(attributes_to_edit) + print(f"Attributes with base message added: {attributes_to_edit}") + + # Sanitize attribute keys + attributes_to_edit = _replace_special_characters_in_attribute_keys(attributes_to_edit) + print(f"Attributes after sanitizing keys: {attributes_to_edit}") + + # Encode all string values in latin1 + attributes_to_edit = _encode_values(attributes_to_edit) + print(f"Attributes after encoding: {attributes_to_edit}") + + try: + print("Sending POST request to the server...") + response = requests.post(self._url, data=attributes_to_edit, files=new_attachment_list, + allow_redirects=False, verify=False, timeout=timeout) + print("Response received:", response) + + resp_message, resp_headers, resp_msg_id = _validate_response(response) + print(f"Response message: {resp_message}") + print(f"Response headers: {resp_headers}") + print(f"Response msg_id: {resp_msg_id}") + + # Close file-like objects if needed + for file_like_object in objects_to_close: + if hasattr(file_like_object, 'close'): + file_like_object.close() + + except requests.Timeout as e: + print("Request timed out:", e) + raise LogbookServerTimeout(f"Timeout while posting: {e}") + + except requests.RequestException as e: + print("Request exception:", e) + self._check_if_message_on_server(msg_id) # Raises exceptions if no message or no response from server + raise LogbookServerProblem(f"Cannot access logbook server: {e}") + + if not resp_msg_id or resp_msg_id < 1: + > raise LogbookInvalidMessageID(f"Invalid message ID: {resp_msg_id}") + E slic.utils.logbook_exceptions.LogbookInvalidMessageID: Invalid message ID: None - slic/utils/elog.py:39: NameError + slic/utils/logbook.py:276: LogbookInvalidMessageID ``` **_*πŸ“Œ Teardown phase*_** @@ -764,7 +579,7 @@ **duration:** ```python - 0.00014013797044754028 + 0.00018636509776115417 ``` **outcome:** @@ -782,7 +597,7 @@ **duration:** ```python - 0.00011991709470748901 + 0.00013091834262013435 ``` **outcome:** @@ -796,7 +611,7 @@ **duration:** ```python - 0.0015173987485468388 + 0.007740779779851437 ``` **outcome:** @@ -808,32 +623,32 @@ **crash:** ```python - path: /workspace/tligui_y/slic/slic/utils/elog.py - lineno: 39 - message: NameError: name 'elog' is not defined + path: /workspace/tligui_y/slic/slic/utils/logbook.py + lineno: 276 + message: slic.utils.logbook_exceptions.LogbookInvalidMessageID: Invalid message ID: None ``` **traceback:** ```python - path: tests/test_utils_elog.py - lineno: 140 + lineno: 143 message: None - - path: tests/test_utils_elog.py - lineno: 51 - message: in get_test_elog - path: slic/utils/elog.py - lineno: 11 - message: in __init__ + lineno: 22 + message: in screenshot - path: slic/utils/elog.py - lineno: 39 - message: NameError + lineno: 17 + message: in post + - path: slic/utils/logbook.py + lineno: 276 + message: LogbookInvalidMessageID ``` **longrepr:** ```python - mock_screenshot_class = + mock_screenshot_class = @patch("slic.utils.elog.Screenshot") def test_screenshot(mock_screenshot_class): @@ -844,38 +659,190 @@ mock_instance = mock_screenshot_class.return_value mock_instance.shoot.return_value = [fake_path] - > elog = get_test_elog() + elog = get_test_elog() + + test_msg = "SCREENSHOT_INTEGRATION_TEST_MSG_456" + > entry_id = elog.screenshot(message=test_msg) - tests/test_utils_elog.py:140: + tests/test_utils_elog.py:143: _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ - tests/test_utils_elog.py:51: in get_test_elog - return Elog("http://localhost:8080/demo", user="robot", password="testpassword") - slic/utils/elog.py:11: in __init__ - self._log, self.user = get_default_elog_instance(url, **kwargs) + slic/utils/elog.py:22: in screenshot + self.post(message, **kwargs) + slic/utils/elog.py:17: in post + return self._log.post(*args, **kwargs) _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ - url = 'http://localhost:8080/demo' - kwargs = {'password': 'testpassword', 'user': 'robot'}, user = 'robot' + self = + message = 'SCREENSHOT_INTEGRATION_TEST_MSG_456', msg_id = None, reply = False + attributes = {'Author': 'robot', 'When': 1755001837, 'cmd': 'Submit', 'exp': 'demo', ...} + attachments = ['/tmp/tmp1jhsgy5u.png'], suppress_email_notification = False + encoding = None, timeout = None, kwargs = {'Author': 'robot'} + new_attachment_list = [('attfile0', ('tmp1jhsgy5u.png', <_io.BufferedReader name='/tmp/tmp1jhsgy5u.png'>)), ('Text', ('', b'SCREENSHOT_INTEGRATION_TEST_MSG_456'))] + objects_to_close = [<_io.BufferedReader name='/tmp/tmp1jhsgy5u.png'>] + attributes_to_edit = {'Author': b'robot', 'When': 1755001837, 'cmd': b'Submit', 'exp': b'demo', ...} - def get_default_elog_instance(url, **kwargs): - kwargs.setdefault("user", getuser()) - user = kwargs["user"] + def post(self, message, msg_id=None, reply=False, attributes=None, attachments=None, + suppress_email_notification=False, encoding=None, timeout=None, **kwargs): + """ + Posts message to the logbook. If msg_id is not specified new message will be created, otherwise existing + message will be edited, or a reply (if reply=True) to it will be created. This method returns the msg_id + of the newly created message. + """ - if "password" not in kwargs: - try: - home = Path.home() - fn = home / ".elog_psi" - with fn.open() as f: - pw = f.read().strip() - except Exception: - print(f"Enter elog password for user: {user}") - pw = getpass() - kwargs["password"] = pw + print("STARTING POST") + # Ajout des impressions pour dΓ©boguer + print(f"Message to post: {message}") + print(f"msg_id: {msg_id}") + print(f"Attributes: {attributes}") + print(f"Attachments: {attachments}") + print(f"Encoding: {encoding}") + print(f"Timeout: {timeout}") + print(f"Additional kwargs: {kwargs}") - > return elog.open(url, **kwargs), user - E NameError: name 'elog' is not defined + attributes = attributes or {} + attributes = {**attributes, **kwargs} # kwargs as attributes with higher priority + print(f"Updated attributes: {attributes}") + + attachments = attachments or [] + print(f"Attachments list: {attachments}") + + if encoding is not None: + if encoding not in ['plain', 'HTML', 'ELCode']: + raise LogbookMessageRejected('Invalid message encoding. Valid options: plain, HTML, ELCode.') + attributes['Encoding'] = encoding + + if suppress_email_notification: + attributes["suppress"] = 1 + + # Prepare attachments + if attachments: + new_attachment_list, objects_to_close = self._prepare_attachments(attachments) + print(f"New attachments prepared: {new_attachment_list}") + else: + objects_to_close = [] + new_attachment_list = [] + + attributes_to_edit = dict() + + if msg_id: + print(f"Editing message with msg_id: {msg_id}") + if reply: + print(f"Replying to message with msg_id: {msg_id}") + self._check_if_message_on_server(msg_id) + attributes['reply_to'] = str(msg_id) + else: + print("Editing existing message.") + attributes['edit_id'] = str(msg_id) + attributes['skiplock'] = '1' + msg_to_edit, attributes_to_edit, existing_attachments_list = self.read(msg_id) + + # Merge new attributes + for attribute, data in attributes.items(): + if data is not None: + attributes_to_edit[attribute] = data + + print(f"Attributes after merging: {attributes_to_edit}") + + # Process existing attachments + i = 0 + existing_attachments_filename_list = [] + for attachment in existing_attachments_list: + attributes_to_edit[f'attachment{i}'] = os.path.basename(attachment) + existing_attachments_filename_list.append(os.path.basename(attachment)[14:]) + i += 1 + + print(f"Existing attachments: {existing_attachments_filename_list}") + + duplicate_attachment_list = [] + for new_attachment in new_attachment_list: + new_attachment_filename = new_attachment[1][0] + print(f"Checking new attachment: {new_attachment_filename}") + if new_attachment_filename in existing_attachments_filename_list: + # Same attachment exists on the server, compare content + new_attachment_content = new_attachment[1][1].read() + new_attachment[1][1].seek(0) + attachment_index = existing_attachments_filename_list.index(new_attachment_filename) + existing_attachment_content = self.download_attachment( + url=existing_attachments_list[attachment_index], + timeout=timeout + ) + if new_attachment_content == existing_attachment_content: + print(f"Duplicate attachment detected: {new_attachment_filename}") + duplicate_attachment_list.append(new_attachment) + else: + print(f"Attachment content has changed: {new_attachment_filename}") + self.delete_attachment(msg_id, attributes=attributes_to_edit, + attachment_id=attachment_index, + timeout=timeout, text=msg_to_edit) + existing_attachments_filename_list.pop(attachment_index) + existing_attachments_list.pop(attachment_index) + + print(f"Duplicate attachments to remove: {duplicate_attachment_list}") + + # Remove duplicates + for attach in duplicate_attachment_list: + new_attachment_list.remove(attach) + + print(f"Final new attachments list: {new_attachment_list}") + else: + # Creating a new message, add timestamp if not present + if 'When' not in attributes: + attributes['When'] = int(datetime.now().timestamp()) + + # Final check on attributes + if not attributes_to_edit: + attributes_to_edit = attributes + + print(f"Final attributes to send to the server: {attributes_to_edit}") + + # Remove reserved attributes + _remove_reserved_attributes(attributes_to_edit) + + new_attachment_list.append(('Text', ('', message.encode('iso-8859-1')))) + print(f"Final attachment list including message text: {new_attachment_list}") + + # Add base message attributes + self._add_base_msg_attributes(attributes_to_edit) + print(f"Attributes with base message added: {attributes_to_edit}") + + # Sanitize attribute keys + attributes_to_edit = _replace_special_characters_in_attribute_keys(attributes_to_edit) + print(f"Attributes after sanitizing keys: {attributes_to_edit}") + + # Encode all string values in latin1 + attributes_to_edit = _encode_values(attributes_to_edit) + print(f"Attributes after encoding: {attributes_to_edit}") + + try: + print("Sending POST request to the server...") + response = requests.post(self._url, data=attributes_to_edit, files=new_attachment_list, + allow_redirects=False, verify=False, timeout=timeout) + print("Response received:", response) + + resp_message, resp_headers, resp_msg_id = _validate_response(response) + print(f"Response message: {resp_message}") + print(f"Response headers: {resp_headers}") + print(f"Response msg_id: {resp_msg_id}") + + # Close file-like objects if needed + for file_like_object in objects_to_close: + if hasattr(file_like_object, 'close'): + file_like_object.close() + + except requests.Timeout as e: + print("Request timed out:", e) + raise LogbookServerTimeout(f"Timeout while posting: {e}") + + except requests.RequestException as e: + print("Request exception:", e) + self._check_if_message_on_server(msg_id) # Raises exceptions if no message or no response from server + raise LogbookServerProblem(f"Cannot access logbook server: {e}") + + if not resp_msg_id or resp_msg_id < 1: + > raise LogbookInvalidMessageID(f"Invalid message ID: {resp_msg_id}") + E slic.utils.logbook_exceptions.LogbookInvalidMessageID: Invalid message ID: None - slic/utils/elog.py:39: NameError + slic/utils/logbook.py:276: LogbookInvalidMessageID ``` **_*πŸ“Œ Teardown phase*_** @@ -883,7 +850,155 @@ **duration:** ```python - 0.00016004778444766998 + 0.00018721073865890503 + ``` + + **outcome:** + + ```python + passed + ``` +
+
+ + + +
+βœ… Passed (3) + + -
+ πŸ“„ test_utils_elog.py + + ↳ Function: test_get_default_elog_instance_with_direct_password_and_real_check + -
+ βœ… Test 2 + + **_*πŸ“Œ Setup phase*_** + + **duration:** + + ```python + 0.00014514895156025887 + ``` + + **outcome:** + + ```python + passed + ``` + + **_*πŸ“Œ Call phase*_** + + **duration:** + + ```python + 0.009522773325443268 + ``` + + **outcome:** + + ```python + passed + ``` + + **_*πŸ“Œ Teardown phase*_** + + **duration:** + + ```python + 0.00015535438433289528 + ``` + + **outcome:** + + ```python + passed + ``` +
+ ↳ Function: test_get_default_elog_instance_asks_password_and_opens + -
+ βœ… Test 3 + + **_*πŸ“Œ Setup phase*_** + + **duration:** + + ```python + 0.0001318720169365406 + ``` + + **outcome:** + + ```python + passed + ``` + + **_*πŸ“Œ Call phase*_** + + **duration:** + + ```python + 0.009336789138615131 + ``` + + **outcome:** + + ```python + passed + ``` + + **_*πŸ“Œ Teardown phase*_** + + **duration:** + + ```python + 0.0001459629274904728 + ``` + + **outcome:** + + ```python + passed + ``` +
+ ↳ Function: test_get_default_elog_with_path_home + -
+ βœ… Test 4 + + **_*πŸ“Œ Setup phase*_** + + **duration:** + + ```python + 0.000128098763525486 + ``` + + **outcome:** + + ```python + passed + ``` + + **_*πŸ“Œ Call phase*_** + + **duration:** + + ```python + 0.009406064637005329 + ``` + + **outcome:** + + ```python + passed + ``` + + **_*πŸ“Œ Teardown phase*_** + + **duration:** + + ```python + 0.00014375103637576103 ``` **outcome:** diff --git a/markdown/coverage-summary.md b/markdown/coverage-summary.md index c02fd84e..c1f07d87 100644 --- a/markdown/coverage-summary.md +++ b/markdown/coverage-summary.md @@ -8779,3 +8779,230 @@ | slic/utils/utils.py | 17 | 8 | 53% | | slic/utils/xrange.py | 33 | 30 | 9% | | **TOTAL** | **9942** | **7348** | **26%** | +| Name | Stmts | Miss | Cover | +|----------------------------------------------- | -------: | -------: | ------: | +| slic/\_\_init\_\_.py | 20 | 2 | 90% | +| slic/core/\_\_init\_\_.py | 5 | 0 | 100% | +| slic/core/acquisition/\_\_init\_\_.py | 7 | 0 | 100% | +| slic/core/acquisition/acquisition.py | 56 | 42 | 25% | +| slic/core/acquisition/baseacquisition.py | 5 | 1 | 80% | +| slic/core/acquisition/broker/\_\_init\_\_.py | 2 | 0 | 100% | +| slic/core/acquisition/broker/brokerclient.py | 104 | 80 | 23% | +| slic/core/acquisition/broker/brokerconfig.py | 102 | 87 | 15% | +| slic/core/acquisition/broker/pedestal.py | 66 | 57 | 14% | +| slic/core/acquisition/broker/pids.py | 17 | 12 | 29% | +| slic/core/acquisition/broker/post\_retrieve.py | 120 | 120 | 0% | +| slic/core/acquisition/broker/requeststatus.py | 77 | 73 | 5% | +| slic/core/acquisition/broker/restapi.py | 147 | 107 | 27% | +| slic/core/acquisition/broker/tools.py | 52 | 18 | 65% | +| slic/core/acquisition/bsacquisition.py | 10 | 3 | 70% | +| slic/core/acquisition/bschannels.py | 30 | 19 | 37% | +| slic/core/acquisition/channels.py | 51 | 30 | 41% | +| slic/core/acquisition/dbacquisition.py | 19 | 12 | 37% | +| slic/core/acquisition/detcfg.py | 82 | 35 | 57% | +| slic/core/acquisition/diaacquisition.py | 111 | 111 | 0% | +| slic/core/acquisition/diaconfig.py | 36 | 36 | 0% | +| slic/core/acquisition/dummyacquisition.py | 12 | 12 | 0% | +| slic/core/acquisition/fakeacquisition.py | 76 | 53 | 30% | +| slic/core/acquisition/pedestals.py | 69 | 69 | 0% | +| slic/core/acquisition/pvacquisition.py | 60 | 47 | 22% | +| slic/core/acquisition/pvchannels.py | 13 | 7 | 46% | +| slic/core/acquisition/sfacquisition.py | 159 | 129 | 19% | +| slic/core/acquisition/sfpaths.py | 23 | 19 | 17% | +| slic/core/acquisition/spreadsheet.py | 45 | 45 | 0% | +| slic/core/adjustable/\_\_init\_\_.py | 11 | 0 | 100% | +| slic/core/adjustable/adjustable.py | 50 | 30 | 40% | +| slic/core/adjustable/baseadjustable.py | 28 | 18 | 36% | +| slic/core/adjustable/collection.py | 22 | 14 | 36% | +| slic/core/adjustable/combined.py | 15 | 8 | 47% | +| slic/core/adjustable/convenience.py | 35 | 20 | 43% | +| slic/core/adjustable/converted.py | 16 | 10 | 38% | +| slic/core/adjustable/dummyadjustable.py | 41 | 30 | 27% | +| slic/core/adjustable/error.py | 2 | 0 | 100% | +| slic/core/adjustable/genericadjustable.py | 32 | 24 | 25% | +| slic/core/adjustable/limited.py | 29 | 18 | 38% | +| slic/core/adjustable/linked.py | 22 | 15 | 32% | +| slic/core/adjustable/pvadjustable.py | 119 | 91 | 24% | +| slic/core/adjustable/pvchangemon.py | 77 | 56 | 27% | +| slic/core/adjustable/pvenumadjustable.py | 38 | 22 | 42% | +| slic/core/adjustable/scaler.py | 22 | 16 | 27% | +| slic/core/condition/\_\_init\_\_.py | 2 | 0 | 100% | +| slic/core/condition/basecondition.py | 8 | 2 | 75% | +| slic/core/condition/condition.py | 107 | 79 | 26% | +| slic/core/condition/pvcondition.py | 21 | 12 | 43% | +| slic/core/condition/valuecondition.py | 22 | 15 | 32% | +| slic/core/device/\_\_init\_\_.py | 2 | 0 | 100% | +| slic/core/device/auto.py | 12 | 12 | 0% | +| slic/core/device/basedevice.py | 2 | 0 | 100% | +| slic/core/device/device.py | 46 | 35 | 24% | +| slic/core/device/filtered.py | 23 | 23 | 0% | +| slic/core/device/simpledevice.py | 6 | 2 | 67% | +| slic/core/scanner/\_\_init\_\_.py | 1 | 0 | 100% | +| slic/core/scanner/runname.py | 36 | 23 | 36% | +| slic/core/scanner/scanbackend.py | 232 | 197 | 15% | +| slic/core/scanner/scaninfo.py | 45 | 35 | 22% | +| slic/core/scanner/scanner.py | 136 | 89 | 35% | +| slic/core/sensor/\_\_init\_\_.py | 8 | 0 | 100% | +| slic/core/sensor/basesensor.py | 12 | 3 | 75% | +| slic/core/sensor/bscombined.py | 9 | 5 | 44% | +| slic/core/sensor/bsmonitor.py | 102 | 73 | 28% | +| slic/core/sensor/bsnorm.py | 12 | 7 | 42% | +| slic/core/sensor/bssensor.py | 6 | 2 | 67% | +| slic/core/sensor/combined.py | 31 | 20 | 35% | +| slic/core/sensor/monitor.py | 62 | 51 | 18% | +| slic/core/sensor/norm.py | 9 | 5 | 44% | +| slic/core/sensor/pvsensor.py | 32 | 20 | 38% | +| slic/core/sensor/remoteplot.py | 15 | 10 | 33% | +| slic/core/sensor/sensor.py | 60 | 42 | 30% | +| slic/core/task/\_\_init\_\_.py | 4 | 0 | 100% | +| slic/core/task/basetask.py | 11 | 3 | 73% | +| slic/core/task/daqtask.py | 23 | 16 | 30% | +| slic/core/task/loop.py | 57 | 40 | 30% | +| slic/core/task/producer.py | 25 | 18 | 28% | +| slic/core/task/task.py | 62 | 46 | 26% | +| slic/devices/\_\_init\_\_.py | 7 | 0 | 100% | +| slic/devices/cameras/\_\_init\_\_.py | 4 | 0 | 100% | +| slic/devices/cameras/basler.py | 8 | 4 | 50% | +| slic/devices/cameras/camera\_bs.py | 13 | 8 | 38% | +| slic/devices/cameras/camera\_ca.py | 34 | 19 | 44% | +| slic/devices/cameras/camerabase.py | 17 | 12 | 29% | +| slic/devices/cameras/screenpanel.py | 31 | 21 | 32% | +| slic/devices/endstations/\_\_init\_\_.py | 3 | 0 | 100% | +| slic/devices/endstations/alvra\_flex.py | 10 | 5 | 50% | +| slic/devices/endstations/alvra\_huber.py | 8 | 4 | 50% | +| slic/devices/endstations/alvra\_prime.py | 48 | 34 | 29% | +| slic/devices/endstations/alvra\_xtg.py | 8 | 8 | 0% | +| slic/devices/endstations/bernina\_cameras.py | 33 | 33 | 0% | +| slic/devices/endstations/bernina\_platform.py | 46 | 46 | 0% | +| slic/devices/general/\_\_init\_\_.py | 4 | 0 | 100% | +| slic/devices/general/delay\_compensation.py | 13 | 13 | 0% | +| slic/devices/general/delay\_stage.py | 57 | 30 | 47% | +| slic/devices/general/detectors/\_\_init\_\_.py | 2 | 0 | 100% | +| slic/devices/general/detectors/buffer.py | 66 | 35 | 47% | +| slic/devices/general/detectors/digitizer.py | 13 | 7 | 46% | +| slic/devices/general/detectors/pvdatastream.py | 33 | 24 | 27% | +| slic/devices/general/detectors/timer.py | 15 | 9 | 40% | +| slic/devices/general/micosstage.py | 7 | 7 | 0% | +| slic/devices/general/motor.py | 128 | 88 | 31% | +| slic/devices/general/shutter.py | 22 | 12 | 45% | +| slic/devices/general/shutterctx.py | 18 | 7 | 61% | +| slic/devices/general/smaract.py | 169 | 125 | 26% | +| slic/devices/loptics/\_\_init\_\_.py | 2 | 0 | 100% | +| slic/devices/loptics/alvra\_explaser.py | 29 | 21 | 28% | +| slic/devices/loptics/bernina\_explaser.py | 28 | 28 | 0% | +| slic/devices/loptics/lasershutter.py | 22 | 14 | 36% | +| slic/devices/timing/\_\_init\_\_.py | 0 | 0 | 100% | +| slic/devices/timing/events/\_\_init\_\_.py | 3 | 3 | 0% | +| slic/devices/timing/events/codes.py | 5 | 5 | 0% | +| slic/devices/timing/events/ctaseq.py | 190 | 190 | 0% | +| slic/devices/timing/events/evr.py | 37 | 37 | 0% | +| slic/devices/timing/events/tma.py | 40 | 40 | 0% | +| slic/devices/timing/lasertiming.py | 253 | 184 | 27% | +| slic/devices/xdiagnostics/\_\_init\_\_.py | 2 | 0 | 100% | +| slic/devices/xdiagnostics/intensitymonitor.py | 124 | 92 | 26% | +| slic/devices/xdiagnostics/profilemonitor.py | 19 | 9 | 53% | +| slic/devices/xdiagnostics/timetools.py | 48 | 48 | 0% | +| slic/devices/xoptics/\_\_init\_\_.py | 7 | 0 | 100% | +| slic/devices/xoptics/aramis\_attenuator.py | 96 | 66 | 31% | +| slic/devices/xoptics/aramis\_reflaser.py | 23 | 15 | 35% | +| slic/devices/xoptics/dcm.py | 211 | 162 | 23% | +| slic/devices/xoptics/kb.py | 30 | 18 | 40% | +| slic/devices/xoptics/offsetmirrors.py | 9 | 5 | 44% | +| slic/devices/xoptics/pulsepicker.py | 56 | 34 | 39% | +| slic/devices/xoptics/slits/\_\_init\_\_.py | 5 | 0 | 100% | +| slic/devices/xoptics/slits/slitblades.py | 66 | 48 | 27% | +| slic/devices/xoptics/slits/slittwinunit.py | 12 | 6 | 50% | +| slic/devices/xoptics/slits/slitunit.py | 14 | 8 | 43% | +| slic/devices/xoptics/slits/slitunitcw.py | 7 | 3 | 57% | +| slic/devices/xoptics/slits/slitunitjj.py | 8 | 5 | 38% | +| slic/gui/\_\_init\_\_.py | 1 | 0 | 100% | +| slic/gui/daqframe.py | 75 | 54 | 28% | +| slic/gui/daqpanels/\_\_init\_\_.py | 6 | 0 | 100% | +| slic/gui/daqpanels/config.py | 98 | 80 | 18% | +| slic/gui/daqpanels/goto.py | 92 | 76 | 17% | +| slic/gui/daqpanels/run.py | 56 | 46 | 18% | +| slic/gui/daqpanels/scan2d.py | 77 | 66 | 14% | +| slic/gui/daqpanels/scan.py | 63 | 54 | 14% | +| slic/gui/daqpanels/sfx.py | 77 | 60 | 22% | +| slic/gui/daqpanels/special.py | 63 | 54 | 14% | +| slic/gui/daqpanels/static.py | 46 | 37 | 20% | +| slic/gui/daqpanels/tools.py | 140 | 114 | 19% | +| slic/gui/daqpanels/tweak.py | 149 | 127 | 15% | +| slic/gui/gui.py | 16 | 10 | 38% | +| slic/gui/icon.py | 8 | 4 | 50% | +| slic/gui/persist.py | 68 | 48 | 29% | +| slic/gui/widgets/\_\_init\_\_.py | 11 | 0 | 100% | +| slic/gui/widgets/alarm.py | 21 | 9 | 57% | +| slic/gui/widgets/alternative.py | 51 | 40 | 22% | +| slic/gui/widgets/boxes.py | 33 | 26 | 21% | +| slic/gui/widgets/checkbox.py | 8 | 4 | 50% | +| slic/gui/widgets/completers.py | 27 | 19 | 30% | +| slic/gui/widgets/dyncombo.py | 49 | 49 | 0% | +| slic/gui/widgets/entries.py | 253 | 195 | 23% | +| slic/gui/widgets/exc2warn.py | 15 | 13 | 13% | +| slic/gui/widgets/fname.py | 60 | 47 | 22% | +| slic/gui/widgets/jfcfg.py | 290 | 230 | 21% | +| slic/gui/widgets/jfmodcoords.py | 88 | 70 | 20% | +| slic/gui/widgets/labeled.py | 19 | 7 | 63% | +| slic/gui/widgets/lists.py | 96 | 73 | 24% | +| slic/gui/widgets/mods.py | 25 | 17 | 32% | +| slic/gui/widgets/nope.py | 26 | 19 | 27% | +| slic/gui/widgets/plotting.py | 68 | 47 | 31% | +| slic/gui/widgets/tools.py | 11 | 7 | 36% | +| slic/gui/widgets/twobuttons.py | 43 | 30 | 30% | +| slic/gui/wxdebug.py | 15 | 7 | 53% | +| slic/utils/\_\_init\_\_.py | 24 | 0 | 100% | +| slic/utils/argfwd.py | 53 | 14 | 74% | +| slic/utils/ask\_yes\_no.py | 27 | 20 | 26% | +| slic/utils/channels.py | 17 | 12 | 29% | +| slic/utils/config.py | 5 | 2 | 60% | +| slic/utils/cprint.py | 41 | 16 | 61% | +| slic/utils/dbusnotify.py | 40 | 25 | 38% | +| slic/utils/debug.py | 16 | 12 | 25% | +| slic/utils/dictext.py | 30 | 19 | 37% | +| slic/utils/dotdir.py | 10 | 1 | 90% | +| slic/utils/duo.py | 77 | 45 | 42% | +| slic/utils/elog.py | 31 | 0 | 100% | +| slic/utils/eval.py | 49 | 37 | 24% | +| slic/utils/exceptions.py | 22 | 14 | 36% | +| slic/utils/get\_adj.py | 17 | 11 | 35% | +| slic/utils/hastyepics.py | 37 | 25 | 32% | +| slic/utils/ioc/\_\_init\_\_.py | 1 | 1 | 0% | +| slic/utils/ioc/adjdrv.py | 31 | 31 | 0% | +| slic/utils/ioc/ioc.py | 63 | 63 | 0% | +| slic/utils/ipy.py | 22 | 15 | 32% | +| slic/utils/jsonext.py | 24 | 16 | 33% | +| slic/utils/lazypv.py | 12 | 12 | 0% | +| slic/utils/logbook.py | 409 | 258 | 37% | +| slic/utils/logbook\_exceptions.py | 13 | 0 | 100% | +| slic/utils/logcfg.py | 52 | 2 | 96% | +| slic/utils/logign.py | 22 | 14 | 36% | +| slic/utils/marker.py | 48 | 31 | 35% | +| slic/utils/metaclasses.py | 8 | 0 | 100% | +| slic/utils/namespace.py | 5 | 1 | 80% | +| slic/utils/npy.py | 71 | 56 | 21% | +| slic/utils/opmsg.py | 122 | 122 | 0% | +| slic/utils/path.py | 32 | 24 | 25% | +| slic/utils/picklio.py | 7 | 3 | 57% | +| slic/utils/printing.py | 77 | 61 | 21% | +| slic/utils/pv.py | 30 | 20 | 33% | +| slic/utils/pvpreload.py | 50 | 19 | 62% | +| slic/utils/pyepics.py | 64 | 39 | 39% | +| slic/utils/rangebar.py | 92 | 61 | 34% | +| slic/utils/readable.py | 12 | 9 | 25% | +| slic/utils/registry.py | 33 | 19 | 42% | +| slic/utils/reprate.py | 46 | 33 | 28% | +| slic/utils/richcfg.py | 21 | 11 | 48% | +| slic/utils/run\_later.py | 64 | 64 | 0% | +| slic/utils/screenshot.py | 30 | 19 | 37% | +| slic/utils/sendmail.py | 49 | 49 | 0% | +| slic/utils/sendsms.py | 5 | 5 | 0% | +| slic/utils/shortcut.py | 38 | 20 | 47% | +| slic/utils/snapshot.py | 6 | 3 | 50% | +| slic/utils/termtitle.py | 3 | 0 | 100% | +| slic/utils/tqdm\_mod.py | 28 | 18 | 36% | +| slic/utils/trinary.py | 4 | 2 | 50% | +| slic/utils/typecast.py | 19 | 19 | 0% | +| slic/utils/utils.py | 17 | 8 | 53% | +| slic/utils/xrange.py | 33 | 30 | 9% | +| **TOTAL** | **9942** | **7225** | **27%** | diff --git a/markdown/pytest-report.json b/markdown/pytest-report.json index b6e5904f..278b6920 100644 --- a/markdown/pytest-report.json +++ b/markdown/pytest-report.json @@ -1 +1 @@ -{"created": 1755001695.4245157, "duration": 4.76562237739563, "exitcode": 1, "root": "/workspace/tligui_y/slic", "environment": {}, "summary": {"failed": 6, "total": 6, "collected": 6}, "collectors": [{"nodeid": "", "outcome": "passed", "result": [{"nodeid": "tests/test_utils_elog.py", "type": "Module"}]}, {"nodeid": "tests/test_utils_elog.py", "outcome": "passed", "result": [{"nodeid": "tests/test_utils_elog.py::test_post_local", "type": "Function", "lineno": 29}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_with_direct_password_and_real_check", "type": "Function", "lineno": 52}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_asks_password_and_opens", "type": "Function", "lineno": 65}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_with_path_home", "type": "Function", "lineno": 83}, {"nodeid": "tests/test_utils_elog.py::test_post", "type": "Function", "lineno": 114}, {"nodeid": "tests/test_utils_elog.py::test_screenshot", "type": "Function", "lineno": 130}]}], "tests": [{"nodeid": "tests/test_utils_elog.py::test_post_local", "lineno": 29, "outcome": "failed", "keywords": ["test_post_local", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.0003777281381189823, "outcome": "passed"}, "call": {"duration": 0.009992022067308426, "outcome": "failed", "crash": {"path": "/workspace/tligui_y/slic/.pixi/envs/default/lib/python3.8/site-packages/elog/logbook.py", "lineno": 315, "message": "elog.logbook_exceptions.LogbookInvalidMessageID: Invalid message ID: None returned"}, "traceback": [{"path": "tests/test_utils_elog.py", "lineno": 45, "message": ""}, {"path": ".pixi/envs/default/lib/python3.8/site-packages/elog/logbook.py", "lineno": 315, "message": "LogbookInvalidMessageID"}], "longrepr": "def test_post_local():\n logbook = elog.open(\n hostname=\"http://localhost\",\n port=8080,\n user=\"robot\",\n password=\"testpassword\",\n use_ssl=False,\n logbook=\"demo\"\n )\n attributes = {\n \"Author\": \"robot\",\n \"Subject\": \"Test simple\",\n \"Category\": \"General\",\n }\n message = \"Hello from local test\"\n> msg_id = logbook.post(message, attributes=attributes, encoding=\"HTML\")\n\ntests/test_utils_elog.py:45: \n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \n\nself = \nmessage = 'Hello from local test', msg_id = None, reply = False\nattributes = {'Author': 'robot', 'Category': 'General', 'Encoding': 'HTML', 'Subject': 'Test simple', ...}\nattachments = [], suppress_email_notification = False, encoding = 'HTML'\ntimeout = None, kwargs = {}\nnew_attachment_list = [('Text', ('', b'Hello from local test'))]\nobjects_to_close = []\nattributes_to_edit = {'Author': b'robot', 'Category': b'General', 'Encoding': b'HTML', 'Subject': b'Test simple', ...}\n\n def post(self, message, msg_id=None, reply=False, attributes=None, attachments=None,\n suppress_email_notification=False, encoding=None, timeout=None, **kwargs):\n \"\"\"\n Posts message to the logbook. If msg_id is not specified new message will be created, otherwise existing\n message will be edited, or a reply (if reply=True) to it will be created. This method returns the msg_id\n of the newly created message.\n \n :param message: string with message text\n :param msg_id: ID number of message to edit or reply. If not specified new message is created.\n :param reply: If 'True' reply to existing message is created instead of editing it\n :param attributes: Dictionary of attributes. Following attributes are used internally by the elog and will be\n ignored: Text, Date, Encoding, Reply to, In reply to, Locked by, Attachment\n :param attachments: list of:\n - file like objects which read() will return bytes (if file_like_object.name is not\n defined, default name \"attachment\" will be used.\n - paths to the files\n All items will be appended as attachment to the elog entry. In case of unknown\n attachment an exception LogbookInvalidAttachment will be raised.\n :param suppress_email_notification: If set to True or 1, E-Mail notification will be suppressed, defaults to False.\n :param encoding: Defines encoding of the message. Can be: 'plain' -> plain text, 'html'->html-text,\n 'ELCode' --> elog formatting syntax\n :param timeout: Define the timeout to be used by the post request. Its value is directly passed to the requests\n post. Use None to disable the request timeout.\n :param kwargs: Anything in the kwargs will be interpreted as attribute. e.g.: logbook.post('Test text',\n Author='Rok Vintar), \"Author\" will be sent as an attribute. If named same as one of the\n attributes defined in \"attributes\", kwargs will have priority.\n \n :return: msg_id\n \"\"\"\n \n attributes = attributes or {}\n attributes = {**attributes, **kwargs} # kwargs as attributes with higher priority\n \n attachments = attachments or []\n \n if encoding is not None:\n if encoding not in ['plain', 'HTML', 'ELCode']:\n raise LogbookMessageRejected('Invalid message encoding. Valid options: plain, HTML, ELCode.')\n attributes['Encoding'] = encoding\n \n if suppress_email_notification:\n attributes[\"suppress\"] = 1\n \n # THE ATTACHMENT STRATEGY WHEN DEALING WITH POST MODIFICATION\n #\n # 1. Does the message on the server have already attachments?\n # 1.1 - We read the message getting the existing attachment list.\n # 1.2 - Add to the attributes dictionary one line for each attachment like this:\n # attributes['attachmentN'] = timestamped_filename_name\n #\n # 2. Do we have new attachments?\n # 2.1 - Those are in the new_attachment_list. This is a list of this type:\n # [ ('attfileN', ('filename', fileobject)) ]\n # 2.2 - We need to loop over all the new attachments:\n # 2.2.1 - Does a file already on the server with the same name exist?\n # 2.2.1.1 - No: OK. Then we go ahead with the next attachment.\n # 2.2.1.2 - Yes:\n # 2.2.1.2.1 - Are the two files identical?\n # 2.2.1.2.1.1 - Yes: then we remove this current entry from the new_attachment_list and we leave the one\n # already on server.\n # 2.2.1.2.1.2 - No:\n # 2.2.1.2.1.2.1 - Then the file has been update.\n # 2.2.1.2.1.2.2 - We need to remove the file on server first (using special post)\n # 2.2.1.2.1.2.3 - We have to remove the old attachment from the attributes dictionary.\n #\n \n if attachments:\n # here we accomplish point 2.1.\n # new_attachment_list is something like [ ('attfileN', ('filename', fileobject)) ]\n new_attachment_list, objects_to_close = self._prepare_attachments(attachments)\n else:\n objects_to_close = list()\n new_attachment_list = list()\n \n attributes_to_edit = dict()\n if msg_id:\n # Message exists, we can continue\n if reply:\n # Verify that there is a message on the server, otherwise do not reply to it!\n self._check_if_message_on_server(msg_id) # raises exception in case of none existing message\n attributes['reply_to'] = str(msg_id)\n else: # Edit existing\n attributes['edit_id'] = str(msg_id)\n attributes['skiplock'] = '1'\n \n # here we accomplish point 1.1.\n # existing_attachments_list is something like:\n # [ 'https://elog.url.com/logbook/timestamped_filename' ]\n msg_to_edit, attributes_to_edit, existing_attachments_list = self.read(msg_id)\n \n for attribute, data in attributes.items():\n new_data = attributes.get(attribute)\n if new_data is not None:\n attributes_to_edit[attribute] = new_data\n \n i = 0\n existing_attachments_filename_list = list()\n for attachment in existing_attachments_list:\n # here we accomplish point 1.2. We strip the timestamped_filename from the whole URL.\n attributes_to_edit[f'attachment{i}'] = os.path.basename(attachment)\n existing_attachments_filename_list.append(os.path.basename(attachment)[14:])\n i += 1\n \n # let's accomplish 2.2. Loop over all new attachment\n duplicate_attachment_list = list()\n for new_attachment in new_attachment_list:\n # the new_attachment_list is something like:\n # [ ('attfileN', ('filename', fileobject)) ]\n new_attachment_filename = new_attachment[1][0]\n if new_attachment_filename in existing_attachments_filename_list:\n # a file with the same name existing already on the server.\n # we need to check if the two files are the same.\n # read the content of the new file\n new_attachment_content = new_attachment[1][1].read()\n # don't forget to reset the fileobj to the beginning of the file\n new_attachment[1][1].seek(0)\n # get the existing attachment content\n attachment_index = existing_attachments_filename_list.index(new_attachment_filename)\n existing_attachment_content = self.download_attachment(\n url=existing_attachments_list[attachment_index],\n timeout=timeout\n )\n # check if the two contents are the same\n if new_attachment_content == existing_attachment_content:\n # yes. then we don't upload a second copy. we remove the current entry from the list\n duplicate_attachment_list.append(new_attachment)\n else:\n # no. they are not the same file. we will replace the existing file with the new one\n # first: we need to remove the attachment from the server using the dedicated method\n self.delete_attachment(msg_id, attributes=attributes_to_edit,\n attachment_id=attachment_index,\n timeout=timeout, text=msg_to_edit)\n # now we can remove this attachment from the auxiliary lists.\n existing_attachments_filename_list.pop(attachment_index)\n existing_attachments_list.pop(attachment_index)\n # now we need to rebuild the attributes dictionary for the part concerning the attachments.\n # we remove all of them first\n keys_to_be_removed = list()\n for key in attributes_to_edit.keys():\n if key.startswith('attachment'):\n keys_to_be_removed.append(key)\n if key.startswith('delatt'):\n keys_to_be_removed.append(key)\n for key in keys_to_be_removed:\n del attributes_to_edit[key]\n \n # now we rebuild it\n for i, attachment in enumerate(existing_attachments_list):\n attributes_to_edit[f'attachment{i}'] = os.path.basename(attachment)\n \n # remove all duplicate attachments from the new_attachment_list\n for attach in duplicate_attachment_list:\n new_attachment_list.remove(attach)\n \n else:\n # As we create a new message, specify creation time if not already specified in attributes\n if 'When' not in attributes:\n attributes['When'] = int(datetime.now().timestamp())\n \n if not attributes_to_edit:\n attributes_to_edit = attributes\n \n # Remove any attributes that should not be sent\n _remove_reserved_attributes(attributes_to_edit)\n \n # Make requests module think that Text is a \"file\". This is the only way to force requests to send data as\n # multipart/form-data even if there are no attachments. Elog understands only multipart/form-data\n new_attachment_list.append(('Text', ('', message.encode('iso-8859-1'))))\n \n # Base attributes are common to all messages\n self._add_base_msg_attributes(attributes_to_edit)\n \n # Keys in attributes cannot have certain characters like whitespaces or dashes for the http request\n attributes_to_edit = _replace_special_characters_in_attribute_keys(attributes_to_edit)\n \n # All string values in the attributes must be encoded in latin1\n attributes_to_edit = _encode_values(attributes_to_edit)\n \n try:\n response = requests.post(self._url, data=attributes_to_edit, files=new_attachment_list,\n allow_redirects=False, verify=False, timeout=timeout)\n \n # Validate response. Any problems will raise an Exception.\n resp_message, resp_headers, resp_msg_id = _validate_response(response)\n \n # Close file like objects that were opened by the elog (if path\n for file_like_object in objects_to_close:\n if hasattr(file_like_object, 'close'):\n file_like_object.close()\n \n except requests.Timeout as e:\n # Catch here a timeout o the post request.\n # Raise the logbook excetion and let the user handle it\n raise LogbookServerTimeout('{0} method cannot be completed because of a network timeout:\\n' +\n '{1}'.format(sys._getframe().f_code.co_name, e))\n \n except requests.RequestException as e:\n # Check if message on server.\n self._check_if_message_on_server(msg_id) # raises exceptions if no message or no response from server\n \n # If here: message is on server but cannot be downloaded (should never happen)\n raise LogbookServerProblem('Cannot access logbook server to post a message, ' + 'because of:\\n' +\n '{0}'.format(e))\n \n # Any error before here should raise an exception, but check again for nay case.\n if not resp_msg_id or resp_msg_id < 1:\n> raise LogbookInvalidMessageID('Invalid message ID: ' + str(resp_msg_id) + ' returned')\nE elog.logbook_exceptions.LogbookInvalidMessageID: Invalid message ID: None returned\n\n.pixi/envs/default/lib/python3.8/site-packages/elog/logbook.py:315: LogbookInvalidMessageID"}, "teardown": {"duration": 0.0003286711871623993, "outcome": "passed"}}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_with_direct_password_and_real_check", "lineno": 52, "outcome": "failed", "keywords": ["test_get_default_elog_instance_with_direct_password_and_real_check", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.0001472141593694687, "outcome": "passed"}, "call": {"duration": 0.0002653147093951702, "outcome": "failed", "crash": {"path": "/workspace/tligui_y/slic/slic/utils/elog.py", "lineno": 39, "message": "NameError: name 'elog' is not defined"}, "traceback": [{"path": "tests/test_utils_elog.py", "lineno": 58, "message": ""}, {"path": "slic/utils/elog.py", "lineno": 39, "message": "NameError"}], "longrepr": "def test_get_default_elog_instance_with_direct_password_and_real_check():\n url = \"http://localhost:8080/demo\"\n user = \"robot\"\n password = \"testpassword\"\n \n> elog_instance, returned_user = get_default_elog_instance(url, user=user, password=password)\n\ntests/test_utils_elog.py:58: \n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \n\nurl = 'http://localhost:8080/demo'\nkwargs = {'password': 'testpassword', 'user': 'robot'}, user = 'robot'\n\n def get_default_elog_instance(url, **kwargs):\n kwargs.setdefault(\"user\", getuser())\n user = kwargs[\"user\"]\n \n if \"password\" not in kwargs:\n try:\n home = Path.home()\n fn = home / \".elog_psi\"\n with fn.open() as f:\n pw = f.read().strip()\n except Exception:\n print(f\"Enter elog password for user: {user}\")\n pw = getpass()\n kwargs[\"password\"] = pw\n \n> return elog.open(url, **kwargs), user\nE NameError: name 'elog' is not defined\n\nslic/utils/elog.py:39: NameError"}, "teardown": {"duration": 0.000166354700922966, "outcome": "passed"}}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_asks_password_and_opens", "lineno": 65, "outcome": "failed", "keywords": ["test_get_default_elog_instance_asks_password_and_opens", "__wrapped__", "patchings", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.00012920983135700226, "outcome": "passed"}, "call": {"duration": 0.001176837831735611, "outcome": "failed", "crash": {"path": "/workspace/tligui_y/slic/slic/utils/elog.py", "lineno": 39, "message": "NameError: name 'elog' is not defined"}, "traceback": [{"path": "tests/test_utils_elog.py", "lineno": 75, "message": ""}, {"path": "slic/utils/elog.py", "lineno": 39, "message": "NameError"}], "longrepr": "mock_home = \nmock_getpass = \n\n @patch(\"slic.utils.elog.getpass\")\n @patch(\"slic.utils.elog.Path.home\")\n def test_get_default_elog_instance_asks_password_and_opens(mock_home, mock_getpass):\n mock_home.return_value = Path(\"/does/not/exist\") # Fausse home \u2192 lecture \u00e9choue\n mock_getpass.return_value = \"testpassword\"\n \n url = \"http://localhost:8080/demo\"\n user = \"robot\"\n \n> elog_instance, returned_user = get_default_elog_instance(url, user=user)\n\ntests/test_utils_elog.py:75: \n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \n\nurl = 'http://localhost:8080/demo'\nkwargs = {'password': 'testpassword', 'user': 'robot'}, user = 'robot'\nhome = PosixPath('/does/not/exist'), fn = PosixPath('/does/not/exist/.elog_psi')\npw = 'testpassword'\n\n def get_default_elog_instance(url, **kwargs):\n kwargs.setdefault(\"user\", getuser())\n user = kwargs[\"user\"]\n \n if \"password\" not in kwargs:\n try:\n home = Path.home()\n fn = home / \".elog_psi\"\n with fn.open() as f:\n pw = f.read().strip()\n except Exception:\n print(f\"Enter elog password for user: {user}\")\n pw = getpass()\n kwargs[\"password\"] = pw\n \n> return elog.open(url, **kwargs), user\nE NameError: name 'elog' is not defined\n\nslic/utils/elog.py:39: NameError"}, "teardown": {"duration": 0.0001467810943722725, "outcome": "passed"}}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_with_path_home", "lineno": 83, "outcome": "failed", "keywords": ["test_get_default_elog_with_path_home", "__wrapped__", "patchings", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.00011813687160611153, "outcome": "passed"}, "call": {"duration": 0.0015169298276305199, "outcome": "failed", "crash": {"path": "/workspace/tligui_y/slic/slic/utils/elog.py", "lineno": 39, "message": "NameError: name 'elog' is not defined"}, "traceback": [{"path": "tests/test_utils_elog.py", "lineno": 102, "message": ""}, {"path": "slic/utils/elog.py", "lineno": 39, "message": "NameError"}], "longrepr": "mock_home = \nmock_getuser = \nmock_getpass = \n\n @patch(\"slic.utils.elog.getpass\")\n @patch(\"slic.utils.elog.getuser\")\n @patch(\"slic.utils.elog.Path.home\")\n def test_get_default_elog_with_path_home(mock_home, mock_getuser, mock_getpass):\n fake_user = \"robot\"\n fake_pw = \"testpassword\"\n mock_getuser.return_value = fake_user\n mock_getpass.return_value = fake_pw # fallback safety\n \n tmp_home = Path(\"/tmp/fake_home_for_robot\")\n tmp_home.mkdir(parents=True, exist_ok=True)\n pw_file = tmp_home / \".elog_psi\"\n pw_file.write_text(fake_pw)\n mock_home.return_value = tmp_home\n \n url = \"http://localhost:8080/demo\"\n \n try:\n> elog_instance, returned_user = get_default_elog_instance(url)\n\ntests/test_utils_elog.py:102: \n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \n\nurl = 'http://localhost:8080/demo'\nkwargs = {'password': 'testpassword', 'user': 'robot'}, user = 'robot'\nhome = PosixPath('/tmp/fake_home_for_robot')\nfn = PosixPath('/tmp/fake_home_for_robot/.elog_psi')\nf = <_io.TextIOWrapper name='/tmp/fake_home_for_robot/.elog_psi' mode='r' encoding='UTF-8'>\npw = 'testpassword'\n\n def get_default_elog_instance(url, **kwargs):\n kwargs.setdefault(\"user\", getuser())\n user = kwargs[\"user\"]\n \n if \"password\" not in kwargs:\n try:\n home = Path.home()\n fn = home / \".elog_psi\"\n with fn.open() as f:\n pw = f.read().strip()\n except Exception:\n print(f\"Enter elog password for user: {user}\")\n pw = getpass()\n kwargs[\"password\"] = pw\n \n> return elog.open(url, **kwargs), user\nE NameError: name 'elog' is not defined\n\nslic/utils/elog.py:39: NameError"}, "teardown": {"duration": 0.0001523476094007492, "outcome": "passed"}}, {"nodeid": "tests/test_utils_elog.py::test_post", "lineno": 114, "outcome": "failed", "keywords": ["test_post", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.0001242789439857006, "outcome": "passed"}, "call": {"duration": 0.00016640778630971909, "outcome": "failed", "crash": {"path": "/workspace/tligui_y/slic/slic/utils/elog.py", "lineno": 39, "message": "NameError: name 'elog' is not defined"}, "traceback": [{"path": "tests/test_utils_elog.py", "lineno": 116, "message": ""}, {"path": "tests/test_utils_elog.py", "lineno": 51, "message": "in get_test_elog"}, {"path": "slic/utils/elog.py", "lineno": 11, "message": "in __init__"}, {"path": "slic/utils/elog.py", "lineno": 39, "message": "NameError"}], "longrepr": "def test_post():\n> elog = get_test_elog()\n\ntests/test_utils_elog.py:116: \n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \ntests/test_utils_elog.py:51: in get_test_elog\n return Elog(\"http://localhost:8080/demo\", user=\"robot\", password=\"testpassword\")\nslic/utils/elog.py:11: in __init__\n self._log, self.user = get_default_elog_instance(url, **kwargs)\n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \n\nurl = 'http://localhost:8080/demo'\nkwargs = {'password': 'testpassword', 'user': 'robot'}, user = 'robot'\n\n def get_default_elog_instance(url, **kwargs):\n kwargs.setdefault(\"user\", getuser())\n user = kwargs[\"user\"]\n \n if \"password\" not in kwargs:\n try:\n home = Path.home()\n fn = home / \".elog_psi\"\n with fn.open() as f:\n pw = f.read().strip()\n except Exception:\n print(f\"Enter elog password for user: {user}\")\n pw = getpass()\n kwargs[\"password\"] = pw\n \n> return elog.open(url, **kwargs), user\nE NameError: name 'elog' is not defined\n\nslic/utils/elog.py:39: NameError"}, "teardown": {"duration": 0.00014013797044754028, "outcome": "passed"}}, {"nodeid": "tests/test_utils_elog.py::test_screenshot", "lineno": 130, "outcome": "failed", "keywords": ["test_screenshot", "__wrapped__", "patchings", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.00011991709470748901, "outcome": "passed"}, "call": {"duration": 0.0015173987485468388, "outcome": "failed", "crash": {"path": "/workspace/tligui_y/slic/slic/utils/elog.py", "lineno": 39, "message": "NameError: name 'elog' is not defined"}, "traceback": [{"path": "tests/test_utils_elog.py", "lineno": 140, "message": ""}, {"path": "tests/test_utils_elog.py", "lineno": 51, "message": "in get_test_elog"}, {"path": "slic/utils/elog.py", "lineno": 11, "message": "in __init__"}, {"path": "slic/utils/elog.py", "lineno": 39, "message": "NameError"}], "longrepr": "mock_screenshot_class = \n\n @patch(\"slic.utils.elog.Screenshot\")\n def test_screenshot(mock_screenshot_class):\n with tempfile.NamedTemporaryFile(delete=False, suffix=\".png\") as tmp:\n fake_path = tmp.name\n tmp.write(b\"fake image data\")\n \n mock_instance = mock_screenshot_class.return_value\n mock_instance.shoot.return_value = [fake_path]\n \n> elog = get_test_elog()\n\ntests/test_utils_elog.py:140: \n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \ntests/test_utils_elog.py:51: in get_test_elog\n return Elog(\"http://localhost:8080/demo\", user=\"robot\", password=\"testpassword\")\nslic/utils/elog.py:11: in __init__\n self._log, self.user = get_default_elog_instance(url, **kwargs)\n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \n\nurl = 'http://localhost:8080/demo'\nkwargs = {'password': 'testpassword', 'user': 'robot'}, user = 'robot'\n\n def get_default_elog_instance(url, **kwargs):\n kwargs.setdefault(\"user\", getuser())\n user = kwargs[\"user\"]\n \n if \"password\" not in kwargs:\n try:\n home = Path.home()\n fn = home / \".elog_psi\"\n with fn.open() as f:\n pw = f.read().strip()\n except Exception:\n print(f\"Enter elog password for user: {user}\")\n pw = getpass()\n kwargs[\"password\"] = pw\n \n> return elog.open(url, **kwargs), user\nE NameError: name 'elog' is not defined\n\nslic/utils/elog.py:39: NameError"}, "teardown": {"duration": 0.00016004778444766998, "outcome": "passed"}}], "warnings": [{"message": "invalid escape sequence \\-", "category": "DeprecationWarning", "when": "collect", "filename": "/workspace/tligui_y/slic/.pixi/envs/default/lib/python3.8/site-packages/bsread/h5.py", "lineno": 207}, {"message": "The module numpy.dual is deprecated. Instead of using dual, use the functions directly from numpy or scipy.", "category": "DeprecationWarning", "when": "collect", "filename": "/workspace/tligui_y/slic/.pixi/envs/default/lib/python3.8/site-packages/scipy/fft/__init__.py", "lineno": 97}]} \ No newline at end of file +{"created": 1755001838.3606393, "duration": 4.833057641983032, "exitcode": 1, "root": "/workspace/tligui_y/slic", "environment": {}, "summary": {"failed": 3, "passed": 3, "total": 6, "collected": 6}, "collectors": [{"nodeid": "", "outcome": "passed", "result": [{"nodeid": "tests/test_utils_elog.py", "type": "Module"}]}, {"nodeid": "tests/test_utils_elog.py", "outcome": "passed", "result": [{"nodeid": "tests/test_utils_elog.py::test_post_local", "type": "Function", "lineno": 29}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_with_direct_password_and_real_check", "type": "Function", "lineno": 52}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_asks_password_and_opens", "type": "Function", "lineno": 65}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_with_path_home", "type": "Function", "lineno": 83}, {"nodeid": "tests/test_utils_elog.py::test_post", "type": "Function", "lineno": 114}, {"nodeid": "tests/test_utils_elog.py::test_screenshot", "type": "Function", "lineno": 130}]}], "tests": [{"nodeid": "tests/test_utils_elog.py::test_post_local", "lineno": 29, "outcome": "failed", "keywords": ["test_post_local", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.0005297278985381126, "outcome": "passed"}, "call": {"duration": 0.008404782973229885, "outcome": "failed", "crash": {"path": "/workspace/tligui_y/slic/.pixi/envs/default/lib/python3.8/site-packages/elog/logbook.py", "lineno": 315, "message": "elog.logbook_exceptions.LogbookInvalidMessageID: Invalid message ID: None returned"}, "traceback": [{"path": "tests/test_utils_elog.py", "lineno": 45, "message": ""}, {"path": ".pixi/envs/default/lib/python3.8/site-packages/elog/logbook.py", "lineno": 315, "message": "LogbookInvalidMessageID"}], "longrepr": "def test_post_local():\n logbook = elog.open(\n hostname=\"http://localhost\",\n port=8080,\n user=\"robot\",\n password=\"testpassword\",\n use_ssl=False,\n logbook=\"demo\"\n )\n attributes = {\n \"Author\": \"robot\",\n \"Subject\": \"Test simple\",\n \"Category\": \"General\",\n }\n message = \"Hello from local test\"\n> msg_id = logbook.post(message, attributes=attributes, encoding=\"HTML\")\n\ntests/test_utils_elog.py:45: \n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \n\nself = \nmessage = 'Hello from local test', msg_id = None, reply = False\nattributes = {'Author': 'robot', 'Category': 'General', 'Encoding': 'HTML', 'Subject': 'Test simple', ...}\nattachments = [], suppress_email_notification = False, encoding = 'HTML'\ntimeout = None, kwargs = {}\nnew_attachment_list = [('Text', ('', b'Hello from local test'))]\nobjects_to_close = []\nattributes_to_edit = {'Author': b'robot', 'Category': b'General', 'Encoding': b'HTML', 'Subject': b'Test simple', ...}\n\n def post(self, message, msg_id=None, reply=False, attributes=None, attachments=None,\n suppress_email_notification=False, encoding=None, timeout=None, **kwargs):\n \"\"\"\n Posts message to the logbook. If msg_id is not specified new message will be created, otherwise existing\n message will be edited, or a reply (if reply=True) to it will be created. This method returns the msg_id\n of the newly created message.\n \n :param message: string with message text\n :param msg_id: ID number of message to edit or reply. If not specified new message is created.\n :param reply: If 'True' reply to existing message is created instead of editing it\n :param attributes: Dictionary of attributes. Following attributes are used internally by the elog and will be\n ignored: Text, Date, Encoding, Reply to, In reply to, Locked by, Attachment\n :param attachments: list of:\n - file like objects which read() will return bytes (if file_like_object.name is not\n defined, default name \"attachment\" will be used.\n - paths to the files\n All items will be appended as attachment to the elog entry. In case of unknown\n attachment an exception LogbookInvalidAttachment will be raised.\n :param suppress_email_notification: If set to True or 1, E-Mail notification will be suppressed, defaults to False.\n :param encoding: Defines encoding of the message. Can be: 'plain' -> plain text, 'html'->html-text,\n 'ELCode' --> elog formatting syntax\n :param timeout: Define the timeout to be used by the post request. Its value is directly passed to the requests\n post. Use None to disable the request timeout.\n :param kwargs: Anything in the kwargs will be interpreted as attribute. e.g.: logbook.post('Test text',\n Author='Rok Vintar), \"Author\" will be sent as an attribute. If named same as one of the\n attributes defined in \"attributes\", kwargs will have priority.\n \n :return: msg_id\n \"\"\"\n \n attributes = attributes or {}\n attributes = {**attributes, **kwargs} # kwargs as attributes with higher priority\n \n attachments = attachments or []\n \n if encoding is not None:\n if encoding not in ['plain', 'HTML', 'ELCode']:\n raise LogbookMessageRejected('Invalid message encoding. Valid options: plain, HTML, ELCode.')\n attributes['Encoding'] = encoding\n \n if suppress_email_notification:\n attributes[\"suppress\"] = 1\n \n # THE ATTACHMENT STRATEGY WHEN DEALING WITH POST MODIFICATION\n #\n # 1. Does the message on the server have already attachments?\n # 1.1 - We read the message getting the existing attachment list.\n # 1.2 - Add to the attributes dictionary one line for each attachment like this:\n # attributes['attachmentN'] = timestamped_filename_name\n #\n # 2. Do we have new attachments?\n # 2.1 - Those are in the new_attachment_list. This is a list of this type:\n # [ ('attfileN', ('filename', fileobject)) ]\n # 2.2 - We need to loop over all the new attachments:\n # 2.2.1 - Does a file already on the server with the same name exist?\n # 2.2.1.1 - No: OK. Then we go ahead with the next attachment.\n # 2.2.1.2 - Yes:\n # 2.2.1.2.1 - Are the two files identical?\n # 2.2.1.2.1.1 - Yes: then we remove this current entry from the new_attachment_list and we leave the one\n # already on server.\n # 2.2.1.2.1.2 - No:\n # 2.2.1.2.1.2.1 - Then the file has been update.\n # 2.2.1.2.1.2.2 - We need to remove the file on server first (using special post)\n # 2.2.1.2.1.2.3 - We have to remove the old attachment from the attributes dictionary.\n #\n \n if attachments:\n # here we accomplish point 2.1.\n # new_attachment_list is something like [ ('attfileN', ('filename', fileobject)) ]\n new_attachment_list, objects_to_close = self._prepare_attachments(attachments)\n else:\n objects_to_close = list()\n new_attachment_list = list()\n \n attributes_to_edit = dict()\n if msg_id:\n # Message exists, we can continue\n if reply:\n # Verify that there is a message on the server, otherwise do not reply to it!\n self._check_if_message_on_server(msg_id) # raises exception in case of none existing message\n attributes['reply_to'] = str(msg_id)\n else: # Edit existing\n attributes['edit_id'] = str(msg_id)\n attributes['skiplock'] = '1'\n \n # here we accomplish point 1.1.\n # existing_attachments_list is something like:\n # [ 'https://elog.url.com/logbook/timestamped_filename' ]\n msg_to_edit, attributes_to_edit, existing_attachments_list = self.read(msg_id)\n \n for attribute, data in attributes.items():\n new_data = attributes.get(attribute)\n if new_data is not None:\n attributes_to_edit[attribute] = new_data\n \n i = 0\n existing_attachments_filename_list = list()\n for attachment in existing_attachments_list:\n # here we accomplish point 1.2. We strip the timestamped_filename from the whole URL.\n attributes_to_edit[f'attachment{i}'] = os.path.basename(attachment)\n existing_attachments_filename_list.append(os.path.basename(attachment)[14:])\n i += 1\n \n # let's accomplish 2.2. Loop over all new attachment\n duplicate_attachment_list = list()\n for new_attachment in new_attachment_list:\n # the new_attachment_list is something like:\n # [ ('attfileN', ('filename', fileobject)) ]\n new_attachment_filename = new_attachment[1][0]\n if new_attachment_filename in existing_attachments_filename_list:\n # a file with the same name existing already on the server.\n # we need to check if the two files are the same.\n # read the content of the new file\n new_attachment_content = new_attachment[1][1].read()\n # don't forget to reset the fileobj to the beginning of the file\n new_attachment[1][1].seek(0)\n # get the existing attachment content\n attachment_index = existing_attachments_filename_list.index(new_attachment_filename)\n existing_attachment_content = self.download_attachment(\n url=existing_attachments_list[attachment_index],\n timeout=timeout\n )\n # check if the two contents are the same\n if new_attachment_content == existing_attachment_content:\n # yes. then we don't upload a second copy. we remove the current entry from the list\n duplicate_attachment_list.append(new_attachment)\n else:\n # no. they are not the same file. we will replace the existing file with the new one\n # first: we need to remove the attachment from the server using the dedicated method\n self.delete_attachment(msg_id, attributes=attributes_to_edit,\n attachment_id=attachment_index,\n timeout=timeout, text=msg_to_edit)\n # now we can remove this attachment from the auxiliary lists.\n existing_attachments_filename_list.pop(attachment_index)\n existing_attachments_list.pop(attachment_index)\n # now we need to rebuild the attributes dictionary for the part concerning the attachments.\n # we remove all of them first\n keys_to_be_removed = list()\n for key in attributes_to_edit.keys():\n if key.startswith('attachment'):\n keys_to_be_removed.append(key)\n if key.startswith('delatt'):\n keys_to_be_removed.append(key)\n for key in keys_to_be_removed:\n del attributes_to_edit[key]\n \n # now we rebuild it\n for i, attachment in enumerate(existing_attachments_list):\n attributes_to_edit[f'attachment{i}'] = os.path.basename(attachment)\n \n # remove all duplicate attachments from the new_attachment_list\n for attach in duplicate_attachment_list:\n new_attachment_list.remove(attach)\n \n else:\n # As we create a new message, specify creation time if not already specified in attributes\n if 'When' not in attributes:\n attributes['When'] = int(datetime.now().timestamp())\n \n if not attributes_to_edit:\n attributes_to_edit = attributes\n \n # Remove any attributes that should not be sent\n _remove_reserved_attributes(attributes_to_edit)\n \n # Make requests module think that Text is a \"file\". This is the only way to force requests to send data as\n # multipart/form-data even if there are no attachments. Elog understands only multipart/form-data\n new_attachment_list.append(('Text', ('', message.encode('iso-8859-1'))))\n \n # Base attributes are common to all messages\n self._add_base_msg_attributes(attributes_to_edit)\n \n # Keys in attributes cannot have certain characters like whitespaces or dashes for the http request\n attributes_to_edit = _replace_special_characters_in_attribute_keys(attributes_to_edit)\n \n # All string values in the attributes must be encoded in latin1\n attributes_to_edit = _encode_values(attributes_to_edit)\n \n try:\n response = requests.post(self._url, data=attributes_to_edit, files=new_attachment_list,\n allow_redirects=False, verify=False, timeout=timeout)\n \n # Validate response. Any problems will raise an Exception.\n resp_message, resp_headers, resp_msg_id = _validate_response(response)\n \n # Close file like objects that were opened by the elog (if path\n for file_like_object in objects_to_close:\n if hasattr(file_like_object, 'close'):\n file_like_object.close()\n \n except requests.Timeout as e:\n # Catch here a timeout o the post request.\n # Raise the logbook excetion and let the user handle it\n raise LogbookServerTimeout('{0} method cannot be completed because of a network timeout:\\n' +\n '{1}'.format(sys._getframe().f_code.co_name, e))\n \n except requests.RequestException as e:\n # Check if message on server.\n self._check_if_message_on_server(msg_id) # raises exceptions if no message or no response from server\n \n # If here: message is on server but cannot be downloaded (should never happen)\n raise LogbookServerProblem('Cannot access logbook server to post a message, ' + 'because of:\\n' +\n '{0}'.format(e))\n \n # Any error before here should raise an exception, but check again for nay case.\n if not resp_msg_id or resp_msg_id < 1:\n> raise LogbookInvalidMessageID('Invalid message ID: ' + str(resp_msg_id) + ' returned')\nE elog.logbook_exceptions.LogbookInvalidMessageID: Invalid message ID: None returned\n\n.pixi/envs/default/lib/python3.8/site-packages/elog/logbook.py:315: LogbookInvalidMessageID"}, "teardown": {"duration": 0.00036900071427226067, "outcome": "passed"}}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_with_direct_password_and_real_check", "lineno": 52, "outcome": "passed", "keywords": ["test_get_default_elog_instance_with_direct_password_and_real_check", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.00014514895156025887, "outcome": "passed"}, "call": {"duration": 0.009522773325443268, "outcome": "passed"}, "teardown": {"duration": 0.00015535438433289528, "outcome": "passed"}}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_asks_password_and_opens", "lineno": 65, "outcome": "passed", "keywords": ["test_get_default_elog_instance_asks_password_and_opens", "__wrapped__", "patchings", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.0001318720169365406, "outcome": "passed"}, "call": {"duration": 0.009336789138615131, "outcome": "passed"}, "teardown": {"duration": 0.0001459629274904728, "outcome": "passed"}}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_with_path_home", "lineno": 83, "outcome": "passed", "keywords": ["test_get_default_elog_with_path_home", "__wrapped__", "patchings", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.000128098763525486, "outcome": "passed"}, "call": {"duration": 0.009406064637005329, "outcome": "passed"}, "teardown": {"duration": 0.00014375103637576103, "outcome": "passed"}}, {"nodeid": "tests/test_utils_elog.py::test_post", "lineno": 114, "outcome": "failed", "keywords": ["test_post", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.00012351106852293015, "outcome": "passed"}, "call": {"duration": 0.005688583943992853, "outcome": "failed", "crash": {"path": "/workspace/tligui_y/slic/slic/utils/logbook.py", "lineno": 276, "message": "slic.utils.logbook_exceptions.LogbookInvalidMessageID: Invalid message ID: None"}, "traceback": [{"path": "tests/test_utils_elog.py", "lineno": 122, "message": ""}, {"path": "slic/utils/elog.py", "lineno": 17, "message": "in post"}, {"path": "slic/utils/logbook.py", "lineno": 276, "message": "LogbookInvalidMessageID"}], "longrepr": "def test_post():\n elog = get_test_elog()\n \n title = \"AUTHOR_OVERRIDE_TEST\"\n text = \"This is a message\"\n author = \"robot\"\n \n> elog.post(text, attributes = {\"Author\": author})\n\ntests/test_utils_elog.py:122: \n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \nslic/utils/elog.py:17: in post\n return self._log.post(*args, **kwargs)\n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \n\nself = \nmessage = 'This is a message', msg_id = None, reply = False\nattributes = {'Author': 'robot', 'When': 1755001837, 'cmd': 'Submit', 'exp': 'demo', ...}\nattachments = [], suppress_email_notification = False, encoding = None\ntimeout = None, kwargs = {'Author': 'robot'}\nnew_attachment_list = [('Text', ('', b'This is a message'))]\nobjects_to_close = []\nattributes_to_edit = {'Author': b'robot', 'When': 1755001837, 'cmd': b'Submit', 'exp': b'demo', ...}\n\n def post(self, message, msg_id=None, reply=False, attributes=None, attachments=None,\n suppress_email_notification=False, encoding=None, timeout=None, **kwargs):\n \"\"\"\n Posts message to the logbook. If msg_id is not specified new message will be created, otherwise existing\n message will be edited, or a reply (if reply=True) to it will be created. This method returns the msg_id\n of the newly created message.\n \"\"\"\n \n print(\"STARTING POST\")\n # Ajout des impressions pour d\u00e9boguer\n print(f\"Message to post: {message}\")\n print(f\"msg_id: {msg_id}\")\n print(f\"Attributes: {attributes}\")\n print(f\"Attachments: {attachments}\")\n print(f\"Encoding: {encoding}\")\n print(f\"Timeout: {timeout}\")\n print(f\"Additional kwargs: {kwargs}\")\n \n attributes = attributes or {}\n attributes = {**attributes, **kwargs} # kwargs as attributes with higher priority\n print(f\"Updated attributes: {attributes}\")\n \n attachments = attachments or []\n print(f\"Attachments list: {attachments}\")\n \n if encoding is not None:\n if encoding not in ['plain', 'HTML', 'ELCode']:\n raise LogbookMessageRejected('Invalid message encoding. Valid options: plain, HTML, ELCode.')\n attributes['Encoding'] = encoding\n \n if suppress_email_notification:\n attributes[\"suppress\"] = 1\n \n # Prepare attachments\n if attachments:\n new_attachment_list, objects_to_close = self._prepare_attachments(attachments)\n print(f\"New attachments prepared: {new_attachment_list}\")\n else:\n objects_to_close = []\n new_attachment_list = []\n \n attributes_to_edit = dict()\n \n if msg_id:\n print(f\"Editing message with msg_id: {msg_id}\")\n if reply:\n print(f\"Replying to message with msg_id: {msg_id}\")\n self._check_if_message_on_server(msg_id)\n attributes['reply_to'] = str(msg_id)\n else:\n print(\"Editing existing message.\")\n attributes['edit_id'] = str(msg_id)\n attributes['skiplock'] = '1'\n msg_to_edit, attributes_to_edit, existing_attachments_list = self.read(msg_id)\n \n # Merge new attributes\n for attribute, data in attributes.items():\n if data is not None:\n attributes_to_edit[attribute] = data\n \n print(f\"Attributes after merging: {attributes_to_edit}\")\n \n # Process existing attachments\n i = 0\n existing_attachments_filename_list = []\n for attachment in existing_attachments_list:\n attributes_to_edit[f'attachment{i}'] = os.path.basename(attachment)\n existing_attachments_filename_list.append(os.path.basename(attachment)[14:])\n i += 1\n \n print(f\"Existing attachments: {existing_attachments_filename_list}\")\n \n duplicate_attachment_list = []\n for new_attachment in new_attachment_list:\n new_attachment_filename = new_attachment[1][0]\n print(f\"Checking new attachment: {new_attachment_filename}\")\n if new_attachment_filename in existing_attachments_filename_list:\n # Same attachment exists on the server, compare content\n new_attachment_content = new_attachment[1][1].read()\n new_attachment[1][1].seek(0)\n attachment_index = existing_attachments_filename_list.index(new_attachment_filename)\n existing_attachment_content = self.download_attachment(\n url=existing_attachments_list[attachment_index],\n timeout=timeout\n )\n if new_attachment_content == existing_attachment_content:\n print(f\"Duplicate attachment detected: {new_attachment_filename}\")\n duplicate_attachment_list.append(new_attachment)\n else:\n print(f\"Attachment content has changed: {new_attachment_filename}\")\n self.delete_attachment(msg_id, attributes=attributes_to_edit,\n attachment_id=attachment_index,\n timeout=timeout, text=msg_to_edit)\n existing_attachments_filename_list.pop(attachment_index)\n existing_attachments_list.pop(attachment_index)\n \n print(f\"Duplicate attachments to remove: {duplicate_attachment_list}\")\n \n # Remove duplicates\n for attach in duplicate_attachment_list:\n new_attachment_list.remove(attach)\n \n print(f\"Final new attachments list: {new_attachment_list}\")\n else:\n # Creating a new message, add timestamp if not present\n if 'When' not in attributes:\n attributes['When'] = int(datetime.now().timestamp())\n \n # Final check on attributes\n if not attributes_to_edit:\n attributes_to_edit = attributes\n \n print(f\"Final attributes to send to the server: {attributes_to_edit}\")\n \n # Remove reserved attributes\n _remove_reserved_attributes(attributes_to_edit)\n \n new_attachment_list.append(('Text', ('', message.encode('iso-8859-1'))))\n print(f\"Final attachment list including message text: {new_attachment_list}\")\n \n # Add base message attributes\n self._add_base_msg_attributes(attributes_to_edit)\n print(f\"Attributes with base message added: {attributes_to_edit}\")\n \n # Sanitize attribute keys\n attributes_to_edit = _replace_special_characters_in_attribute_keys(attributes_to_edit)\n print(f\"Attributes after sanitizing keys: {attributes_to_edit}\")\n \n # Encode all string values in latin1\n attributes_to_edit = _encode_values(attributes_to_edit)\n print(f\"Attributes after encoding: {attributes_to_edit}\")\n \n try:\n print(\"Sending POST request to the server...\")\n response = requests.post(self._url, data=attributes_to_edit, files=new_attachment_list,\n allow_redirects=False, verify=False, timeout=timeout)\n print(\"Response received:\", response)\n \n resp_message, resp_headers, resp_msg_id = _validate_response(response)\n print(f\"Response message: {resp_message}\")\n print(f\"Response headers: {resp_headers}\")\n print(f\"Response msg_id: {resp_msg_id}\")\n \n # Close file-like objects if needed\n for file_like_object in objects_to_close:\n if hasattr(file_like_object, 'close'):\n file_like_object.close()\n \n except requests.Timeout as e:\n print(\"Request timed out:\", e)\n raise LogbookServerTimeout(f\"Timeout while posting: {e}\")\n \n except requests.RequestException as e:\n print(\"Request exception:\", e)\n self._check_if_message_on_server(msg_id) # Raises exceptions if no message or no response from server\n raise LogbookServerProblem(f\"Cannot access logbook server: {e}\")\n \n if not resp_msg_id or resp_msg_id < 1:\n> raise LogbookInvalidMessageID(f\"Invalid message ID: {resp_msg_id}\")\nE slic.utils.logbook_exceptions.LogbookInvalidMessageID: Invalid message ID: None\n\nslic/utils/logbook.py:276: LogbookInvalidMessageID"}, "teardown": {"duration": 0.00018636509776115417, "outcome": "passed"}}, {"nodeid": "tests/test_utils_elog.py::test_screenshot", "lineno": 130, "outcome": "failed", "keywords": ["test_screenshot", "__wrapped__", "patchings", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.00013091834262013435, "outcome": "passed"}, "call": {"duration": 0.007740779779851437, "outcome": "failed", "crash": {"path": "/workspace/tligui_y/slic/slic/utils/logbook.py", "lineno": 276, "message": "slic.utils.logbook_exceptions.LogbookInvalidMessageID: Invalid message ID: None"}, "traceback": [{"path": "tests/test_utils_elog.py", "lineno": 143, "message": ""}, {"path": "slic/utils/elog.py", "lineno": 22, "message": "in screenshot"}, {"path": "slic/utils/elog.py", "lineno": 17, "message": "in post"}, {"path": "slic/utils/logbook.py", "lineno": 276, "message": "LogbookInvalidMessageID"}], "longrepr": "mock_screenshot_class = \n\n @patch(\"slic.utils.elog.Screenshot\")\n def test_screenshot(mock_screenshot_class):\n with tempfile.NamedTemporaryFile(delete=False, suffix=\".png\") as tmp:\n fake_path = tmp.name\n tmp.write(b\"fake image data\")\n \n mock_instance = mock_screenshot_class.return_value\n mock_instance.shoot.return_value = [fake_path]\n \n elog = get_test_elog()\n \n test_msg = \"SCREENSHOT_INTEGRATION_TEST_MSG_456\"\n> entry_id = elog.screenshot(message=test_msg)\n\ntests/test_utils_elog.py:143: \n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \nslic/utils/elog.py:22: in screenshot\n self.post(message, **kwargs)\nslic/utils/elog.py:17: in post\n return self._log.post(*args, **kwargs)\n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \n\nself = \nmessage = 'SCREENSHOT_INTEGRATION_TEST_MSG_456', msg_id = None, reply = False\nattributes = {'Author': 'robot', 'When': 1755001837, 'cmd': 'Submit', 'exp': 'demo', ...}\nattachments = ['/tmp/tmp1jhsgy5u.png'], suppress_email_notification = False\nencoding = None, timeout = None, kwargs = {'Author': 'robot'}\nnew_attachment_list = [('attfile0', ('tmp1jhsgy5u.png', <_io.BufferedReader name='/tmp/tmp1jhsgy5u.png'>)), ('Text', ('', b'SCREENSHOT_INTEGRATION_TEST_MSG_456'))]\nobjects_to_close = [<_io.BufferedReader name='/tmp/tmp1jhsgy5u.png'>]\nattributes_to_edit = {'Author': b'robot', 'When': 1755001837, 'cmd': b'Submit', 'exp': b'demo', ...}\n\n def post(self, message, msg_id=None, reply=False, attributes=None, attachments=None,\n suppress_email_notification=False, encoding=None, timeout=None, **kwargs):\n \"\"\"\n Posts message to the logbook. If msg_id is not specified new message will be created, otherwise existing\n message will be edited, or a reply (if reply=True) to it will be created. This method returns the msg_id\n of the newly created message.\n \"\"\"\n \n print(\"STARTING POST\")\n # Ajout des impressions pour d\u00e9boguer\n print(f\"Message to post: {message}\")\n print(f\"msg_id: {msg_id}\")\n print(f\"Attributes: {attributes}\")\n print(f\"Attachments: {attachments}\")\n print(f\"Encoding: {encoding}\")\n print(f\"Timeout: {timeout}\")\n print(f\"Additional kwargs: {kwargs}\")\n \n attributes = attributes or {}\n attributes = {**attributes, **kwargs} # kwargs as attributes with higher priority\n print(f\"Updated attributes: {attributes}\")\n \n attachments = attachments or []\n print(f\"Attachments list: {attachments}\")\n \n if encoding is not None:\n if encoding not in ['plain', 'HTML', 'ELCode']:\n raise LogbookMessageRejected('Invalid message encoding. Valid options: plain, HTML, ELCode.')\n attributes['Encoding'] = encoding\n \n if suppress_email_notification:\n attributes[\"suppress\"] = 1\n \n # Prepare attachments\n if attachments:\n new_attachment_list, objects_to_close = self._prepare_attachments(attachments)\n print(f\"New attachments prepared: {new_attachment_list}\")\n else:\n objects_to_close = []\n new_attachment_list = []\n \n attributes_to_edit = dict()\n \n if msg_id:\n print(f\"Editing message with msg_id: {msg_id}\")\n if reply:\n print(f\"Replying to message with msg_id: {msg_id}\")\n self._check_if_message_on_server(msg_id)\n attributes['reply_to'] = str(msg_id)\n else:\n print(\"Editing existing message.\")\n attributes['edit_id'] = str(msg_id)\n attributes['skiplock'] = '1'\n msg_to_edit, attributes_to_edit, existing_attachments_list = self.read(msg_id)\n \n # Merge new attributes\n for attribute, data in attributes.items():\n if data is not None:\n attributes_to_edit[attribute] = data\n \n print(f\"Attributes after merging: {attributes_to_edit}\")\n \n # Process existing attachments\n i = 0\n existing_attachments_filename_list = []\n for attachment in existing_attachments_list:\n attributes_to_edit[f'attachment{i}'] = os.path.basename(attachment)\n existing_attachments_filename_list.append(os.path.basename(attachment)[14:])\n i += 1\n \n print(f\"Existing attachments: {existing_attachments_filename_list}\")\n \n duplicate_attachment_list = []\n for new_attachment in new_attachment_list:\n new_attachment_filename = new_attachment[1][0]\n print(f\"Checking new attachment: {new_attachment_filename}\")\n if new_attachment_filename in existing_attachments_filename_list:\n # Same attachment exists on the server, compare content\n new_attachment_content = new_attachment[1][1].read()\n new_attachment[1][1].seek(0)\n attachment_index = existing_attachments_filename_list.index(new_attachment_filename)\n existing_attachment_content = self.download_attachment(\n url=existing_attachments_list[attachment_index],\n timeout=timeout\n )\n if new_attachment_content == existing_attachment_content:\n print(f\"Duplicate attachment detected: {new_attachment_filename}\")\n duplicate_attachment_list.append(new_attachment)\n else:\n print(f\"Attachment content has changed: {new_attachment_filename}\")\n self.delete_attachment(msg_id, attributes=attributes_to_edit,\n attachment_id=attachment_index,\n timeout=timeout, text=msg_to_edit)\n existing_attachments_filename_list.pop(attachment_index)\n existing_attachments_list.pop(attachment_index)\n \n print(f\"Duplicate attachments to remove: {duplicate_attachment_list}\")\n \n # Remove duplicates\n for attach in duplicate_attachment_list:\n new_attachment_list.remove(attach)\n \n print(f\"Final new attachments list: {new_attachment_list}\")\n else:\n # Creating a new message, add timestamp if not present\n if 'When' not in attributes:\n attributes['When'] = int(datetime.now().timestamp())\n \n # Final check on attributes\n if not attributes_to_edit:\n attributes_to_edit = attributes\n \n print(f\"Final attributes to send to the server: {attributes_to_edit}\")\n \n # Remove reserved attributes\n _remove_reserved_attributes(attributes_to_edit)\n \n new_attachment_list.append(('Text', ('', message.encode('iso-8859-1'))))\n print(f\"Final attachment list including message text: {new_attachment_list}\")\n \n # Add base message attributes\n self._add_base_msg_attributes(attributes_to_edit)\n print(f\"Attributes with base message added: {attributes_to_edit}\")\n \n # Sanitize attribute keys\n attributes_to_edit = _replace_special_characters_in_attribute_keys(attributes_to_edit)\n print(f\"Attributes after sanitizing keys: {attributes_to_edit}\")\n \n # Encode all string values in latin1\n attributes_to_edit = _encode_values(attributes_to_edit)\n print(f\"Attributes after encoding: {attributes_to_edit}\")\n \n try:\n print(\"Sending POST request to the server...\")\n response = requests.post(self._url, data=attributes_to_edit, files=new_attachment_list,\n allow_redirects=False, verify=False, timeout=timeout)\n print(\"Response received:\", response)\n \n resp_message, resp_headers, resp_msg_id = _validate_response(response)\n print(f\"Response message: {resp_message}\")\n print(f\"Response headers: {resp_headers}\")\n print(f\"Response msg_id: {resp_msg_id}\")\n \n # Close file-like objects if needed\n for file_like_object in objects_to_close:\n if hasattr(file_like_object, 'close'):\n file_like_object.close()\n \n except requests.Timeout as e:\n print(\"Request timed out:\", e)\n raise LogbookServerTimeout(f\"Timeout while posting: {e}\")\n \n except requests.RequestException as e:\n print(\"Request exception:\", e)\n self._check_if_message_on_server(msg_id) # Raises exceptions if no message or no response from server\n raise LogbookServerProblem(f\"Cannot access logbook server: {e}\")\n \n if not resp_msg_id or resp_msg_id < 1:\n> raise LogbookInvalidMessageID(f\"Invalid message ID: {resp_msg_id}\")\nE slic.utils.logbook_exceptions.LogbookInvalidMessageID: Invalid message ID: None\n\nslic/utils/logbook.py:276: LogbookInvalidMessageID"}, "teardown": {"duration": 0.00018721073865890503, "outcome": "passed"}}], "warnings": [{"message": "invalid escape sequence \\-", "category": "DeprecationWarning", "when": "collect", "filename": "/workspace/tligui_y/slic/.pixi/envs/default/lib/python3.8/site-packages/bsread/h5.py", "lineno": 207}, {"message": "The module numpy.dual is deprecated. Instead of using dual, use the functions directly from numpy or scipy.", "category": "DeprecationWarning", "when": "collect", "filename": "/workspace/tligui_y/slic/.pixi/envs/default/lib/python3.8/site-packages/scipy/fft/__init__.py", "lineno": 97}]} \ No newline at end of file