All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3 0/3] test/py: Rewrite SquashFS commands test suite
@ 2021-06-30 22:45 Joao Marcos Costa
  2021-06-30 22:45 ` [PATCH v3 1/3] test/py: rewrite common tools for SquashFS tests Joao Marcos Costa
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: Joao Marcos Costa @ 2021-06-30 22:45 UTC (permalink / raw)
  To: u-boot
  Cc: thomas.petazzoni, miquel.raynal, richard.genoud, sjg, Joao Marcos Costa

Hello,

This patch series fixes the following issues:
- poor strategy to check if files were properly loaded
- wrong quoting style for strings
- tests failing at the second run because of a wrong clean-up strategy

Finally, it improves:
- code overall documentation level, with more comments and better
  naming for functions and variables
- code readability by adding more helper functions
- completeness: more test cases were added for both sqfsls and sqfsload
  commands

The sqfsload new test suite may fail when testing images with fragmented
files if the patch I previously sent (fs/squashfs: fix reading of
fragmented files) is not applied, so this patch series depends on it.

Changes since V2:
- Add check_mksquashfs_version function to raise an exception if
  mksquashfs version is too old. I chose to set the required version as
  4.4, since it seems to be the most recent one.

Changes since V1:
- Leave the copyright year as it was (2020) instead of changing it to
  2021
- Remove spurious comments and print statements
- Fix the style issues pointed by pylint3

Best regards,
Joao

Joao Marcos Costa (3):
  test/py: rewrite common tools for SquashFS tests
  test/py: rewrite sqfsload command test suite
  test/py: rewrite sqfsls command test suite

 .../test_fs/test_squashfs/sqfs_common.py      | 257 +++++++++++++-----
 .../test_fs/test_squashfs/test_sqfs_load.py   | 168 ++++++++++--
 .../test_fs/test_squashfs/test_sqfs_ls.py     | 140 ++++++++--
 3 files changed, 452 insertions(+), 113 deletions(-)

-- 
2.25.1


^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2021-07-06  1:00 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-30 22:45 [PATCH v3 0/3] test/py: Rewrite SquashFS commands test suite Joao Marcos Costa
2021-06-30 22:45 ` [PATCH v3 1/3] test/py: rewrite common tools for SquashFS tests Joao Marcos Costa
2021-07-05 15:29   ` Simon Glass
2021-07-06  0:59   ` Tom Rini
2021-06-30 22:45 ` [PATCH v3 2/3] test/py: rewrite sqfsload command test suite Joao Marcos Costa
2021-07-05 15:29   ` Simon Glass
2021-07-06  0:59   ` Tom Rini
2021-06-30 22:45 ` [PATCH v3 3/3] test/py: rewrite sqfsls " Joao Marcos Costa
2021-07-05 15:29   ` Simon Glass
2021-07-06  0:59   ` Tom Rini

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.