Re: [Buildroot] [PATCH v4 RESEND 3/3] support/testing: add snappy test

[Flaviu Nistor <flaviu.nistor@gmail.com>]

On Sun, Oct 11, 2026 at 1:14 PM, Peter Korsgaard wrote:
>> Add a simple test case that imports the module and does a simple compress
>> decompress routine.
>
>> Signed-off-by: Flaviu Nistor <flaviu.nistor@gmail.com>
>> ---
>>  DEVELOPERS                                    |  2 ++
>>  .../tests/package/sample_python_snappy.py     |  7 ++++
>>  .../tests/package/test_python_snappy.py       | 34 +++++++++++++++++++
>>  3 files changed, 43 insertions(+)
>>  create mode 100644 support/testing/tests/package/sample_python_snappy.py
>>  create mode 100644 support/testing/tests/package/test_python_snappy.py
>
>> diff --git a/DEVELOPERS b/DEVELOPERS
>> index dbe05b7806..153b2f4af5 100644
>> --- a/DEVELOPERS
>> +++ b/DEVELOPERS
>> @@ -1140,6 +1140,8 @@ F:=09package/python-cramjam/
>>  F:=09package/python-rich/
>>  F:=09package/python-smmap2/
>>  F:=09package/python-snappy/
>> +F:=09support/testing/tests/package/sample_python_snappy.py
>> +F:=09support/testing/tests/package/test_python_snappy.py
>=20
>>  N:=09Fl=C3=A1vio Tapaj=C3=B3s <flavio.tapajos@newtesc.com.br>
>>  F:=09configs/asus_tinker-s_rk3288_defconfig
>> diff --git a/support/testing/tests/package/sample_python_snappy.py b/sup=
>port/testing/tests/package/sample_python_snappy.py
>> new file mode 100644
>> index 0000000000..ff2c07c05a
>> --- /dev/null
>> +++ b/support/testing/tests/package/sample_python_snappy.py
>> @@ -0,0 +1,7 @@
>> +import snappy
>> +
>> +payload =3D b"Buildroot python-snappy runtime test\n"
>> +compressed =3D snappy.compress(payload)
>> +decompressed =3D snappy.uncompress(compressed)
>> +
>> +assert decompressed =3D=3D payload
>> diff --git a/support/testing/tests/package/test_python_snappy.py b/suppo=
>rt/testing/tests/package/test_python_snappy.py
>> new file mode 100644
>> index 0000000000..8c0092bd23
>> --- /dev/null
>> +++ b/support/testing/tests/package/test_python_snappy.py
>> @@ -0,0 +1,34 @@
>> +import os
>> +
>> +from tests.package.test_python import TestPythonPackageBase
>> +
>> +
>> +class TestPythonPy3Snappy(TestPythonPackageBase):
>> +    __test__ =3D True
>> +    config =3D """
>> +        BR2_aarch64=3Dy
>> +        BR2_TOOLCHAIN_EXTERNAL=3Dy
>> +        BR2_TARGET_GENERIC_GETTY_PORT=3D"ttyAMA0"
>> +        BR2_LINUX_KERNEL=3Dy
>> +        BR2_LINUX_KERNEL_CUSTOM_VERSION=3Dy
>> +        BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE=3D"6.6.135"
>> +        BR2_LINUX_KERNEL_USE_CUSTOM_CONFIG=3Dy
>> +        BR2_LINUX_KERNEL_CUSTOM_CONFIG_FILE=3D"board/qemu/aarch64-virt/=
>linux.config"
>> +        BR2_PACKAGE_PYTHON3=3Dy
>> +        BR2_PACKAGE_PYTHON_SNAPPY=3Dy
>> +        BR2_TARGET_ROOTFS_CPIO=3Dy
>> +        # BR2_TARGET_ROOTFS_TAR is not set
>> +        """
>
>Ideally we should fix the issue that you run into with our default armv5
>kernel or at least document why we need this. As an alternative the
>standard armv7 kernel could also be used, similar to E.G. test_bat.py
>

Bump to 2.14.0 solved the issue, so the test runs OK also on default armv5 now.

>
>> +    interpreter =3D "python3"
>

It will be removed in v5.

>The default is fine here, so no need to override.
>
>> +    sample_scripts =3D ["tests/package/sample_python_snappy.py"]
>> +    timeout =3D 20
>
>Why?

It will be removed in v5.

Thanks a lot,
Flaviu.

>--=20
>Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot