diff mbox series

test: Fix skip check for sleep command test

Message ID 20241008120813.622408-1-andrew.goodbody@linaro.org
State Accepted
Commit 2438203728e270d32fd87440992c2944d4ea4c29
Headers show
Series test: Fix skip check for sleep command test | expand

Commit Message

Andrew Goodbody Oct. 8, 2024, 12:08 p.m. UTC
When the config option CMD_MISC was renamed to CMD_SLEEP the check
in the test for the sleep command was not updated. Do that now.

Fixes: 16060854095 ("cmd: Rename CMD_MISC to CMD_SLEEP")
Signed-off-by: Andrew Goodbody <andrew.goodbody@linaro.org>
---

 test/py/tests/test_sleep.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Simon Glass Oct. 9, 2024, 1:57 a.m. UTC | #1
On Tue, 8 Oct 2024 at 06:09, Andrew Goodbody <andrew.goodbody@linaro.org> wrote:
>
> When the config option CMD_MISC was renamed to CMD_SLEEP the check
> in the test for the sleep command was not updated. Do that now.
>
> Fixes: 16060854095 ("cmd: Rename CMD_MISC to CMD_SLEEP")
> Signed-off-by: Andrew Goodbody <andrew.goodbody@linaro.org>
> ---
>
>  test/py/tests/test_sleep.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>

Reviewed-by: Simon Glass <sjg@chromium.org>
Tom Rini Oct. 15, 2024, 7:43 p.m. UTC | #2
On Tue, 08 Oct 2024 13:08:13 +0100, Andrew Goodbody wrote:

> When the config option CMD_MISC was renamed to CMD_SLEEP the check
> in the test for the sleep command was not updated. Do that now.
> 
> 

Applied to u-boot/master, thanks!
diff mbox series

Patch

diff --git a/test/py/tests/test_sleep.py b/test/py/tests/test_sleep.py
index 66a57434bf..8965fc3fea 100644
--- a/test/py/tests/test_sleep.py
+++ b/test/py/tests/test_sleep.py
@@ -27,7 +27,7 @@  def test_sleep(u_boot_console):
     if not sleep_skip:
         pytest.skip('sleep is not accurate')
 
-    if u_boot_console.config.buildconfig.get('config_cmd_misc', 'n') != 'y':
+    if u_boot_console.config.buildconfig.get('config_cmd_sleep', 'n') != 'y':
         pytest.skip('sleep command not supported')
 
     # 3s isn't too long, but is enough to cross a few second boundaries.
@@ -42,7 +42,7 @@  def test_sleep(u_boot_console):
         # margin is hopefully enough to account for any system overhead.
         assert elapsed < (sleep_time + sleep_margin)
 
-@pytest.mark.buildconfigspec("cmd_misc")
+@pytest.mark.buildconfigspec("cmd_time")
 def test_time(u_boot_console):
     """Test the time command, and validate that it gives approximately the
     correct amount of command execution time."""