Re: [U-Boot] [PATCH 3/3] test/py: add a test for the sleep command

[Simon Glass]

Hi Stephen,

On 15 January 2016 at 11:24, Stephen Warren  wrote:
> On 01/06/2016 10:33 AM, Stephen Warren wrote:
>>
>> From: Stephen Warren 
>>
>> Execute "sleep", and validate that it sleeps for approximately the correct
>> amount of time.
>
>
> FYI, I've reposted the series that implements the test/py code, and bundled
> this patch 3/3 in this series into that series.
>
> As such, I believe patches 1/3 and 2/3 in this series can be applied through
> whatever path sandbox (or DM or timer) patches would normally go through.

I've applied them to u-boot-dm.

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 3/3] test/py: add a test for the sleep command

[Stephen Warren]

On 01/06/2016 10:33 AM, Stephen Warren wrote:

From: Stephen Warren 

Execute "sleep", and validate that it sleeps for approximately the correct
amount of time.


FYI, I've reposted the series that implements the test/py code, and 
bundled this patch 3/3 in this series into that series.


As such, I believe patches 1/3 and 2/3 in this series can be applied 
through whatever path sandbox (or DM or timer) patches would normally go 
through.

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 3/3] test/py: add a test for the sleep command

[Simon Glass]

On 6 January 2016 at 10:33, Stephen Warren  wrote:
> From: Stephen Warren 
>
> Execute "sleep", and validate that it sleeps for approximately the correct
> amount of time.
>
> Signed-off-by: Stephen Warren 
> ---
> This patch depends on both the two previous patches, and on my unrelated
> series which adds the test/py code. If I resend that series, I can add
> this patch into it.
> ---
>  test/py/tests/test_sleep.py | 24 
>  1 file changed, 24 insertions(+)
>  create mode 100644 test/py/tests/test_sleep.py

Acked-by: Simon Glass 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 3/3] test/py: add a test for the sleep command

[Stephen Warren]

From: Stephen Warren 

Execute "sleep", and validate that it sleeps for approximately the correct
amount of time.

Signed-off-by: Stephen Warren 
---
This patch depends on both the two previous patches, and on my unrelated
series which adds the test/py code. If I resend that series, I can add
this patch into it.
---
 test/py/tests/test_sleep.py | 24 
 1 file changed, 24 insertions(+)
 create mode 100644 test/py/tests/test_sleep.py

diff --git a/test/py/tests/test_sleep.py b/test/py/tests/test_sleep.py
new file mode 100644
index ..64f1ddf9a09f
--- /dev/null
+++ b/test/py/tests/test_sleep.py
@@ -0,0 +1,24 @@
+# Copyright (c) 2016, NVIDIA CORPORATION. All rights reserved.
+#
+# SPDX-License-Identifier: GPL-2.0
+
+import pytest
+import time
+
+def test_sleep(u_boot_console):
+'''Test the sleep command, and validate that it sleeps for approximately
+the correct amount of time.'''
+
+# Do this before we time anything, to make sure U-Boot is already running.
+# Otherwise, the system boot time is included in the time measurement.
+u_boot_console.ensure_spawned()
+
+# 3s isn't too long, but is enough to cross a few second boundaries.
+sleep_time = 3
+tstart = time.time()
+u_boot_console.run_command('sleep %d' % sleep_time)
+tend = time.time()
+elapsed = tend - tstart
+delta_to_expected = abs(elapsed - sleep_time)
+# 0.25s margin is hopefully enough to account for any system overhead.
+assert delta_to_expected < 0.25
-- 
2.6.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot