Hi Love,
On Tue, Dec 5, 2023 at 12:03 PM Love Kumar wrote:
>
> Add below test cases for mii commands:
> mii_info -To display MII PHY info
> mii_list - To list MII devices
> mii_set_device - To set MII device
> mii_read - To reads register from MII PHY address
> mii_dump - To display data from MII PHY address
>
> Signed-off-by: Love Kumar
> ---
> Changes in v2:
> - Get MII device list from env instead of auto-detecting it
> - Set the MII device to its current device after testing so that it
> won't impact next cases
> ---
> test/py/tests/test_mii.py | 92 +++
> 1 file changed, 92 insertions(+)
> create mode 100644 test/py/tests/test_mii.py
I'll just note that we can write tests in C now, even if they need
Python setup. See [1].
Some nits below, with which:
Reviewed-by: Simon Glass
>
> diff --git a/test/py/tests/test_mii.py b/test/py/tests/test_mii.py
> new file mode 100644
> index ..e9bdbadffc8e
> --- /dev/null
> +++ b/test/py/tests/test_mii.py
> @@ -0,0 +1,92 @@
> +# SPDX-License-Identifier: GPL-2.0
> +# (C) Copyright 2023, Advanced Micro Devices, Inc.
> +
> +import pytest
> +import re
> +
> +"""
> +Note: This test doesn't rely on boardenv_* configuration value but they can
> +change test behavior.
> +
> +For example:
> +
> +# Setup env__mii_deive_test_skip to True if tests with ethernet PHY devices
> +# should be skipped. For example: Missing PHY device
> +env__mii_device_test_skip = True
> +
> +# Setup env__mii_device_test to set the MII device names. Test will be
> skipped
> +# if env_mii_device_test is not set
> +env__mii_device_test = {
> +"device_list": ["eth0", "eth1"],
Please be sure to use single quotes throughout, for consistency.
> +}
> +"""
> +
> +@pytest.mark.buildconfigspec("cmd_mii")
> +def test_mii_info(u_boot_console):
> +if u_boot_console.config.env.get("env__mii_device_test_skip", False):
> +pytest.skip("MII device test is not enabled!")
> +expected_output = "PHY"
> +output = u_boot_console.run_command("mii info")
> +if not re.search(r"PHY (.+?):", output):
> +pytest.skip("PHY device does not exist!")
> +assert expected_output in output
> +
> +@pytest.mark.buildconfigspec("cmd_mii")
> +def test_mii_list(u_boot_console):
> +if u_boot_console.config.env.get("env__mii_device_test_skip", False):
> +pytest.skip("MII device test is not enabled!")
> +
> +f = u_boot_console.config.env.get("env__mii_device_test", None)
> +if not f:
> +pytest.skip("No MII device to test!")
> +
> +dev_list = f.get("device_list")
> +if len(dev_list) == 0:
if not dev_list:
> +pytest.fail("No MII device list provided via env__mii_device_test!")
> +
> +expected_output = "Current device"
> +output = u_boot_console.run_command("mii device")
> +mii_devices = (
> +re.search(r"MII devices: '(.+)'", output).groups()[0].replace("'",
> "").split()
> +)
> +
> +assert len([x for x in dev_list if x in mii_devices]) == len(dev_list)
> +assert expected_output in output
> +
> +@pytest.mark.buildconfigspec("cmd_mii")
> +def test_mii_set_device(u_boot_console):
> +test_mii_list(u_boot_console)
> +f = u_boot_console.config.env.get("env__mii_device_test", None)
> +dev_list = f.get("device_list")
> +output = u_boot_console.run_command("mii device")
> +current_dev = re.search(r"Current device: '(.+?)'", output).groups()[0]
> +
> +for dev in dev_list:
> +u_boot_console.run_command(f"mii device {dev}")
> +output = u_boot_console.run_command("echo $?")
> +assert output.endswith("0")
> +
> +u_boot_console.run_command(f"mii device {current_dev}")
> +output = u_boot_console.run_command("mii device")
> +dev = re.search(r"Current device: '(.+?)'", output).groups()[0]
> +assert current_dev == dev
> +
> +@pytest.mark.buildconfigspec("cmd_mii")
> +def test_mii_read(u_boot_console):
> +test_mii_list(u_boot_console)
> +output = u_boot_console.run_command("mii info")
> +eth_addr = hex(int(re.search(r"PHY (.+?):", output).groups()[0], 16))
> +u_boot_console.run_command(f"mii read {eth_addr} 0")
> +output = u_boot_console.run_command("echo $?")
> +assert output.endswith("0")
> +
> +@pytest.mark.buildconfigspec("cmd_mii")
> +def test_mii_dump(u_boot_console):
> +test_mii_list(u_boot_console)
> +expected_response = "PHY control register"
> +output = u_boot_console.run_command("mii info")
> +eth_addr = hex(int(re.search(r"PHY (.+?):", output).groups()[0], 16))
> +response = u_boot_console.run_command(f"mii dump {eth_addr} 0")
> +assert expected_response in response
> +output = u_boot_console.run_command("echo $?")
> +assert output.endswith("0")
> --
> 2.25.1
>
Regards,
Simon
[1]
https://docs.u-boot.org/en/latest/develop/tests_writing.html#mixing-python-and-c