· /shared/scripts/names.py	(-1, +1)	| History	| Source	| Diff
· /simulator/plugins/alarms/loader.py	(-1, +1)	| History	| Source	| Diff
· /simulator/plugins/createtreatment/loader.py	(-1, +1)	| History	| Source	| Diff
· /simulator/plugins/heparin/loader.py	(-1, +1)	| History	| Source	| Diff
· /simulator/plugins/inlinebloodpressures/loader.py	(-1, +1)	| History	| Source	| Diff
· /simulator/plugins/salinebolus/loader.py	(-1, +1)	| History	| Source	| Diff
· /simulator/plugins/treatmentranges/loader.py	(-1, +1)	| History	| Source	| Diff
· /simulator/plugins/treatmentstates/loader.py	(-1, +1)	| History	| Source	| Diff
· /simulator/plugins/ultrafiltration/loader.py	(-1, +1)	| History	| Source	| Diff
· /simulator/run.py	(-1, +1)	| History	| Source	| Diff
· /simulator/run.sh	(-1, +1)	| History	| Source	| Diff
· /simulator/simulator/dynamicloader.py	(-1, +1)	| History	| Source	| Diff
· /simulator/simulator/loader.py	(-1, +1)	| History	| Source	| Diff
· /tools/check_setup.sh	(-1, +1)	| History	| Source	| Diff
· /tools/create_dialin_link.sh	(-1, +1)	| History	| Source	| Diff
… 51 more files in changeset.

]]> Mon, 09 May 2022 09:27:16 -0700 Dara Navaei https://devapps.diality.us/changelog/testsuites?cs=25829e5793af41ffc76f0862a72d5fa510b14b19 https://devapps.diality.us/changelog/testsuites?cs=25829e5793af41ffc76f0862a72d5fa510b14b19 Dara Navaei committed 25829e5 on branch master_old in testsuites Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

· /shared/scripts/names.py	(-10, +13)	| History	| Source	| Diff
· /simulator/plugins/alarms/loader.py	(-0, +15)	| History	| Source	| Diff
· /simulator/plugins/createtreatment/loader.py	(-0, +15)	| History	| Source	| Diff
· /simulator/plugins/heparin/loader.py	(-0, +15)	| History	| Source	| Diff
· /simulator/plugins/inlinebloodpressures/loader.py	(-0, +15)	| History	| Source	| Diff
· /simulator/plugins/salinebolus/loader.py	(-0, +15)	| History	| Source	| Diff
· /simulator/plugins/treatmentranges/loader.py	(-0, +15)	| History	| Source	| Diff
· /simulator/plugins/treatmentstates/loader.py	(-0, +15)	| History	| Source	| Diff
· /simulator/plugins/ultrafiltration/loader.py	(-0, +15)	| History	| Source	| Diff
· /simulator/run.py	(-0, +15)	| History	| Source	| Diff
· /simulator/run.sh	(-0, +15)	| History	| Source	| Diff
· /simulator/simulator/dynamicloader.py	(-0, +15)	| History	| Source	| Diff
· /simulator/simulator/loader.py	(-0, +15)	| History	| Source	| Diff
· /tools/check_setup.sh	(-0, +15)	| History	| Source	| Diff
· /tools/create_dialin_link.sh	(-9, +8)	| History	| Source	| Diff
… 51 more files in changeset.

]]> Mon, 22 Feb 2021 14:47:46 -0800 Peter Lucia <plucia@diality.com> https://devapps.diality.us/changelog/testsuites?cs=cea6eac6a05f57e3317b45762a470eb18b314ab3 https://devapps.diality.us/changelog/testsuites?cs=cea6eac6a05f57e3317b45762a470eb18b314ab3 Peter Lucia <plucia@diality.com> committed cea6eac on branch master_old in testsuites DEN-7041: Fixed failing tests. Renamed HDSimulator to hd_sim

· /requirements.txt	(-1, +0)	| History	| Source	| Diff
· /simulator/run.py	(-1, +1)	| History	| Source	| Diff
· /simulator/simulator/loader.py	(-3, +3)	| History	| Source	| Diff
· /tst_AlarmStatusData/test.py	(-5, +5)	| History	| Source	| Diff
· /tst_AlarmTriggered/test.py	(-5, +5)	| History	| Source	| Diff
· /tst_CANBusFaultCount/test.py	(-5, +5)	| History	| Source	| Diff
· /tst_DGDrainPumpData/test.py	(-5, +5)	| History	| Source	| Diff
· /tst_DGHeatersData/test.py	(-5, +5)	| History	| Source	| Diff
· /tst_DGLoadCellReadingsData/test.py	(-5, +5)	| History	| Source	| Diff
· /tst_DGOperationMode/test.py	(-6, +6)	| History	| Source	| Diff
· /tst_DGPressureData/test.py	(-5, +5)	| History	| Source	| Diff
· /tst_DGROPumpData/test.py	(-5, +5)	| History	| Source	| Diff
· /tst_DGReservoirData/test.py	(-5, +5)	| History	| Source	| Diff
· /tst_DGTemperaturesData/test.py	(-5, +5)	| History	| Source	| Diff
· /tst_DGValvesStatesData/test.py	(-5, +5)	| History	| Source	| Diff
… 21 more files in changeset.

]]> Fri, 19 Feb 2021 18:24:14 -0800 Peter Lucia <plucia@diality.com> https://devapps.diality.us/changelog/testsuites?cs=ca79dc7f50fc912b05da91321d8652934aee0764 https://devapps.diality.us/changelog/testsuites?cs=ca79dc7f50fc912b05da91321d8652934aee0764 Peter Lucia <plucia@diality.com> committed ca79dc7 on branch master_old in testsuites DEN-7041: Fixed heparin and uf tests

· /tools/run_tests.sh	(-10, +20)	| History	| Source	| Diff
· /tst_DGROPumpData/test.py	(-3, +9)	| History	| Source	| Diff
· /tst_Treatment_Adjustment_Heparin/test.py	(-0, +1)	| History	| Source	| Diff
· /tst_Treatment_Adjustment_Ultrafiltration/test.py	(-51, +61)	| History	| Source	| Diff

]]> Mon, 15 Feb 2021 13:28:57 -0800 plucia https://devapps.diality.us/cru/UI-DEN-5638-1 https://devapps.diality.us/cru/UI-DEN-5638-1 plucia  (deleted user) Closed UI-DEN-5638-1 ]]> Mon, 15 Feb 2021 13:27:43 -0800 https://devapps.diality.us/cru/UI-DEN-5638-1 https://devapps.diality.us/cru/UI-DEN-5638-1 Behrouz NematiPour finished reviewing UI-DEN-5638-1 UI-DEN-5638_Squish Migration ]]> Mon, 15 Feb 2021 10:22:33 -0800 Behrouz NematiPour https://devapps.diality.us/cru/UI-DEN-5638-1/#c8107 https://devapps.diality.us/cru/UI-DEN-5638-1/#c8107 Behrouz NematiPour replied to plucia  (deleted user) in UI-DEN-5638-1

Still don't feel comfortable with this solution and haven't been convinced that it should be the way to get around the SquishQt issue of finding the object within the given timeout.
On the other hand, don't have a better idea to stop the test from failing randomly.
So would be better to keep it for now.

RESOLVED

Please change to TreatmentStates
RESOLVED

Nothing has changed about the usage of the SquishQt API, other than the fact that the dictionary passed to waitForObjectExists previously wasn't really testing anything.
Removing utils.dict_update it will break the test case.

I will discuss with Peman and get back to you.

I believe we discussed enough.
And it was discussed verbally before to not to do it.

Nothing has changed about the usage of the SquishQt API. We pass a dictionary to waitForObjectExists prior to this change and after.
Removing it will break the test case.

If nothing has changed so why your test would break.
Please change it to the recommended ways in SquishQt examples.

The way it was used is in SquishQt examples, but I can't find any example that SquishQt is being used the way you used it.
Please change it to the recommended ways in SquishQt examples.

I think you've misunderstood. The bug is not part of the SquishQt API. It is a problem with the way the SquishQt API is being used.

If there is a bug in SquishQt please report to Froglogic and give feedback and please don't try to fix it by yourself.
In case they fix the bug (if there is a bug) and they fix it, and the fix is not compatible with your fix, then we have self-generated bugs.
Please remove these and let Froglogic know you found a bug and let them help us.

RESOLVED

We are in agreement that each test should be self contained.
To ensure each test is self-contained, each test needs to have a separate instance of HDSimulator.
The design of dialin is not the focus of this review...

Tx_States violates the CapWords convention for classes as per our coding standard, as no underscores should be present between words in a class name.
TreatmentStates is clearer than TxStates. Tx could easily be confused with transmit...

I agree. To ensure I don't miss additional instances of HDSimulator I'll make this change on the latest testsuites branch I'm working on.

I don't mean to limit the HDSimulator to static functions, it can have static and normal methods.

1 - The test scripts are run in a consecutive manner, not in parallel. Each test should be self contained.

I agree in this case but the general implementation and design of HDSimulator lead to that misuse of the class.

Please consider changing the instance name to hd_simulator.
The name of variables is always important, especially in this case which makes it confusing since the name of the class and the variable are the same.

It has been defined as a class not a variable after the name change.
So the name should be "Tx_States"

RESOLVED

Limiting the HDSimulator to static functions prevents you from maintaining any state between successive calls and is incompatible with the DenaliCanMessenger. It is an improvement to hold state information and allows us to use the DenaliCanMessenger. Using the DenaliCanMessenger and dialin eliminates the need to maintain duplicated codebases and allows for bidirectional messaging. Your squish folder was doing the same thing as dialin but on a smaller scale.

1 - The test scripts are run in a consecutive manner, not in parallel. Each test should be self contained.
2 - (See statement above). This comment is about dialin, not testsuites.
3 - (See statement above). This comment is about dialin, not testsuites.