All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH v2] test: py: Add cmd_echo dependency
@ 2017-05-15 12:29 Michal Simek
  2017-05-15 21:49 ` Stephen Warren
  2017-05-22 17:58 ` [U-Boot] [U-Boot,v2] " Tom Rini
  0 siblings, 2 replies; 4+ messages in thread
From: Michal Simek @ 2017-05-15 12:29 UTC (permalink / raw)
  To: u-boot

There is missing dependency on echo command. Mark tests which requires
echo.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

Changes in v2:
- Removed incorrect dependency on echo command
- Add empty line before header and module import
- Mark the whole file with global pytestmark - all reported by Stephen

 test/py/tests/test_env.py          | 3 +++
 test/py/tests/test_shell_basics.py | 4 ++++
 2 files changed, 7 insertions(+)

diff --git a/test/py/tests/test_env.py b/test/py/tests/test_env.py
index 035dbf5cac4c..b7f960c755fe 100644
--- a/test/py/tests/test_env.py
+++ b/test/py/tests/test_env.py
@@ -164,6 +164,7 @@ def test_env_echo_exists(state_test_env):
     value = state_test_env.env[var]
     validate_set(state_test_env, var, value)
 
+ at pytest.mark.buildconfigspec('cmd_echo')
 def test_env_echo_non_existent(state_test_env):
     """Test echoing a variable that doesn't exist."""
 
@@ -179,6 +180,7 @@ def test_env_printenv_non_existent(state_test_env):
         response = c.run_command('printenv %s' % var)
     assert(response == '## Error: "%s" not defined' % var)
 
+ at pytest.mark.buildconfigspec('cmd_echo')
 def test_env_unset_non_existent(state_test_env):
     """Test unsetting a nonexistent variable."""
 
@@ -202,6 +204,7 @@ def test_env_set_existing(state_test_env):
     set_var(state_test_env, var, value)
     validate_set(state_test_env, var, value)
 
+ at pytest.mark.buildconfigspec('cmd_echo')
 def test_env_unset_existing(state_test_env):
     """Test unsetting a variable."""
 
diff --git a/test/py/tests/test_shell_basics.py b/test/py/tests/test_shell_basics.py
index 702e5e27e002..0024d5f7f619 100644
--- a/test/py/tests/test_shell_basics.py
+++ b/test/py/tests/test_shell_basics.py
@@ -4,6 +4,10 @@
 
 # Test basic shell functionality, such as commands separate by semi-colons.
 
+import pytest
+
+pytestmark = pytest.mark.buildconfigspec('cmd_echo')
+
 def test_shell_execute(u_boot_console):
     """Test any shell command."""
 
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH v2] test: py: Add cmd_echo dependency
  2017-05-15 12:29 [U-Boot] [PATCH v2] test: py: Add cmd_echo dependency Michal Simek
@ 2017-05-15 21:49 ` Stephen Warren
  2017-05-17  9:18   ` Michal Simek
  2017-05-22 17:58 ` [U-Boot] [U-Boot,v2] " Tom Rini
  1 sibling, 1 reply; 4+ messages in thread
From: Stephen Warren @ 2017-05-15 21:49 UTC (permalink / raw)
  To: u-boot

On 05/15/2017 06:29 AM, Michal Simek wrote:
> There is missing dependency on echo command. Mark tests which requires
> echo.

Reviewed-by: Stephen Warren <swarren@nvidia.com>

It'd be good to update test_hush_if_test.py too though, even if separately.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH v2] test: py: Add cmd_echo dependency
  2017-05-15 21:49 ` Stephen Warren
@ 2017-05-17  9:18   ` Michal Simek
  0 siblings, 0 replies; 4+ messages in thread
From: Michal Simek @ 2017-05-17  9:18 UTC (permalink / raw)
  To: u-boot

On 15.5.2017 23:49, Stephen Warren wrote:
> On 05/15/2017 06:29 AM, Michal Simek wrote:
>> There is missing dependency on echo command. Mark tests which requires
>> echo.
> 
> Reviewed-by: Stephen Warren <swarren@nvidia.com>
> 
> It'd be good to update test_hush_if_test.py too though, even if separately.

I have sent patch for that please review.

Thanks,
Michal

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [U-Boot,v2] test: py: Add cmd_echo dependency
  2017-05-15 12:29 [U-Boot] [PATCH v2] test: py: Add cmd_echo dependency Michal Simek
  2017-05-15 21:49 ` Stephen Warren
@ 2017-05-22 17:58 ` Tom Rini
  1 sibling, 0 replies; 4+ messages in thread
From: Tom Rini @ 2017-05-22 17:58 UTC (permalink / raw)
  To: u-boot

On Mon, May 15, 2017 at 02:29:02PM +0200, Michal Simek wrote:

> There is missing dependency on echo command. Mark tests which requires
> echo.
> 
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> Reviewed-by: Stephen Warren <swarren@nvidia.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170522/d51f1f56/attachment.sig>

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-05-22 17:58 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-15 12:29 [U-Boot] [PATCH v2] test: py: Add cmd_echo dependency Michal Simek
2017-05-15 21:49 ` Stephen Warren
2017-05-17  9:18   ` Michal Simek
2017-05-22 17:58 ` [U-Boot] [U-Boot,v2] " Tom Rini

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.