[PATCH] binman: Sort tests and rework test-file numbers

Simon Glass sjg at chromium.org
Sat Aug 27 03:59:21 CEST 2022


On Tue, 23 Aug 2022 at 04:46, Stefan Herbrechtsmeier
<stefan.herbrechtsmeier-oss at weidmueller.com> wrote:
>
> From: Stefan Herbrechtsmeier <stefan.herbrechtsmeier at weidmueller.com>
>
> Tests should be in order of the test-file numbers. Sort the tests
> according to the test-file numbers and rework the test-file numbers to
> eliminate duplicate numbers.
>
> Signed-off-by: Stefan Herbrechtsmeier <stefan.herbrechtsmeier at weidmueller.com>
>
> ---
>
>  tools/binman/ftest.py                         | 113 +++++++++---------
>  .../binman/test/{225_dev.key => 230_dev.key}  |   0
>  .../{225_pre_load.dts => 230_pre_load.dts}    |   2 +-
>  ...re_load_pkcs.dts => 231_pre_load_pkcs.dts} |   2 +-
>  ..._pre_load_pss.dts => 232_pre_load_pss.dts} |   2 +-
>  ...g.dts => 233_pre_load_invalid_padding.dts} |   2 +-
>  ...d_sha.dts => 234_pre_load_invalid_sha.dts} |   2 +-
>  ...algo.dts => 235_pre_load_invalid_algo.dts} |   2 +-
>  ...d_key.dts => 236_pre_load_invalid_key.dts} |   2 +-
>  ..._unique_names.dts => 237_unique_names.dts} |   0
>  ...s_multi.dts => 238_unique_names_multi.dts} |   0
>  ...ntool.dts => 239_replace_with_bintool.dts} |   0
>  ...eplace.dts => 240_fit_extract_replace.dts} |   0
>  ...ple.dts => 241_replace_section_simple.dts} |   0
>  ..._mkimage_name.dts => 242_mkimage_name.dts} |   0
>  ...kimage_image.dts => 243_mkimage_image.dts} |   0
>  ...t.dts => 244_mkimage_image_no_content.dts} |   0
>  ...mage_bad.dts => 245_mkimage_image_bad.dts} |   0
>  ...ion_other.dts => 246_collection_other.dts} |   0
>  ..._mkimage_coll.dts => 247_mkimage_coll.dts} |   0
>  ...s => 248_compress_dtb_prepend_invalid.dts} |   0
>  ...ts => 249_compress_dtb_prepend_length.dts} |   0
>  ...valid.dts => 250_compress_dtb_invalid.dts} |   0
>  ...dtb_zstd.dts => 251_compress_dtb_zstd.dts} |   0
>  24 files changed, 64 insertions(+), 63 deletions(-)
>  rename tools/binman/test/{225_dev.key => 230_dev.key} (100%)
>  rename tools/binman/test/{225_pre_load.dts => 230_pre_load.dts} (86%)
>  rename tools/binman/test/{226_pre_load_pkcs.dts => 231_pre_load_pkcs.dts} (87%)
>  rename tools/binman/test/{227_pre_load_pss.dts => 232_pre_load_pss.dts} (87%)
>  rename tools/binman/test/{228_pre_load_invalid_padding.dts => 233_pre_load_invalid_padding.dts} (86%)
>  rename tools/binman/test/{229_pre_load_invalid_sha.dts => 234_pre_load_invalid_sha.dts} (86%)
>  rename tools/binman/test/{230_pre_load_invalid_algo.dts => 235_pre_load_invalid_algo.dts} (86%)
>  rename tools/binman/test/{231_pre_load_invalid_key.dts => 236_pre_load_invalid_key.dts} (86%)
>  rename tools/binman/test/{230_unique_names.dts => 237_unique_names.dts} (100%)
>  rename tools/binman/test/{231_unique_names_multi.dts => 238_unique_names_multi.dts} (100%)
>  rename tools/binman/test/{232_replace_with_bintool.dts => 239_replace_with_bintool.dts} (100%)
>  rename tools/binman/test/{233_fit_extract_replace.dts => 240_fit_extract_replace.dts} (100%)
>  rename tools/binman/test/{234_replace_section_simple.dts => 241_replace_section_simple.dts} (100%)
>  rename tools/binman/test/{235_mkimage_name.dts => 242_mkimage_name.dts} (100%)
>  rename tools/binman/test/{236_mkimage_image.dts => 243_mkimage_image.dts} (100%)
>  rename tools/binman/test/{237_mkimage_image_no_content.dts => 244_mkimage_image_no_content.dts} (100%)
>  rename tools/binman/test/{238_mkimage_image_bad.dts => 245_mkimage_image_bad.dts} (100%)
>  rename tools/binman/test/{239_collection_other.dts => 246_collection_other.dts} (100%)
>  rename tools/binman/test/{240_mkimage_coll.dts => 247_mkimage_coll.dts} (100%)
>  rename tools/binman/test/{235_compress_dtb_prepend_invalid.dts => 248_compress_dtb_prepend_invalid.dts} (100%)
>  rename tools/binman/test/{236_compress_dtb_prepend_length.dts => 249_compress_dtb_prepend_length.dts} (100%)
>  rename tools/binman/test/{237_compress_dtb_invalid.dts => 250_compress_dtb_invalid.dts} (100%)
>  rename tools/binman/test/{238_compress_dtb_zstd.dts => 251_compress_dtb_zstd.dts} (100%)
>

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

Applied to u-boot-dm, thanks!


More information about the U-Boot mailing list