michaelh/zephyr
1
0
Fork 0
Code Releases Activity

scripts: test runners: stm32cubeprogrammer: Update --download-address test

Browse source 
Fix --download-address to expect a signed.bin file when --download-address
is provided.

To allow testing this case despite the checks put in the runner, add a
patch functions which return that zephyr.bin is actually present

Signed-off-by: Erwan Gouriou <erwan.gouriou@st.com>
This commit is contained in:
Erwan Gouriou 2025-02-07 16:34:18 +01:00 • committed by Benjamin Cabé
commit 3bddb13a29
1 changed files with 13 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

16
scripts/west_commands/tests/test_stm32cubeprogrammer.py
View file

@ -3,14 +3,14 @@
# SPDX-License-Identifier: Apache-2.0 # SPDX-License-Identifier: Apache-2.0
import argparse import argparse
import os
from pathlib import Path from pathlib import Path
from unittest.mock import patch, call from unittest.mock import patch, call
import pytest import pytest
from runners.stm32cubeprogrammer import STM32CubeProgrammerBinaryRunner from runners.stm32cubeprogrammer import STM32CubeProgrammerBinaryRunner
from conftest import RC_KERNEL_HEX, RC_KERNEL_ELF from conftest import RC_KERNEL_HEX, RC_KERNEL_ELF, RC_KERNEL_BIN
CLI_PATH = Path("STM32_Programmer_CLI") CLI_PATH = Path("STM32_Programmer_CLI")
"""Default CLI path used in tests.""" """Default CLI path used in tests."""
@ -436,7 +436,7 @@ TEST_CASES = (
"--connect", "--connect",
"port=swd", "port=swd",
"--download", "--download",
RC_KERNEL_HEX, RC_KERNEL_BIN,
"0x80000000", "0x80000000",
"0x1", "0x1",
"--start", "--start",
@ -447,6 +447,12 @@ TEST_CASES = (
) )
"""Test cases.""" """Test cases."""
os_path_isfile = os.path.isfile
def os_path_isfile_patch(filename):
if filename == RC_KERNEL_BIN:
return True
return os_path_isfile(filename)
@pytest.mark.parametrize("tc", TEST_CASES) @pytest.mark.parametrize("tc", TEST_CASES)
@patch("runners.stm32cubeprogrammer.platform.system") @patch("runners.stm32cubeprogrammer.platform.system")
@ -455,7 +461,9 @@ TEST_CASES = (
@patch.dict("runners.stm32cubeprogrammer.os.environ", ENVIRON) @patch.dict("runners.stm32cubeprogrammer.os.environ", ENVIRON)
@patch("runners.core.ZephyrBinaryRunner.require") @patch("runners.core.ZephyrBinaryRunner.require")
@patch("runners.stm32cubeprogrammer.STM32CubeProgrammerBinaryRunner.check_call") @patch("runners.stm32cubeprogrammer.STM32CubeProgrammerBinaryRunner.check_call")
@patch("os.path.isfile", side_effect=os_path_isfile_patch)
def test_stm32cubeprogrammer_init( def test_stm32cubeprogrammer_init(
os_path_isfile_patch,
check_call, require, path_exists, path_home, system, tc, runner_config check_call, require, path_exists, path_home, system, tc, runner_config
): ):
"""Tests that ``STM32CubeProgrammerBinaryRunner`` class can be initialized """Tests that ``STM32CubeProgrammerBinaryRunner`` class can be initialized
@ -494,7 +502,9 @@ def test_stm32cubeprogrammer_init(
@patch.dict("runners.stm32cubeprogrammer.os.environ", ENVIRON) @patch.dict("runners.stm32cubeprogrammer.os.environ", ENVIRON)
@patch("runners.core.ZephyrBinaryRunner.require") @patch("runners.core.ZephyrBinaryRunner.require")
@patch("runners.stm32cubeprogrammer.STM32CubeProgrammerBinaryRunner.check_call") @patch("runners.stm32cubeprogrammer.STM32CubeProgrammerBinaryRunner.check_call")
@patch("os.path.isfile", side_effect=os_path_isfile_patch)
def test_stm32cubeprogrammer_create( def test_stm32cubeprogrammer_create(
os_path_isfile_patch,
check_call, require, path_exists, path_home, system, tc, runner_config check_call, require, path_exists, path_home, system, tc, runner_config
): ):
"""Tests that ``STM32CubeProgrammerBinaryRunner`` class can be created using """Tests that ``STM32CubeProgrammerBinaryRunner`` class can be created using