[riot-notifications] [RIOT-OS/RIOT] tests/periph_rtc: add automatic python test script (#10812)

Leandro Lanzieri notifications at github.com
Fri Jan 18 14:38:13 CET 2019


leandrolanzieri commented on this pull request.



> +        # clock values
+        assert clock_set == clock_value
+
+    child.expect(r'  Setting alarm to   ({})'.format(DATE_PATTERN))
+    alarm_set = child.match.group(1)
+    if BOARD == 'native':
+        child.expect(r'.*rtc_set_alarm: not implemented')
+    child.expect(r'   Alarm is set to   ({})'.format(DATE_PATTERN))
+    alarm_value = child.match.group(1)
+    if BOARD != 'native':
+        # Set alarm is not implemented for native board so no need to compare
+        # alarm values
+        assert alarm_set == alarm_value
+
+    if BOARD != 'native':
+        for _ in range(alarm_count):

```suggestion
        for _ in range(int(alarm_count)):
```
Here should be int

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/10812#pullrequestreview-194097922
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190118/5645e6d3/attachment-0001.html>


More information about the notifications mailing list