[v4,17/19] test_avb: Update pymark.buildconfigspec information for the AVB tests

Message ID 20180925144023.24555-18-jens.wiklander@linaro.org
State Accepted
Commit 07cbae767293dcf07cfc76cd494b8045ef4a8d7e
Headers show
Series
  • AVB using OP-TEE
Related show

Commit Message

Jens Wiklander Sept. 25, 2018, 2:40 p.m.
Update the pymark.buildconfigspec to depend on 'cmd_mmc' in addition to
'cmd_avb' for those tests that needs more a more complete MMC
implementation or the "mmc" command.

Reviewed-by: Simon Glass <sjg@chromium.org>

Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>

---
 test/py/tests/test_avb.py | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

-- 
2.17.1

Comments

Tom Rini Oct. 8, 2018, 1:41 a.m. | #1
On Tue, Sep 25, 2018 at 04:40:21PM +0200, Jens Wiklander wrote:

> Update the pymark.buildconfigspec to depend on 'cmd_mmc' in addition to

> 'cmd_avb' for those tests that needs more a more complete MMC

> implementation or the "mmc" command.

> 

> Reviewed-by: Simon Glass <sjg@chromium.org>

> Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>


Applied to u-boot/master, thanks!

-- 
Tom

Patch

diff --git a/test/py/tests/test_avb.py b/test/py/tests/test_avb.py
index 7996e4845db3..9683fd80d074 100644
--- a/test/py/tests/test_avb.py
+++ b/test/py/tests/test_avb.py
@@ -23,7 +23,7 @@  mmc_dev = 1
 temp_addr = 0x90000000
 temp_addr2 = 0x90002000
 
-@pytest.mark.buildconfigspec('cmd_avb')
+@pytest.mark.buildconfigspec('cmd_avb', 'cmd_mmc')
 def test_avb_verify(u_boot_console):
     """Run AVB 2.0 boot verification chain with avb subset of commands
     """
@@ -36,7 +36,7 @@  def test_avb_verify(u_boot_console):
     assert response.find(success_str)
 
 
-@pytest.mark.buildconfigspec('cmd_avb')
+@pytest.mark.buildconfigspec('cmd_avb', 'cmd_mmc')
 def test_avb_mmc_uuid(u_boot_console):
     """Check if 'avb get_uuid' works, compare results with
     'part list mmc 1' output
@@ -78,6 +78,7 @@  def test_avb_read_rb(u_boot_console):
     assert response == ''
 
     response = u_boot_console.run_command('avb read_rb 1')
+    assert response == 'Rollback index: 0'
 
 
 @pytest.mark.buildconfigspec('cmd_avb')
@@ -89,9 +90,10 @@  def test_avb_is_unlocked(u_boot_console):
     assert response == ''
 
     response = u_boot_console.run_command('avb is_unlocked')
+    assert response == 'Unlocked = 1'
 
 
-@pytest.mark.buildconfigspec('cmd_avb')
+@pytest.mark.buildconfigspec('cmd_avb', 'cmd_mmc')
 def test_avb_mmc_read(u_boot_console):
     """Test mmc read operation
     """