west: runner: rfp: support update test_rfp.py file

This commit to support update test_rfp.py file

Signed-off-by: Hau Ho <hau.ho.xc@bp.renesas.com>

Author: Ta Minh Nhat

scripts/west_commands/tests/test_rfp.py

@@ -11,8 +11,11 @@
 
 TEST_RFP_PORT = 'test-rfp-serial'
 TEST_RFP_PORT_SPEED = '115200'
+TEST_RFP_TOOL = 'jlink'
+TEST_RFP_INTERFACE = 'swd'
 TEST_RFP_DEVICE = 'RA'
 TEST_RFP_USR_LOCAL_RFP_CLI = '/usr/local/bin/rfp-cli'
+TEST_RFP_RPD_FILE = 'test-zephyr.rpd'
 
 EXPECTED_COMMANDS = [
     [
@@ -93,23 +96,55 @@
     ],
 ]
 
+EXPECTED_COMMANDS_WITH_JLINK_HARDWARE = [
+    [
+        TEST_RFP_USR_LOCAL_RFP_CLI,
+        '-tool',
+        TEST_RFP_TOOL,
+        '-interface',
+        TEST_RFP_INTERFACE,
+        '-device',
+        TEST_RFP_DEVICE,
+        '-run',
+        '-erase',
+        '-p',
+        '-file',
+        RC_KERNEL_HEX,
+    ],
+]
+
+EXPECTED_COMMANDS_WITH_PARTITION_DATA = [
+    [
+        TEST_RFP_USR_LOCAL_RFP_CLI,
+        '-device',
+        TEST_RFP_DEVICE,
+        '-tool',
+        TEST_RFP_TOOL,
+        '-fo',
+        'boundary-file',
+        TEST_RFP_RPD_FILE,
+        '-p',
+    ],
+]
+
 
 def require_patch(program):
-    assert program in ['rfp']
+    assert program in ['rfp', 'rfp-cli', TEST_RFP_USR_LOCAL_RFP_CLI]
 
 
 os_path_isfile = os.path.isfile
 
 
 def os_path_isfile_patch(filename):
-    if filename == RC_KERNEL_HEX:
+    if filename == RC_KERNEL_HEX or TEST_RFP_RPD_FILE:
         return True
     return os_path_isfile(filename)
 
 
 @patch('runners.core.ZephyrBinaryRunner.require', side_effect=require_patch)
 @patch('runners.core.ZephyrBinaryRunner.check_call')
-def test_rfp_init(cc, req, runner_config, tmpdir):
+@patch('runners.rfp.RfpBinaryRunner.default_rfp')
+def test_rfp_init(dr, cc, req, runner_config, tmpdir):
     """
     Test commands using a runner created by constructor.
 
@@ -129,7 +164,8 @@ def test_rfp_init(cc, req, runner_config, tmpdir):
 
 @patch('runners.core.ZephyrBinaryRunner.require', side_effect=require_patch)
 @patch('runners.core.ZephyrBinaryRunner.check_call')
-def test_rfp_create(cc, req, runner_config, tmpdir):
+@patch('runners.rfp.RfpBinaryRunner.default_rfp')
+def test_rfp_create(dr, cc, req, runner_config, tmpdir):
     """
     Test commands using a runner created from command line parameters.
 
@@ -154,7 +190,8 @@ def test_rfp_create(cc, req, runner_config, tmpdir):
 
 @patch('runners.core.ZephyrBinaryRunner.require', side_effect=require_patch)
 @patch('runners.core.ZephyrBinaryRunner.check_call')
-def test_rfp_create_with_speed(cc, req, runner_config, tmpdir):
+@patch('runners.rfp.RfpBinaryRunner.default_rfp')
+def test_rfp_create_with_speed(dr, cc, req, runner_config, tmpdir):
     """
     Test commands using a runner created from command line parameters.
 
@@ -185,7 +222,8 @@ def test_rfp_create_with_speed(cc, req, runner_config, tmpdir):
 
 @patch('runners.core.ZephyrBinaryRunner.require', side_effect=require_patch)
 @patch('runners.core.ZephyrBinaryRunner.check_call')
-def test_rfp_create_with_erase(cc, req, runner_config, tmpdir):
+@patch('runners.rfp.RfpBinaryRunner.default_rfp')
+def test_rfp_create_with_erase(dr, cc, req, runner_config, tmpdir):
     """
     Test commands using a runner created from command line parameters.
 
@@ -207,7 +245,8 @@ def test_rfp_create_with_erase(cc, req, runner_config, tmpdir):
 
 @patch('runners.core.ZephyrBinaryRunner.require', side_effect=require_patch)
 @patch('runners.core.ZephyrBinaryRunner.check_call')
-def test_rfp_create_with_verify(cc, req, runner_config, tmpdir):
+@patch('runners.rfp.RfpBinaryRunner.default_rfp')
+def test_rfp_create_with_verify(dr, cc, req, runner_config, tmpdir):
     """
     Test commands using a runner created from command line parameters.
 
@@ -229,7 +268,8 @@ def test_rfp_create_with_verify(cc, req, runner_config, tmpdir):
 
 @patch('runners.core.ZephyrBinaryRunner.require', side_effect=require_patch)
 @patch('runners.core.ZephyrBinaryRunner.check_call')
-def test_rfp_create_with_rfp_cli(cc, req, runner_config, tmpdir):
+@patch('runners.rfp.RfpBinaryRunner.default_rfp')
+def test_rfp_create_with_rfp_cli(dr, cc, req, runner_config, tmpdir):
     """
     Test commands using a runner created from command line parameters.
 
@@ -254,3 +294,74 @@ def test_rfp_create_with_rfp_cli(cc, req, runner_config, tmpdir):
     with patch('os.path.isfile', side_effect=os_path_isfile_patch):
         runner.run('flash')
     assert cc.call_args_list == [call(x) for x in EXPECTED_COMMANDS_WITH_RFP_CLI]
+
+
+@patch('runners.core.ZephyrBinaryRunner.require', side_effect=require_patch)
+@patch('runners.core.ZephyrBinaryRunner.check_call')
+@patch('runners.rfp.RfpBinaryRunner.default_rfp')
+def test_rfp_create_with_jlink_hardware(dr, cc, req, runner_config, tmpdir):
+    """
+    Test commands using a jlink hardware.
+
+    Input:
+        --rfp-cli /usr/local/bin/rfp-cli
+
+    Output:
+        /usr/local/bin/rfp-cli
+    """
+    args = [
+        '--device',
+        str(TEST_RFP_DEVICE),
+        '--tool',
+        str(TEST_RFP_TOOL),
+        '--interface',
+        str(TEST_RFP_INTERFACE),
+        '--erase',
+        '--rfp-cli',
+        str(TEST_RFP_USR_LOCAL_RFP_CLI),
+    ]
+    parser = argparse.ArgumentParser(allow_abbrev=False)
+    RfpBinaryRunner.add_parser(parser)
+    arg_namespace = parser.parse_args(args)
+    runner = RfpBinaryRunner.create(runner_config, arg_namespace)
+    with patch('os.path.isfile', side_effect=os_path_isfile_patch):
+        runner.run('flash')
+    assert cc.call_args_list == [call(x) for x in EXPECTED_COMMANDS_WITH_JLINK_HARDWARE]
+
+
+@patch('runners.core.ZephyrBinaryRunner.require', side_effect=require_patch)
+@patch('runners.core.ZephyrBinaryRunner.check_call')
+@patch('runners.rfp.RfpBinaryRunner.default_rfp')
+def test_rfp_create_with_partition_data(dr, cc, req, runner_config, tmpdir):
+    """
+    Test commands using reresas partition data.
+
+    Input:
+        --rfp-cli /usr/local/bin/rfp-cli
+
+    Output:
+        /usr/local/bin/rfp-cli
+    """
+    args = [
+        '--device',
+        str(TEST_RFP_DEVICE),
+        '--tool',
+        str(TEST_RFP_TOOL),
+        '--interface',
+        str(TEST_RFP_INTERFACE),
+        '--rpd-file',
+        str(TEST_RFP_RPD_FILE),
+        '--erase',
+        '--rfp-cli',
+        str(TEST_RFP_USR_LOCAL_RFP_CLI),
+    ]
+    parser = argparse.ArgumentParser(allow_abbrev=False)
+    RfpBinaryRunner.add_parser(parser)
+    arg_namespace = parser.parse_args(args)
+    print(runner_config)
+    runner = RfpBinaryRunner.create(runner_config, arg_namespace)
+    with patch('os.path.isfile', side_effect=os_path_isfile_patch):
+        runner.run('flash')
+    print(cc.call_args_list)
+    assert [cc.call_args_list[0]] == [call(x) for x in EXPECTED_COMMANDS_WITH_PARTITION_DATA]
+    assert [cc.call_args_list[1]] == [call(x) for x in EXPECTED_COMMANDS_WITH_JLINK_HARDWARE]