diff --git a/markdown/TEST-REPORT.md b/markdown/TEST-REPORT.md index 76074b14c..78910ac08 100644 --- a/markdown/TEST-REPORT.md +++ b/markdown/TEST-REPORT.md @@ -1,20 +1,21 @@ # ๐Ÿงช Test Report -*Generated on 2025-08-15 16:10:31 CEST* +*Generated on 2025-08-15 16:19:28 CEST* ## ๐Ÿงพ General Info -- **duration**: 5.3575439453125 +- **duration**: 5.780713319778442 - **root**: /workspace/tligui_y/slic - **environment**: {} ## ๐Ÿ“‹ Summary -- **Passed**: 1 -- **Total**: 1 -- **Collected**: 1 +- **Passed**: 2 +- **Failed**: 3 +- **Total**: 5 +- **Collected**: 5 ## ๐Ÿ”Ž Tests
-โœ… Passed (1) +โœ… Passed (2) -
๐Ÿ“„ test_utils_elog.py @@ -28,7 +29,7 @@ **duration:** ```python - 0.00040102191269397736 + 0.0004252651706337929 ``` **outcome:** @@ -42,7 +43,7 @@ **duration:** ```python - 0.028066603932529688 + 0.02794818300753832 ``` **outcome:** @@ -56,7 +57,432 @@ **duration:** ```python - 0.0003600991331040859 + 0.0003295326605439186 + ``` + + **outcome:** + + ```python + passed + ``` +
+ โ†ณ Function: test_get_default_elog_instance_asks_password_and_opens + -
+ โœ… Test 3 + + **_*๐Ÿ“Œ Setup phase*_** + + **duration:** + + ```python + 0.00014708423987030983 + ``` + + **outcome:** + + ```python + passed + ``` + + **_*๐Ÿ“Œ Call phase*_** + + **duration:** + + ```python + 0.02600018261000514 + ``` + + **outcome:** + + ```python + passed + ``` + + **_*๐Ÿ“Œ Teardown phase*_** + + **duration:** + + ```python + 0.000192960724234581 + ``` + + **outcome:** + + ```python + passed + ``` +
+
+ + + +
+โŒ Failed (3) + + -
+ ๐Ÿ“„ test_utils_elog.py + + โ†ณ Function: test_get_default_elog_instance_with_wrong_password_and_real_check + -
+ โŒ Test 2 + + **_*๐Ÿ“Œ Setup phase*_** + + **duration:** + + ```python + 0.0001492779701948166 + ``` + + **outcome:** + + ```python + passed + ``` + + **_*๐Ÿ“Œ Call phase*_** + + **duration:** + + ```python + 0.006722141057252884 + ``` + + **outcome:** + + ```python + failed + ``` + + **crash:** + + ```python + path: /workspace/tligui_y/slic/slic/utils/logbook.py + lineno: 833 + message: slic.utils.logbook_exceptions.LogbookAuthenticationError: Invalid username or password. + ``` + + **traceback:** + + ```python + - path: tests/test_utils_elog.py + lineno: 40 + message: None + - path: slic/utils/elog.py + lineno: 17 + message: in post + - path: slic/utils/logbook.py + lineno: 273 + message: in post + - path: slic/utils/logbook.py + lineno: 833 + message: LogbookAuthenticationError + ``` + + **longrepr:** + + ```python + def test_get_default_elog_instance_with_wrong_password_and_real_check(): + url = "http://localhost:8080/demo" + user = "robot" + wrong_password = "wrongpassword" + + elog = Elog(url, user=user, password=wrong_password) + + with pytest.raises(LogbookAuthenticationError): + > elog.post("Should fail because wrong password") + + tests/test_utils_elog.py:40: + _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ + slic/utils/elog.py:17: in post + return self._log.post(*args, **kwargs) + slic/utils/logbook.py:273: in post + resp_message, resp_headers, resp_msg_id = _validate_response(response) + _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ + + response = + + def _validate_response(response): + """ Validate response of the request.""" + print("[DEBUG] _validate_response called") + print(f" HTTP status code : {response.status_code}") + print(f" Response headers : {response.headers}") + print(f" First 300 bytes of content:\n{response.content[:300]!r}") + + msg_id = None + + if response.status_code not in [200, 302]: + print("[DEBUG] Status not 200/302 โ†’ searching for ") + err = re.findall('.*?', + response.content.decode('utf-8', 'ignore'), + flags=re.DOTALL) + if len(err) > 0: + err = re.sub('(?:<.*?>)', '', err[0]) + print(f"[DEBUG] Found error message: {err}") + if err: + raise LogbookMessageRejected('Rejected because of: ' + err) + else: + raise LogbookMessageRejected('Rejected because of unknown error.') + raise LogbookMessageRejected('Rejected because of unknown error.') + else: + location = response.headers.get('Location') + if location is not None: + print(f"[DEBUG] Location header: {location}") + if 'has moved' in location: + raise LogbookServerProblem('Logbook server has moved to another location.') + elif 'fail' in location: + raise LogbookAuthenticationError('Invalid username or password.') + else: + try: + msg_id = int(urllib.parse.urlsplit(location).path.split('/')[-1]) + print(f"[DEBUG] Parsed msg_id from Location: {msg_id}") + except ValueError: + print("[DEBUG] Could not parse msg_id from Location") + msg_id = None + + if b'type=password' in response.content or b'type="password"' in response.content: + print("[DEBUG] Password input detected in response HTML โ€” assuming login page โ†’ AUTH ERROR") + > raise LogbookAuthenticationError('Invalid username or password.') + E slic.utils.logbook_exceptions.LogbookAuthenticationError: Invalid username or password. + + slic/utils/logbook.py:833: LogbookAuthenticationError + ``` + + **_*๐Ÿ“Œ Teardown phase*_** + + **duration:** + + ```python + 0.00030977511778473854 + ``` + + **outcome:** + + ```python + passed + ``` +
+ โ†ณ Function: test_get_default_elog_with_path_home + -
+ โŒ Test 4 + + **_*๐Ÿ“Œ Setup phase*_** + + **duration:** + + ```python + 0.00014149397611618042 + ``` + + **outcome:** + + ```python + passed + ``` + + **_*๐Ÿ“Œ Call phase*_** + + **duration:** + + ```python + 0.015367177315056324 + ``` + + **outcome:** + + ```python + failed + ``` + + **crash:** + + ```python + path: /workspace/tligui_y/slic/tests/test_utils_elog.py + lineno: 97 + message: NameError: name 'user' is not defined + ``` + + **traceback:** + + ```python + - path: tests/test_utils_elog.py + lineno: 97 + 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 + text = "This is a message3" + url = "http://localhost:8080/demo" + + 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 + + try: + elog = Elog("http://localhost:8080/demo") + try: + msg_id = elog.post(text) + except Exception as e: + pytest.fail(f"elog.open() raised an unexpected exception: {e}") + + message, attributes, attachments = elog.read(msg_id) + + assert text in message + > assert attributes.get("Author") == user + E NameError: name 'user' is not defined + + tests/test_utils_elog.py:97: NameError + ``` + + **_*๐Ÿ“Œ Teardown phase*_** + + **duration:** + + ```python + 0.0002582320012152195 + ``` + + **outcome:** + + ```python + passed + ``` +
+ โ†ณ Function: test_screenshot + -
+ โŒ Test 5 + + **_*๐Ÿ“Œ Setup phase*_** + + **duration:** + + ```python + 0.000149404164403677 + ``` + + **outcome:** + + ```python + passed + ``` + + **_*๐Ÿ“Œ Call phase*_** + + **duration:** + + ```python + 0.012440725229680538 + ``` + + **outcome:** + + ```python + failed + ``` + + **crash:** + + ```python + path: /workspace/tligui_y/slic/slic/utils/logbook.py + lineno: 811 + message: slic.utils.logbook_exceptions.LogbookMessageRejected: Rejected because of: Invalid URL: None + ``` + + **traceback:** + + ```python + - path: tests/test_utils_elog.py + lineno: 119 + message: None + - path: slic/utils/logbook.py + lineno: 316 + message: in read + - path: slic/utils/logbook.py + lineno: 575 + message: in _check_if_message_on_server + - path: slic/utils/logbook.py + lineno: 811 + message: LogbookMessageRejected + ``` + + **longrepr:** + + ```python + mock_screenshot_class = + + @patch("slic.utils.elog.Screenshot") + def test_screenshot(mock_screenshot_class): + fake_path = "/tmp/fake_screenshot.png" + with open(fake_path, "wb") as f: + f.write(b"fake image data") + + mock_instance = mock_screenshot_class.return_value + mock_instance.shoot.return_value = [fake_path] + + elog = elog = Elog("http://localhost:8080/demo", user="robot", password="testpassword") + + test_msg = "SCREENSHOT_INTEGRATION_TEST_MSG_456" + msg_id = elog.screenshot(message=test_msg) + + filename = os.path.basename(fake_path) + > message, attributes, attachments = elog.read(msg_id) + + tests/test_utils_elog.py:119: + _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ + slic/utils/logbook.py:316: in read + self._check_if_message_on_server(msg_id) # raises exceptions if no message or no response from server + slic/utils/logbook.py:575: in _check_if_message_on_server + resp_message, resp_headers, resp_msg_id = _validate_response(response) + _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ + + response = + + def _validate_response(response): + """ Validate response of the request.""" + print("[DEBUG] _validate_response called") + print(f" HTTP status code : {response.status_code}") + print(f" Response headers : {response.headers}") + print(f" First 300 bytes of content:\n{response.content[:300]!r}") + + msg_id = None + + if response.status_code not in [200, 302]: + print("[DEBUG] Status not 200/302 โ†’ searching for ") + err = re.findall('.*?', + response.content.decode('utf-8', 'ignore'), + flags=re.DOTALL) + if len(err) > 0: + err = re.sub('(?:<.*?>)', '', err[0]) + print(f"[DEBUG] Found error message: {err}") + if err: + > raise LogbookMessageRejected('Rejected because of: ' + err) + E slic.utils.logbook_exceptions.LogbookMessageRejected: Rejected because of: Invalid URL: None + + slic/utils/logbook.py:811: LogbookMessageRejected + ``` + + **_*๐Ÿ“Œ Teardown phase*_** + + **duration:** + + ```python + 0.0002933577634394169 ``` **outcome:** @@ -100,6 +526,18 @@ - nodeid: tests/test_utils_elog.py::test_get_default_elog_instance_with_direct_password_and_real_check type: Function lineno: 12 + - nodeid: tests/test_utils_elog.py::test_get_default_elog_instance_with_wrong_password_and_real_check + type: Function + lineno: 31 + - nodeid: tests/test_utils_elog.py::test_get_default_elog_instance_asks_password_and_opens + type: Function + lineno: 42 + - nodeid: tests/test_utils_elog.py::test_get_default_elog_with_path_home + type: Function + lineno: 69 + - nodeid: tests/test_utils_elog.py::test_screenshot + type: Function + lineno: 103 ```
diff --git a/markdown/coverage-summary.md b/markdown/coverage-summary.md index a85dd73c2..82c512723 100644 --- a/markdown/coverage-summary.md +++ b/markdown/coverage-summary.md @@ -38416,3 +38416,230 @@ | slic/utils/utils.py | 17 | 8 | 53% | | slic/utils/xrange.py | 33 | 30 | 9% | | **TOTAL** | **9985** | **7235** | **28%** | +| 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 | 24 | 40% | +| 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 | 32 | 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 | 442 | 213 | 52% | +| 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 | 55 | 42 | 24% | +| 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** | **9985** | **7188** | **28%** | diff --git a/markdown/pytest-report.json b/markdown/pytest-report.json index cf58138c1..7b557bd2e 100644 --- a/markdown/pytest-report.json +++ b/markdown/pytest-report.json @@ -1 +1 @@ -{"created": 1755267029.7344332, "duration": 5.3575439453125, "exitcode": 0, "root": "/workspace/tligui_y/slic", "environment": {}, "summary": {"passed": 1, "total": 1, "collected": 1}, "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_get_default_elog_instance_with_direct_password_and_real_check", "type": "Function", "lineno": 12}]}], "tests": [{"nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_with_direct_password_and_real_check", "lineno": 12, "outcome": "passed", "keywords": ["test_get_default_elog_instance_with_direct_password_and_real_check", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.00040102191269397736, "outcome": "passed"}, "call": {"duration": 0.028066603932529688, "outcome": "passed"}, "teardown": {"duration": 0.0003600991331040859, "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": 1755267567.0307307, "duration": 5.780713319778442, "exitcode": 1, "root": "/workspace/tligui_y/slic", "environment": {}, "summary": {"passed": 2, "failed": 3, "total": 5, "collected": 5}, "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_get_default_elog_instance_with_direct_password_and_real_check", "type": "Function", "lineno": 12}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_with_wrong_password_and_real_check", "type": "Function", "lineno": 31}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_asks_password_and_opens", "type": "Function", "lineno": 42}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_with_path_home", "type": "Function", "lineno": 69}, {"nodeid": "tests/test_utils_elog.py::test_screenshot", "type": "Function", "lineno": 103}]}], "tests": [{"nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_with_direct_password_and_real_check", "lineno": 12, "outcome": "passed", "keywords": ["test_get_default_elog_instance_with_direct_password_and_real_check", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.0004252651706337929, "outcome": "passed"}, "call": {"duration": 0.02794818300753832, "outcome": "passed"}, "teardown": {"duration": 0.0003295326605439186, "outcome": "passed"}}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_with_wrong_password_and_real_check", "lineno": 31, "outcome": "failed", "keywords": ["test_get_default_elog_instance_with_wrong_password_and_real_check", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.0001492779701948166, "outcome": "passed"}, "call": {"duration": 0.006722141057252884, "outcome": "failed", "crash": {"path": "/workspace/tligui_y/slic/slic/utils/logbook.py", "lineno": 833, "message": "slic.utils.logbook_exceptions.LogbookAuthenticationError: Invalid username or password."}, "traceback": [{"path": "tests/test_utils_elog.py", "lineno": 40, "message": ""}, {"path": "slic/utils/elog.py", "lineno": 17, "message": "in post"}, {"path": "slic/utils/logbook.py", "lineno": 273, "message": "in post"}, {"path": "slic/utils/logbook.py", "lineno": 833, "message": "LogbookAuthenticationError"}], "longrepr": "def test_get_default_elog_instance_with_wrong_password_and_real_check():\n url = \"http://localhost:8080/demo\"\n user = \"robot\"\n wrong_password = \"wrongpassword\"\n \n elog = Elog(url, user=user, password=wrong_password)\n \n with pytest.raises(LogbookAuthenticationError):\n> elog.post(\"Should fail because wrong password\")\n\ntests/test_utils_elog.py:40: \n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \nslic/utils/elog.py:17: in post\n return self._log.post(*args, **kwargs)\nslic/utils/logbook.py:273: in post\n resp_message, resp_headers, resp_msg_id = _validate_response(response)\n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \n\nresponse = \n\n def _validate_response(response):\n \"\"\" Validate response of the request.\"\"\"\n print(\"[DEBUG] _validate_response called\")\n print(f\" HTTP status code : {response.status_code}\")\n print(f\" Response headers : {response.headers}\")\n print(f\" First 300 bytes of content:\\n{response.content[:300]!r}\")\n \n msg_id = None\n \n if response.status_code not in [200, 302]:\n print(\"[DEBUG] Status not 200/302 \u2192 searching for \")\n err = re.findall('.*?',\n response.content.decode('utf-8', 'ignore'),\n flags=re.DOTALL)\n if len(err) > 0:\n err = re.sub('(?:<.*?>)', '', err[0])\n print(f\"[DEBUG] Found error message: {err}\")\n if err:\n raise LogbookMessageRejected('Rejected because of: ' + err)\n else:\n raise LogbookMessageRejected('Rejected because of unknown error.')\n raise LogbookMessageRejected('Rejected because of unknown error.')\n else:\n location = response.headers.get('Location')\n if location is not None:\n print(f\"[DEBUG] Location header: {location}\")\n if 'has moved' in location:\n raise LogbookServerProblem('Logbook server has moved to another location.')\n elif 'fail' in location:\n raise LogbookAuthenticationError('Invalid username or password.')\n else:\n try:\n msg_id = int(urllib.parse.urlsplit(location).path.split('/')[-1])\n print(f\"[DEBUG] Parsed msg_id from Location: {msg_id}\")\n except ValueError:\n print(\"[DEBUG] Could not parse msg_id from Location\")\n msg_id = None\n \n if b'type=password' in response.content or b'type=\"password\"' in response.content:\n print(\"[DEBUG] Password input detected in response HTML \u2014 assuming login page \u2192 AUTH ERROR\")\n> raise LogbookAuthenticationError('Invalid username or password.')\nE slic.utils.logbook_exceptions.LogbookAuthenticationError: Invalid username or password.\n\nslic/utils/logbook.py:833: LogbookAuthenticationError"}, "teardown": {"duration": 0.00030977511778473854, "outcome": "passed"}}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_asks_password_and_opens", "lineno": 42, "outcome": "passed", "keywords": ["test_get_default_elog_instance_asks_password_and_opens", "__wrapped__", "patchings", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.00014708423987030983, "outcome": "passed"}, "call": {"duration": 0.02600018261000514, "outcome": "passed"}, "teardown": {"duration": 0.000192960724234581, "outcome": "passed"}}, {"nodeid": "tests/test_utils_elog.py::test_get_default_elog_with_path_home", "lineno": 69, "outcome": "failed", "keywords": ["test_get_default_elog_with_path_home", "__wrapped__", "patchings", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.00014149397611618042, "outcome": "passed"}, "call": {"duration": 0.015367177315056324, "outcome": "failed", "crash": {"path": "/workspace/tligui_y/slic/tests/test_utils_elog.py", "lineno": 97, "message": "NameError: name 'user' is not defined"}, "traceback": [{"path": "tests/test_utils_elog.py", "lineno": 97, "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 text = \"This is a message3\"\n url = \"http://localhost:8080/demo\"\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 try:\n elog = Elog(\"http://localhost:8080/demo\")\n try:\n msg_id = elog.post(text)\n except Exception as e:\n pytest.fail(f\"elog.open() raised an unexpected exception: {e}\")\n \n message, attributes, attachments = elog.read(msg_id)\n \n assert text in message\n> assert attributes.get(\"Author\") == user\nE NameError: name 'user' is not defined\n\ntests/test_utils_elog.py:97: NameError"}, "teardown": {"duration": 0.0002582320012152195, "outcome": "passed"}}, {"nodeid": "tests/test_utils_elog.py::test_screenshot", "lineno": 103, "outcome": "failed", "keywords": ["test_screenshot", "__wrapped__", "patchings", "test_utils_elog.py", "tests", "slic", ""], "setup": {"duration": 0.000149404164403677, "outcome": "passed"}, "call": {"duration": 0.012440725229680538, "outcome": "failed", "crash": {"path": "/workspace/tligui_y/slic/slic/utils/logbook.py", "lineno": 811, "message": "slic.utils.logbook_exceptions.LogbookMessageRejected: Rejected because of: Invalid URL: None"}, "traceback": [{"path": "tests/test_utils_elog.py", "lineno": 119, "message": ""}, {"path": "slic/utils/logbook.py", "lineno": 316, "message": "in read"}, {"path": "slic/utils/logbook.py", "lineno": 575, "message": "in _check_if_message_on_server"}, {"path": "slic/utils/logbook.py", "lineno": 811, "message": "LogbookMessageRejected"}], "longrepr": "mock_screenshot_class = \n\n @patch(\"slic.utils.elog.Screenshot\")\n def test_screenshot(mock_screenshot_class):\n fake_path = \"/tmp/fake_screenshot.png\"\n with open(fake_path, \"wb\") as f:\n f.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 = elog = Elog(\"http://localhost:8080/demo\", user=\"robot\", password=\"testpassword\")\n \n test_msg = \"SCREENSHOT_INTEGRATION_TEST_MSG_456\"\n msg_id = elog.screenshot(message=test_msg)\n \n filename = os.path.basename(fake_path)\n> message, attributes, attachments = elog.read(msg_id)\n\ntests/test_utils_elog.py:119: \n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \nslic/utils/logbook.py:316: in read\n self._check_if_message_on_server(msg_id) # raises exceptions if no message or no response from server\nslic/utils/logbook.py:575: in _check_if_message_on_server\n resp_message, resp_headers, resp_msg_id = _validate_response(response)\n_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ \n\nresponse = \n\n def _validate_response(response):\n \"\"\" Validate response of the request.\"\"\"\n print(\"[DEBUG] _validate_response called\")\n print(f\" HTTP status code : {response.status_code}\")\n print(f\" Response headers : {response.headers}\")\n print(f\" First 300 bytes of content:\\n{response.content[:300]!r}\")\n \n msg_id = None\n \n if response.status_code not in [200, 302]:\n print(\"[DEBUG] Status not 200/302 \u2192 searching for \")\n err = re.findall('.*?',\n response.content.decode('utf-8', 'ignore'),\n flags=re.DOTALL)\n if len(err) > 0:\n err = re.sub('(?:<.*?>)', '', err[0])\n print(f\"[DEBUG] Found error message: {err}\")\n if err:\n> raise LogbookMessageRejected('Rejected because of: ' + err)\nE slic.utils.logbook_exceptions.LogbookMessageRejected: Rejected because of: Invalid URL: None\n\nslic/utils/logbook.py:811: LogbookMessageRejected"}, "teardown": {"duration": 0.0002933577634394169, "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 diff --git a/markdown/runtime_params.json b/markdown/runtime_params.json index d929ca634..b7679c72f 100644 --- a/markdown/runtime_params.json +++ b/markdown/runtime_params.json @@ -2,5 +2,21 @@ { "nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_with_direct_password_and_real_check", "callspec": null + }, + { + "nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_with_wrong_password_and_real_check", + "callspec": null + }, + { + "nodeid": "tests/test_utils_elog.py::test_get_default_elog_instance_asks_password_and_opens", + "callspec": null + }, + { + "nodeid": "tests/test_utils_elog.py::test_get_default_elog_with_path_home", + "callspec": null + }, + { + "nodeid": "tests/test_utils_elog.py::test_screenshot", + "callspec": null } ] \ No newline at end of file