leahi-testsuites / suite_leahi

tst_rinseback

Clone Tools

Stats

Commits this week: 1
Total Committers: 3
Last Commit: 02 Jul
Commits this week: 1

Lines of code count not available

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Wednesday 01 Jul
2:09 pm

Nicholas Ramirez

1:13 pm

Stephen Quong and Behrouz NematiPour

1:11 pm

Stephen Quong

9:50 am

Nicholas Ramirez

BN approves

BN approves

9:50 am

Nicholas Ramirez

BN approves

BN approves

9:50 am

Nicholas Ramirez

BN approves

BN approves

Friday 26 Jun
2:33 am

Vendor - TEL - Divya Sree Nidamamidi

Updated the test script with TDTreaRinsebackStates

Updated the test script with TDTreaRinsebackStates

2:32 am

Vendor - TEL - Divya Sree Nidamamidi

As per suggestion using TDTreaRinsebackStates in test script

As per suggestion using TDTreaRinsebackStates in test script

2:22 am

Vendor - TEL - Divya Sree Nidamamidi

I have updated the step by getting the value from rejections.conf file. For all the rejections message we will create new test script

I have updated the step by getting the value from rejections.conf file. For all the rejections message we will create new test script

Thursday 25 Jun
5:01 pm

Nicholas Ramirez

i am good with the implementaiton

i am good with the implementaiton

8:42 am

Nicholas Ramirez

i agree lets use the definitions in td_defs as these are the most updated ones and use TDTreaRinsebackStates.RINSEBACK_STOP_INIT_STATE.value

i agree lets use the definitions in td_defs as these are the most updated ones and use TDTreaRinsebackStates.RINSEBACK_STOP_INIT_STATE.value

Wednesday 24 Jun
7:32 pm

Behrouz NematiPour

The states that need .value come from the td_defs that inherit from DialinEnum, and the ones that don't have .value come from the ui_defs, which are simple classes, so they don't need .value since ...

The states that need .value come from the td_defs that inherit from DialinEnum, and the ones that don't have .value come from the ui_defs, which are simple classes, so they don't need .value since those do not support metadata.
The UI_defs seem to be what I think aren't maintained anymore and last time updated was the 21 months ago.

Anything we want shall be available in the td_defs.
as an example:

# TDTreatmentStates.TREATMENT_RINSEBACK_STATE sub states
@unique
class TDTreaRinsebackStates(DialinEnum):
    RINSEBACK_STOP_INIT_STATE           = 0                     # Start state (stopped) of the rinseback sub-mode state machine
    RINSEBACK_RUN_STATE                 = 1                     # Rinseback running state of the rinseback sub-mode state machine
    RINSEBACK_PAUSED_STATE              = 2                     # Rinseback paused state of the rinseback sub-mode state machine
    RINSEBACK_STOP_STATE                = 3                     # Rinseback stopped (done) state of the rinseback sub-mode state machine
    RINSEBACK_RUN_ADDITIONAL_STATE      = 4                     # Additional rinseback volume (10 mL) state of the rinseback sub-mode state machine
    RINSEBACK_RECONNECT_PATIENT_STATE   = 5                     # Rinseback patient reconnect state of the rinseback sub-mode state machine
    NUM_OF_RINSEBACK_STATES             = 6                     # Number of rinseback sub-mode states


We should follow the same rule in the simulators and plugins.

Nicholas Ramirez I still need your feedback and correct me if I am wrong.

6:23 pm

Behrouz NematiPour

Nicholas Ramirez Please investigate the Dial-In code.

Nicholas Ramirez Please investigate the Dial-In code.

6:22 pm

Behrouz NematiPour

Oh, what is the interval of the lambda call? I have never used it this way. Not sure if that is a good or bad idea. Did we check the performance, or if it blocks the test? Nicholas Ramirez Resolve ...

Oh, what is the interval of the lambda call?
I have never used it this way.
Not sure if that is a good or bad idea.
Did we check the performance, or if it blocks the test?
Nicholas Ramirez Resolve if you are fine with the implementation.

5:08 pm

Behrouz NematiPour

RESOLVED

RESOLVED

Wednesday 17 Jun
2:45 pm

Nicholas Ramirez

we can make a function to just def rinsebackrequest_Check(): return rinsebackrequest == 8 if this works better? but well have alot more functions elsewhere. Let us know Behrouz NematiPour but the...

we can make a function to just

def rinsebackrequest_Check():
return rinsebackrequest == 8

if this works better? but well have alot more functions elsewhere. Let us know Behrouz NematiPour but the majority of the test cases do indeed use the lambda

2:43 pm

Nicholas Ramirez

we the lamba because we are using the waitFor() to wait for this condition to be true. waitFor() calls the lambda multiple times until it returns True or the timeout expires.

we the lamba because we are using the waitFor() to wait for this condition to be true.

waitFor() calls the lambda multiple times until it returns True or the timeout expires.

2:14 pm

Behrouz NematiPour

Thanks for providing the definition of the lambda. https://devapps.diality.us/static/s73b8n/2static/images/wiki/icons/emoticons/biggrin.gif My main question is, why lambda? What is that condition i...

Thanks for providing the definition of the lambda.
My main question is, why lambda?
What is that condition in this lambda (what is rinsebackrequest == 8)?
If that has a common use case, better to be a function?

2:10 pm

Behrouz NematiPour

Nicholas Ramirez Could you please check why that is, what the difference is between the two, and whether there are any inconveniences with Dial-In we need to take care of?

Nicholas Ramirez Could you please check why that is, what the difference is between the two, and whether there are any inconveniences with Dial-In we need to take care of?

2:07 pm

Behrouz NematiPour

Does this work? Have you tested it? It seems like while all are using equal, they should overwrite the previous one. It is just a preference; if you can't find a way, that's fine to leave it as it...

Does this work?
Have you tested it?
It seems like while all are using equal, they should overwrite the previous one.

It is just a preference; if you can't find a way, that's fine to leave it as it was.

Tuesday 16 Jun
2:21 am

Vendor - TEL - Divya Sree Nidamamidi

Lambda is used here to pass a condition as a function that can be evaluated repeatedly without writing a full function definition.

Lambda is used here to pass a condition as a function that can be evaluated repeatedly without writing a full function definition.

2:03 am

Vendor - TEL - Divya Sree Nidamamidi

Added the separate line for each test cases, using \ getting error

Added the separate line for each test cases, using \ getting error

Monday 15 Jun
11:41 pm

Vendor - TEL - Divya Sree Nidamamidi

For TX states does not require the ".value"

For TX states does not require the ".value"

11:40 pm

Vendor - TEL - Divya Sree Nidamamidi

For TXStates.RINSEBACK_STOP_INIT_STATE does not require ".value"

For TXStates.RINSEBACK_STOP_INIT_STATE does not require ".value"

10:27 am

Behrouz NematiPour

can each tst_ be on a line, and break these down into lines with some \ or something?

can each tst_ be on a line, and break these down into lines with some \ or something?

10:25 am

Behrouz NematiPour

Please shortly explain the lambda use case her.

Please shortly explain the lambda use case her.

10:21 am

Behrouz NematiPour

Same comment for the enum " .value " for the rest of the calls here.

Same comment for the enum " .value " for the rest of the calls here.

10:18 am

Behrouz NematiPour

doesn't the TXStates.RINSEBACK_STOP_INIT_STATE need " .value " like the other enums?

doesn't the TXStates.RINSEBACK_STOP_INIT_STATE need " .value " like the other enums?

10:15 am

Behrouz NematiPour

What is this variable? If that is the rejection reason ID: 1, I don't think we should use it this way? We need to find the reason message/text for the rejection by the rejection ID. If that is the ...

What is this variable?
If that is the rejection reason ID: 1, I don't think we should use it this way?
We need to find the reason message/text for the rejection by the rejection ID.
If that is the only case, let me know where it is used and why only one case was tested, rather than testing the rest.
The entire rejection system needs to be tested in a separate test suite if we have not already done so.

Sunday 07 Jun
11:50 pm

Vendor - TEL - Divya Sree Nidamamidi

Payload has been updated and test script is working

Payload has been updated and test script is working