This is a list of all comments for LEAHI-TESTSUITES-LDT-1078-1. Review Summary: No summary ---------------------------------------- File: suite_leahi/shared/scripts/configuration/strings.py Revision Comment by Nicholas Ramirez on 11 November 2025, 14:45 https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1078-1#c25060 remove this file and port anything needed into config.py for consistency just like we are doing in http://devapps.diality.us:8060/cru/LEAHI-TESTSUITES-LDT-1650-1 Reply by Vendor - TEL - Shivaprasad Basavaraju on 12 November 2025, 22:40 > I have deleted strings.py file from Bitbucket branch > feature/LDT-1078-Device-Settings-Information-Version ---------------------------------------- File: suite_leahi/shared/scripts/names.py Revision Comment by Nicholas Ramirez on 11 November 2025, 14:46 https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1078-1#c25061 please update indentation Reply by Vendor - TEL - Shivaprasad Basavaraju on 18 November 2025, 12:05 > updated ---------------------------------------- File: suite_leahi/tst_device_settings_information_version/test.py Revision Comment by Nicholas Ramirez on 03 December 2025, 10:35 https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1078-1#c25481 can we make this a more general method and put in a shared file and update throughout.. update to something like {code}def setObjectText(obj, text): obj["text"] = text return obj:{code} Reply by Vendor - TEL - Shivaprasad Basavaraju on 05 December 2025, 00:36 > Now using setText(obj, text) method and moved to > configuration folder utility.py file Revision Comment by Nicholas Ramirez on 03 December 2025, 10:35 https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1078-1#c25482 move outside as its used in another test case and update other test case Reply by Vendor - TEL - Shivaprasad Basavaraju on 05 December 2025, 00:37 > Now get_aut_version(aut_path): method is moved to > configuration folder application_init.py file Revision Comment by Nicholas Ramirez on 14 November 2025, 09:31 https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1078-1#c25157 for this do not hard code as it will only work with this specific build. Use"./leahi --version" to get the version then test that with what we display in GUI Reply by Vendor - TEL - Shivaprasad Basavaraju on 18 November 2025, 12:02 > Add get_aut_version function and getting UI version using > subprocess Revision Comment by Behrouz NematiPour on 05 December 2025, 12:00 https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1078-1#c25585 Not a comment to be done right now. But better to create arrays of test data on top of the function/file, loop through them, and call the function (verify_td_and_dd_versions) to use those. So we can more easily add multiple test data cases later to cover more edge cases. Revision Comment by Nicholas Ramirez on 03 December 2025, 10:18 https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1078-1#c25478 ive noticed this common method being used in other test cases. If its a general method that is used throughout please move outside in a shared file and call from there. Please update other repositories that use this as well. Reply by Vendor - TEL - Shivaprasad Basavaraju on 05 December 2025, 00:38 > Now findObjectById(parent, id): method is moved to > configuration folder utility.py file ---------------------------------------- File: suite_leahi/shared/scripts/configuration/config.py Revision Comment by Nicholas Ramirez on 19 November 2025, 12:11 https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1078-1#c25192 can you get the path relatively and not hard coded. This path is specific to your VM Reply by Vendor - TEL - Shivaprasad Basavaraju on 26 November 2025, 08:16 > Now I am getting AUT path using aut_path = > currentApplicationContext().cwd in test script --- ID: LEAHI-TESTSUITES-LDT-1078-1 https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1078-1 Title: LEAHI-TESTSUITES-LDT-1078_Device Settings - SW - 02 - Information - Version - Q&R Statement of Objectives: State: Closed Summary: Author: Vendor - TEL - Sivvanarayana Kurapati Moderator: Nicholas Ramirez Reviewers: (5 active, 1 completed*) Behrouz NematiPour (*) Dara Navaei Vendor - TEL - Shivaprasad Basavaraju Daniel Ho Stephen Quong Vendor - TEL - Divya Sree Nidamamidi