<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/test/py/tests/test_fs, branch v2021.04</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<id>http://cgit.235523.xyz/u-boot.git/atom/test/py/tests/test_fs?h=v2021.04</id>
<link rel='self' href='http://cgit.235523.xyz/u-boot.git/atom/test/py/tests/test_fs?h=v2021.04'/>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/'/>
<updated>2021-03-31T11:50:17Z</updated>
<entry>
<title>test: Don't unmount not (yet) mounted system</title>
<updated>2021-03-31T11:50:17Z</updated>
<author>
<name>Andy Shevchenko</name>
<email>andriy.shevchenko@linux.intel.com</email>
</author>
<published>2021-02-11T14:40:12Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=1ba21bb06b08d2f5d62afac29549ade8616929ce'/>
<id>urn:sha1:1ba21bb06b08d2f5d62afac29549ade8616929ce</id>
<content type='text'>
When test suite tries to create a file for a new filesystem test case and fails,
the clean up of the exception tries to unmount the image, that has not yet been
mounted. When it happens, the fuse_mounted global variable is set to False and
inconveniently the test case tries to use sudo, so without this change the
admin of the machine gets an (annoying) email:

  Subject: *** SECURITY information for example.com ***

  example.com : Feb  5 19:43:47 : ... COMMAND=/bin/umount .../build-sandbox/persistent-data/mnt

and second run of the test cases on uncleaned build folder will ask for sudo
which is not what expected.

Besides that there is a double unmount calls during successfully run test case.

All of these due to over engineered Python try-except clause and people didn't
get it properly at all. The rule of thumb is that don't use more keywords than
try-except in the exception handling code. Nevertheless, here we adjust code
to be less intrusive to the initial logic behind that complex and unclear
constructions in the test case, although it adds a lot of lines of the code,
i.e. splits one exception handler to three, so on each step we know what
cleanup shall perform.

Signed-off-by: Andy Shevchenko &lt;andriy.shevchenko@linux.intel.com&gt;
Reviewed-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>test: Update test_fs to not use deprecated pytest.yield_fixture()</title>
<updated>2021-01-29T15:36:49Z</updated>
<author>
<name>Tom Rini</name>
<email>trini@konsulko.com</email>
</author>
<published>2021-01-28T19:39:56Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=a59e59c9be844553384b67c36a8040f65ec10011'/>
<id>urn:sha1:a59e59c9be844553384b67c36a8040f65ec10011</id>
<content type='text'>
As noted in comments, yield_fixture has been deprecated for longer than
our minimum required version of pytest.  Newer versions of pytest cause
this to be a louder warning, and as the migration is trivial, perform it
now.

Signed-off-by: Tom Rini &lt;trini@konsulko.com&gt;
</content>
</entry>
<entry>
<title>cmd: fat: Use do_save() for fatwrite</title>
<updated>2020-10-23T17:33:07Z</updated>
<author>
<name>Lad Prabhakar</name>
<email>prabhakar.mahadev-lad.rj@bp.renesas.com</email>
</author>
<published>2020-10-20T07:45:46Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=274227500abc9592f92f5255f6db65e927bc2edd'/>
<id>urn:sha1:274227500abc9592f92f5255f6db65e927bc2edd</id>
<content type='text'>
do_save() function defined in fs.c also supports FAT file system
re-use the same for fatwrite command.

Also fix the FAT test script to match the expected output.

Signed-off-by: Lad Prabhakar &lt;prabhakar.mahadev-lad.rj@bp.renesas.com&gt;
Reviewed-by: Biju Das &lt;biju.das.jz@bp.renesas.com&gt;
</content>
</entry>
<entry>
<title>test/py: Add tests for LZO and ZSTD</title>
<updated>2020-08-24T18:11:31Z</updated>
<author>
<name>Joao Marcos Costa</name>
<email>joaomarcos.costa@bootlin.com</email>
</author>
<published>2020-08-18T15:17:25Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=91f6c1ca2e4400f30b5faee04f1dd9001abd326f'/>
<id>urn:sha1:91f6c1ca2e4400f30b5faee04f1dd9001abd326f</id>
<content type='text'>
Improve SquashFS tests architecture. Add 'Compression' class. LZO
algorithm may crash if the file is fragmented, so the fragments are
disabled when testing LZO.

Signed-off-by: Joao Marcos Costa &lt;joaomarcos.costa@bootlin.com&gt;
</content>
</entry>
<entry>
<title>test/py: fix SquashFS tests</title>
<updated>2020-08-10T14:37:50Z</updated>
<author>
<name>Joao Marcos Costa</name>
<email>joaomarcos.costa@bootlin.com</email>
</author>
<published>2020-08-10T12:37:27Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=74795f1e35b624cdba451aeaea0af2f1211e4a91'/>
<id>urn:sha1:74795f1e35b624cdba451aeaea0af2f1211e4a91</id>
<content type='text'>
Use "cons.config.build_dir" instead of writing to the source directory
(read-only). This will fix the test failures in Azure.

Signed-off-by: Joao Marcos Costa &lt;joaomarcos.costa@bootlin.com&gt;
</content>
</entry>
<entry>
<title>tests: support mkfs.ext4 without metadata_csum</title>
<updated>2020-08-08T02:31:32Z</updated>
<author>
<name>Stephen Warren</name>
<email>swarren@nvidia.com</email>
</author>
<published>2020-08-04T17:28:33Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=cc886253704424f2a332dade72dc1853e78bca04'/>
<id>urn:sha1:cc886253704424f2a332dade72dc1853e78bca04</id>
<content type='text'>
Modify various test/py filesystem creation routines to support systems
that don't implement the metadata_csum ext4 feature.

Signed-off-by: Stephen Warren &lt;swarren@nvidia.com&gt;
</content>
</entry>
<entry>
<title>test/py: Add tests for the SquashFS commands</title>
<updated>2020-08-08T02:31:32Z</updated>
<author>
<name>Joao Marcos Costa</name>
<email>joaomarcos.costa@bootlin.com</email>
</author>
<published>2020-07-30T13:33:52Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=f428e33b6bd7d01dc488534f59f56c4907c76b2f'/>
<id>urn:sha1:f428e33b6bd7d01dc488534f59f56c4907c76b2f</id>
<content type='text'>
Add Python scripts to test 'ls' and 'load' commands. The scripts
generate a SquashFS image and clean the directory after the assertions,
or if an exception is raised.

Signed-off-by: Joao Marcos Costa &lt;joaomarcos.costa@bootlin.com&gt;
</content>
</entry>
<entry>
<title>cmd: fs: Add command to list supported fs types</title>
<updated>2020-07-07T19:36:59Z</updated>
<author>
<name>Niel Fourie</name>
<email>lusus@denx.de</email>
</author>
<published>2020-03-24T15:17:04Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=2280fa56a00a63bbabc2076330367ec2863a474b'/>
<id>urn:sha1:2280fa56a00a63bbabc2076330367ec2863a474b</id>
<content type='text'>
Added command "fstypes" to list supported/included filesystems.

Signed-off-by: Niel Fourie &lt;lusus@denx.de&gt;
Cc: Simon Glass &lt;sjg@chromium.org&gt;
Reviewed-by: Simon Glass &lt;sjg@chromium.org&gt;
[trini: Limit to sandbox]
Signed-off-by: Tom Rini &lt;trini@konsulko.com&gt;
</content>
</entry>
<entry>
<title>test: test_fs error message</title>
<updated>2020-04-27T18:55:29Z</updated>
<author>
<name>Heinrich Schuchardt</name>
<email>xypron.glpk@gmx.de</email>
</author>
<published>2020-04-20T18:48:40Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=d497821ebf302754ff4cd7265e5945c996e8fc34'/>
<id>urn:sha1:d497821ebf302754ff4cd7265e5945c996e8fc34</id>
<content type='text'>
For non-root users mkfs.vfat is not in the search path at least on Debian.
Hence when running 'make tests' a message indicates that file system tests
have been skipped:

SKIPPED [13] test/py/tests/test_fs/conftest.py:340: Setup failed for
filesystem: fat16

This message is not really helpful as the executed program is not
indicated. Provide a more complete message like

SKIPPED [13] test/py/tests/test_fs/conftest.py:340: Setup failed for
filesystem: fat16.
Command 'mkfs.vfat -F 16 build-sandbox/persistent-data/3GB.fat16.img'
returned non-zero exit status 127.

Signed-off-by: Heinrich Schuchardt &lt;xypron.glpk@gmx.de&gt;
</content>
</entry>
<entry>
<title>test/py: test_fs: add tests for creating/deleting many files</title>
<updated>2019-12-05T15:28:38Z</updated>
<author>
<name>AKASHI Takahiro</name>
<email>takahiro.akashi@linaro.org</email>
</author>
<published>2019-11-26T08:28:49Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=995237b04921574c3922734005d1493d5ec8be97'/>
<id>urn:sha1:995237b04921574c3922734005d1493d5ec8be97</id>
<content type='text'>
# This is actually a resent patch of
# [1] https://lists.denx.de/pipermail/u-boot/2019-May/369170.html

Two test cases are added under test_fs_ext:
    test case 10: for root directory
    test case 11: for non-root directory

Those will verify a behavior fixed by the commits related to
root directory
("fs: fat: allocate a new cluster for root directory of fat32" and
"fs: fat: flush a directory cluster properly"), and focus on
handling long-file-name directory entries under a directory.

Signed-off-by: AKASHI Takahiro &lt;takahiro.akashi@linaro.org&gt;
</content>
</entry>
</feed>
