This is a list of all comments for LEAHI-TESTSUITES-LDT-1802-1.

Review Summary: No summary

General Comment by Nicholas Ramirez on 05 December 2025, 14:48 
https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1802-1#c25596

  Please add the test case to the suite.conf

  Reply by Vendor - TEL - Shivaprasad Basavaraju on 08 December 2025, 21:32
  > Add tst_general_alarm_requirements_instruction to suite.conf
  > file


----------------------------------------
File: suite_leahi/shared/scripts/names.py
Revision Comment by Nicholas Ramirez on 26 November 2025, 10:24 
https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1802-1#c25313

  please keep this CR to only items used for this CR.

  Reply by Vendor - TEL - Shivaprasad Basavaraju on 02 December 2025, 03:16
  > updated. Removed header bar information and device settings
  > information objects

----------------------------------------
File: suite_leahi/tst_general_alarm_requirements_instruction/test.py
Revision Comment by Nicholas Ramirez on 02 December 2025, 15:02 
https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1802-1#c25439

  Please update the report and all test cases to match this
  workflow.

  Reply by Vendor - TEL - Shivaprasad Basavaraju on 03 December 2025, 01:32
  > I have updated the workflow text

Revision Comment by Behrouz NematiPour on 12 December 2025, 01:53 
https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1802-1#c25771

  Shouldn't this function be defined in a more general
  location/file?
  And same comment here to be aware of the translation
  flexibility for later.

Revision Comment by Behrouz NematiPour on 12 December 2025, 01:56 
https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1802-1#c25772

  Since this is being used everywhere, I would recommend defining
  a global variable(s) so that we can later be flexible to adjust
  that for all the mouse clicks or watforobject functions

Revision Comment by Nicholas Ramirez on 03 December 2025, 09:44 
https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1802-1#c25471

  remove the outer startSection("Verification of General Alarm
  Requirements ") as it encapsulates the entire test case not
  section. Please reupload new report as well

  Reply by Vendor - TEL - Shivaprasad Basavaraju on 08 December 2025, 21:30
  > removed the outer startSection and uploaded the new test
  > report

----------------------------------------
File: suite_leahi/shared/scripts/configuration/config.py
Revision Comment by Behrouz NematiPour on 12 December 2025, 01:50 
https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1802-1#c25770

  Not able to find in your test.py, but these types of texts will
  change with the translation, and if the text is used as has
  been defined here, your test will fail in other languages.
  Just be aware and think of a solution to be flexible for later
  when testing with other languages.

----------------------------------------
File: suite_leahi/suite.conf
Revision Comment by Nicholas Ramirez on 10 December 2025, 08:38 
https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1802-1#c25672

  all the test cases need ot be listed here. Only add on your new
  test case and do not remove the others

  Reply by Vendor - TEL - Shivaprasad Basavaraju on 10 December 2025, 22:32
  > updated as per comment


---

ID: LEAHI-TESTSUITES-LDT-1802-1 https://devapps.diality.us/cru/LEAHI-TESTSUITES-LDT-1802-1

Title: LEAHI-TESTSUITES-LDT-1802_General Alarm Requirements - SW  - 02 - Instructions with new design - Q&R
Statement of Objectives:


State: Review

Author: Vendor - TEL - Shivaprasad Basavaraju
Moderator: Nicholas Ramirez
Reviewers: (7 active, 0 completed*)
     Tiffany Mejia
     Dara Navaei
     Vendor - TEL - Sivvanarayana Kurapati
     Behrouz NematiPour
     Daniel Ho
     Stephen Quong
     Vendor - TEL - Divya Sree Nidamamidi