revpi/revpimodio2
1
0 Fork 0
mirror of https://github.com/naruxde/revpimodio2.git synced 2026-09-30 23:04:29 +02:00
Code Issues Packages Releases Wiki Activity

test: Return -1 to indicate an error in get_switching_cycles method

Browse Source 
The old behavior was the return of the maximum value. However, this can
also be a valid value. Now all positive values are valid and a negative
value of -1 describes an error in the query.

Signed-off-by: Sven Sager <akira@narux.de>
This commit is contained in:
akira committed 2026-09-01 08:43:26 +02:00
1 parent f86527b5fd
commit 15b8d03772
3 files changed
+575
Download Patch File Download Diff File Expand all files Collapse all files

No files matched your search

tests/io_tests/config_ro_multi.rsc
+309
View File
@@ -0,0 +1,309 @@
{
"App": {
"name": "PiCtory",
"version": "2.17.3",
"saveTS": "20260901080641",
"language": "en",
"layout": {
"north": {
"size": 70,
"initClosed": false,
"initHidden": false
},
"south": {
"size": 353,
"initClosed": false,
"initHidden": false,
"children": {
"layout1": {
"east": {
"size": 500,
"initClosed": false,
"initHidden": false
}
}
}
},
"east": {
"size": 70,
"initClosed": true,
"initHidden": false,
"children": {}
},
"west": {
"size": 340,
"initClosed": false,
"initHidden": false,
"children": {
"layout1": {}
}
}
}
},
"Summary": {
"inpTotal": 7,
"outTotal": 8
},
"Devices": [
{
"GUID": "fa3b69a8-77f1-7f15-d9f0-339646253ace",
"id": "device_RevPiConnect5_20240315_1_0_001",
"type": "BASE",
"productType": "138",
"position": "0",
"name": "RevPi Connect 5",
"bmk": "RevPi Connect 5",
"inpVariant": 0,
"outVariant": 0,
"comment": "This is a RevPi Connect 5 Device",
"offset": 0,
"inp": {
"0": [
"RevPiStatus",
"0",
"8",
"0",
true,
"0000",
"",
""
],
"1": [
"RevPiIOCycle",
"0",
"8",
"1",
true,
"0001",
"",
""
],
"2": [
"RS485ErrorCnt",
"0",
"16",
"2",
false,
"0002",
"",
""
],
"3": [
"Core_Temperature",
"0",
"8",
"4",
false,
"0003",
"",
""
],
"4": [
"Core_Frequency",
"0",
"8",
"5",
false,
"0004",
"",
""
]
},
"out": {
"0": [
"RevPiReservedByte",
"0",
"8",
"6",
false,
"0005",
"",
""
],
"1": [
"RS485ErrorLimit1",
"10",
"16",
"7",
false,
"0006",
"",
""
],
"2": [
"RS485ErrorLimit2",
"1000",
"16",
"9",
false,
"0007",
"",
""
],
"3": [
"RevPiLED",
"0",
"16",
"11",
true,
"0008",
"",
""
]
},
"mem": {},
"extend": {}
},
{
"GUID": "53c792e0-ec3d-8047-bb0c-1afa6772873f",
"id": "device_RevPiRO_20231018_1_0_001",
"type": "LEFT_RIGHT",
"productType": "137",
"position": "32",
"name": "RevPi RO",
"bmk": "RevPi RO",
"inpVariant": 0,
"outVariant": 0,
"comment": "",
"offset": 13,
"inp": {
"0": [
"Status",
"0",
"8",
"0",
false,
"0000",
"",
""
]
},
"out": {
"0": [
"RelayOutput_1",
"0",
"1",
"1",
true,
"0001",
"",
"0"
],
"1": [
"RelayOutput_2",
"0",
"1",
"1",
true,
"0002",
"",
"1"
],
"2": [
"RelayOutput_3",
"0",
"1",
"1",
true,
"0003",
"",
"2"
],
"3": [
"RelayOutput_4",
"0",
"1",
"1",
true,
"0004",
"",
"3"
],
"4": [
"RelayOutputPadding_5",
"0",
"1",
"1",
true,
"0005",
"",
"4"
],
"5": [
"RelayOutputPadding_6",
"0",
"1",
"1",
true,
"0006",
"",
"5"
],
"6": [
"RelayOutputPadding_7",
"0",
"1",
"1",
true,
"0007",
"",
"6"
],
"7": [
"RelayOutputPadding_8",
"0",
"1",
"1",
true,
"0008",
"",
"7"
]
},
"mem": {
"0": [
"RelayCycleWarningThreshold_1",
"0",
"32",
"2",
false,
"0009",
"",
""
],
"1": [
"RelayCycleWarningThreshold_2",
"0",
"32",
"6",
false,
"0010",
"",
""
],
"2": [
"RelayCycleWarningThreshold_3",
"0",
"32",
"10",
false,
"0011",
"",
""
],
"3": [
"RelayCycleWarningThreshold_4",
"0",
"32",
"14",
false,
"0012",
"",
""
]
},
"extend": {}
}
],
"Connections": []
}
tests/io_tests/config_ro_single.rsc
+239
View File
@@ -0,0 +1,239 @@
{
"App": {
"name": "PiCtory",
"version": "2.17.3",
"saveTS": "20260901080758",
"language": "en",
"layout": {
"north": {
"size": 70,
"initClosed": false,
"initHidden": false
},
"south": {
"size": 353,
"initClosed": false,
"initHidden": false,
"children": {
"layout1": {
"east": {
"size": 500,
"initClosed": false,
"initHidden": false
}
}
}
},
"east": {
"size": 70,
"initClosed": true,
"initHidden": false,
"children": {}
},
"west": {
"size": 340,
"initClosed": false,
"initHidden": false,
"children": {
"layout1": {}
}
}
}
},
"Summary": {
"inpTotal": 7,
"outTotal": 8
},
"Devices": [
{
"GUID": "fa3b69a8-77f1-7f15-d9f0-339646253ace",
"id": "device_RevPiConnect5_20240315_1_0_001",
"type": "BASE",
"productType": "138",
"position": "0",
"name": "RevPi Connect 5",
"bmk": "RevPi Connect 5",
"inpVariant": 0,
"outVariant": 0,
"comment": "This is a RevPi Connect 5 Device",
"offset": 0,
"inp": {
"0": [
"RevPiStatus",
"0",
"8",
"0",
true,
"0000",
"",
""
],
"1": [
"RevPiIOCycle",
"0",
"8",
"1",
true,
"0001",
"",
""
],
"2": [
"RS485ErrorCnt",
"0",
"16",
"2",
false,
"0002",
"",
""
],
"3": [
"Core_Temperature",
"0",
"8",
"4",
false,
"0003",
"",
""
],
"4": [
"Core_Frequency",
"0",
"8",
"5",
false,
"0004",
"",
""
]
},
"out": {
"0": [
"RevPiReservedByte",
"0",
"8",
"6",
false,
"0005",
"",
""
],
"1": [
"RS485ErrorLimit1",
"10",
"16",
"7",
false,
"0006",
"",
""
],
"2": [
"RS485ErrorLimit2",
"1000",
"16",
"9",
false,
"0007",
"",
""
],
"3": [
"RevPiLED",
"0",
"16",
"11",
true,
"0008",
"",
""
]
},
"mem": {},
"extend": {}
},
{
"GUID": "53c792e0-ec3d-8047-bb0c-1afa6772873f",
"id": "device_RevPiRO_20231018_1_0_001",
"type": "LEFT_RIGHT",
"productType": "137",
"position": "32",
"name": "RevPi RO",
"bmk": "RevPi RO",
"inpVariant": 0,
"outVariant": 1,
"comment": "",
"offset": 13,
"inp": {
"0": [
"Status_i02",
"0",
"8",
"0",
false,
"0000",
"",
""
]
},
"out": {
"0": [
"RelayOutputs",
"0",
"8",
"1",
true,
"0001",
"",
""
]
},
"mem": {
"0": [
"RelayCycleWarningThreshold_1_i02",
"0",
"32",
"2",
false,
"0002",
"",
""
],
"1": [
"RelayCycleWarningThreshold_2_i02",
"0",
"32",
"6",
false,
"0003",
"",
""
],
"2": [
"RelayCycleWarningThreshold_3_i02",
"0",
"32",
"10",
false,
"0004",
"",
""
],
"3": [
"RelayCycleWarningThreshold_4_i02",
"0",
"32",
"14",
false,
"0005",
"",
""
]
},
"extend": {}
}
],
"Connections": []
}
tests/io_tests/test_ios.py
+27
View File
@@ -105,3 +105,30 @@ class TestIos(TestRevPiModIO):
self.assertEqual(rpi.device[65]._ba_devdata[32:38], b"\x20\x00\x00\x00\x00\x00") self.assertEqual(rpi.device[65]._ba_devdata[32:38], b"\x20\x00\x00\x00\x00\x00")
rpi.io.OutBit_48.value = True rpi.io.OutBit_48.value = True
self.assertEqual(rpi.device[65]._ba_devdata[32:38], b"\x20\x00\x00\x00\x00\x80") self.assertEqual(rpi.device[65]._ba_devdata[32:38], b"\x20\x00\x00\x00\x00\x80")
def test_get_switching_cycles(self):
rpi = self.modio(configrsc="config_ro_multi.rsc")
self.assertEqual(rpi.io.RelayOutput_1.get_switching_cycles(), 0)
self.assertEqual(rpi.io.RelayOutput_2.get_switching_cycles(), 0)
self.assertEqual(rpi.io.RelayOutput_3.get_switching_cycles(), 0)
self.assertEqual(rpi.io.RelayOutput_4.get_switching_cycles(), 0)
# This will use ioctl calls
rpi._run_on_pi = True
with self.assertWarnsRegex(RuntimeWarning, r"'rocounter' and count \d"):
self.assertEqual(rpi.io.RelayOutput_1.get_switching_cycles(), -1)
self.assertEqual(rpi.io.RelayOutput_2.get_switching_cycles(), -1)
self.assertEqual(rpi.io.RelayOutput_3.get_switching_cycles(), -1)
self.assertEqual(rpi.io.RelayOutput_4.get_switching_cycles(), -1)
self.assertEqual(rpi.ioerrors, 4)
rpi = self.modio(configrsc="config_ro_single.rsc")
self.assertEqual(rpi.io.RelayOutputs.get_switching_cycles(), (0, 0, 0, 0))
# This will use ioctl calls
rpi._run_on_pi = True
with self.assertWarnsRegex(RuntimeWarning, r"'rocounter' and count \d"):
self.assertEqual(rpi.io.RelayOutputs.get_switching_cycles(), (-1, -1, -1, -1))
self.assertEqual(rpi.ioerrors, 1)