Removed the ine

delete line

· /suite_leahi/shared/scripts/configuration/config.py	(-2, +4)	| History	| Source	| Diff
· /suite_leahi/shared/scripts/configuration/navigation.py	(-4, +37)	| History	| Source	| Diff
· /suite_leahi/shared/scripts/configuration/utility.py	(-12, +13)	| History	| Source	| Diff
· /suite_leahi/shared/scripts/names.py	(-34, +67)	| History	| Source	| Diff
· /suite_leahi/suite.conf	(-1, +1)	| History	| Source	| Diff
· /suite_leahi/tst_heparin/test.py	(-3, +3)	| History	| Source	| Diff
· /suite_leahi/tst_isolated_uf/test.py	(-0, +213)	| History	| Source
· /suite_leahi/tst_main_treatment/test.py	(-2, +2)	| History	| Source	| Diff
· /suite_leahi/tst_ultrafiltration/test.py	(-8, +8)	| History	| Source	| Diff

]]> Thu, 05 Feb 2026 16:43:06 -0800 Nicholas Ramirez https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-2971-1 https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-2971-1 Nicholas Ramirez Closed LEAHI-TESTSUITES-LDT-2971-1 ]]> Thu, 05 Feb 2026 15:25:24 -0800 https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-2971-1 https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-2971-1 Behrouz NematiPour finished reviewing LEAHI-TESTSUITES-LDT-2971-1 LEAHI-TESTSUITES-LDT-2971_Heparin - SW - 02 - Q&R ]]> Thu, 05 Feb 2026 15:22:40 -0800 Behrouz NematiPour https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-2971-1/#c27071 https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-2971-1/#c27071 Behrouz NematiPour replied to Nicholas Ramirez in LEAHI-TESTSUITES-LDT-2971-1

ok them be careful to always use it as:

import squish

NOT the following as has been stated:

from squish import  *

i believe it is necessary for a custom module.. https://doc.qt.io/squish/python-notes.html#squish-funcs-in-py-modules

Removed the from squish import * line and keeping the import squish in utility.py and navigation.py files

without squish import in utility file scripts will fail so added import squish line in utility.py and navigation.py file

Please remove squish import everywhere.
It is also duplicate here.

try catch is avoided as per suggestion

try catch is removed

Moved all the constants to config.py file

Removed the line

Method name is changed and added docstring and method is available in navigation.py file

Method is moved to navigation.py file

code is aligned

All strings are placed in config.py

All strings are placed in config.py

· /suite_leahi/shared/scripts/configuration/utility.py	(-2, +2)	| History	| Source	| Diff
· /suite_leahi/suite.conf	(-1, +1)	| History	| Source	| Diff
· /suite_leahi/tst_heparin/test.py	(-2, +2)	| History	| Source	| Diff

]]> Mon, 02 Feb 2026 02:08:51 -0800 Vendor - TEL - Divya Sree Nidamamidi https://devapps.diality.us/changelog/leahi-testsuites?cs=fb27385d1c510f6d3b433ace7e2f2e7d5dda4012 https://devapps.diality.us/changelog/leahi-testsuites?cs=fb27385d1c510f6d3b433ace7e2f2e7d5dda4012 Vendor - TEL - Divya Sree Nidamamidi committed fb27385 on branch feature/LDT-2278-Patient-Vitals, feature/LDT-2741-Service and 14 more in leahi-testsuites LDT -2973 Heparin - SW - 02 - Q&R - 15: SIT - Software Integration Tests - Squish Qt

· /suite_leahi/shared/scripts/configuration/config.py	(-0, +29)	| History	| Source	| Diff
· /suite_leahi/shared/scripts/configuration/navigation.py	(-1, +26)	| History	| Source	| Diff
· /suite_leahi/shared/scripts/configuration/utility.py	(-44, +22)	| History	| Source	| Diff
· /suite_leahi/tst_heparin/test.py	(-34, +18)	| History	| Source	| Diff

]]> Fri, 30 Jan 2026 10:13:46 -0800 Behrouz NematiPour https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-2971-1/#c26884 https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-2971-1/#c26884 Behrouz NematiPour commented on LEAHI-TESTSUITES-LDT-2971-1

Please add a docstring for this function.
And the naming seems like to be better like:

navigation_pageIndicator_step(vStep_name)

Same (translation)

The translatable strings shall be defined in a global space.
Generally, please make sure this is handled everywhere.

should this method be moved to navigation.py?

remove commented out line

please move these into the config.py to be consistent with other test cases

When appropriate, align the code.

As a general note, avoid try-catch.
Your code should be prepared for exceptions, not to be surprised.