28 Nov, 2017
1 commit
-
When tests pass an error message is printed because of a variable that is
not initialised. Fix this.Signed-off-by: Simon Glass
23 Nov, 2017
1 commit
-
This script runs the tests but does not report failure. Also it always
returns an exit code of 0 even on failure.Fix these problems by checking the result of each test.
Signed-off-by: Simon Glass
17 Nov, 2017
1 commit
-
It can be useful to record how long tests take; this can help debug slow
running test systems or track changes in performance over time. Enhance
the test system to record timestamps while running test:
- Whenever a new log file section is started.
- After U-Boot is started and communication has been established.
- After each host or U-Boot command is executed.Signed-off-by: Stephen Warren
06 Nov, 2017
1 commit
-
test_gpt generates a persistent disk image which can be re-used across
multiple test runs. Currently, if the Python code that generates the disk
image change, the image is not regenerated, which could cause test
failures e.g. if a test was updated to expect some new partition name or
size, yet the persistent disk image contained the old name or size. This
change introduces functionality to regenerate the disk image if the
instructions to generate the image have changed.Signed-off-by: Stephen Warren
24 Oct, 2017
6 commits
-
Signed-off-by: Masahiro Yamada
Reviewed-by: Stephen Warren -
don't use prettyprint_part_size() in create_gpt_partitions_list()
that avoid to align offset and size to 1 MiB and increase precision for
start and size.
This patch avoid the risk to change partition size and lost data during
rename or swap.Signed-off-by: Patrick Delaunay
Acked-by: Stephen Warren
Tested-by: Stephen Warren -
Add test of first and last LBA in gpt for rename and swap.
Only the name is expected to change, so test 3 columns
for part command
1: first LBA (start)
2: last LBA (end)
3: partition nameAfter rename, the last LBA change and it is a error in current U-Boot code
+ "first" = 0x7ff : invalid value ( size increasing !Acked-by: Stephen Warren
Tested-by: Stephen Warren
Signed-off-by: Patrick Delaunay -
+ test write for one partition on all the device (size=0)
+ test write with disk uuid and 2 partitionsAcked-by: Stephen Warren
Tested-by: Stephen Warren
Signed-off-by: Patrick Delaunay -
add sandbox test for some gpt sub-command
- gpt read / part list : read the gpt partition created by sgdisk on host
test start, size, LBA and name output
- gpt verify : verify the gpt partition create by sgdisk on hostPS: persistent data test_gpt_disk_image.bin are udpated
Acked-by: Stephen Warren
Tested-by: Stephen Warren
Signed-off-by: Patrick Delaunay -
copy the persistent gpt binary file as it can be modified during the test
that avoid issue if the test fail: the test always restart with clean fileAcked-by: Stephen Warren
Tested-by: Stephen Warren
Signed-off-by: Patrick Delaunay
17 Oct, 2017
1 commit
-
On qemu errors like
assert 2.999650001525879 >= 3
occur.According to the comment in the code the test is meant to be
approximate. So we should accept some milliseconds less.Cc: Stephen Warren
Signed-off-by: Heinrich Schuchardt
Reviewed-by: Stephen Warren
10 Oct, 2017
2 commits
09 Oct, 2017
1 commit
-
Add a simple test for long strings.
Signed-off-by: Rob Clark
Reviewed-by: Simon Glass
06 Oct, 2017
4 commits
-
Coverity scan has identified potential buffer overruns in these tests.
Correct this by zeroing our buffer and using strncpy not strcpy.Reported-by: Coverity (CID: 155462, 155463)
Cc: Joe Hershberger
Cc: Simon Glass
Cc: Bin Meng
Signed-off-by: Tom Rini
Reviewed-by: Simon Glass
Reviewed-by: Joe Hershberger -
With the overlay tests now being built in sandbox Coverity has found a
number of issues in the tests. In short, if malloc ever failed we would
leak the previous mallocs, so we need to do the usual goto pattern to
free each in turn. Finally, we always looked at the free()d location to
see how many tests had failed for the return code.Reported-by: Coverity (CID: 167224, 167227, 167230, 167236)
Signed-off-by: Tom Rini -
The previous commit fixed a problem in FAT code where going back to the
root directory using '..' wouldn't work correctly on FAT12 or FAT16.
Add a test to exercise this case (which was once fixed in commit
18a10d46f26 "fat: handle paths that include ../" but reintroduced due to
the directory iterator refactoring).This test only very barely catches the problem - without the fix the
size command still gives valid output but the additional spurious
"Invalid FAT entry" error message makes it not get caught in the
'egrep -A3 ' output. I tried to make a proper test that grows the root
directory to two clusters lots of with dummy files but that causes the
write tests to crash the sandbox totally...Signed-off-by: Tuomas Tynkkynen
Reviewed-by: Tom Rini -
Currently we can only test FAT32 which is the default FAT version that
mkfs.vfat creates by default. Instead make it explicitly create either a
FAT16 or a FAT32 volume. This allows us to exercise more code, for
instance the root directory handling is done differently in FAT32 than
the older FATs.Adding FAT12 support is a much bigger job since the test creates a 2.5GB
file and the FAT12 maximum partition size is way smaller than that.Signed-off-by: Tuomas Tynkkynen
Reviewed-by: Tom Rini
02 Oct, 2017
1 commit
01 Oct, 2017
2 commits
-
Now that we have changed to remove all devices under the root hub in
usb_stop(), and corrected the USB emulator select logic, it makes no
sense to do various tests based on 'usb tree' output since the order
of devices is no longer fixed. Remove these USB test cases related
to 'usb tree'.For the USB remove test, ideally we should remove an emulator device
node from the device tree, but this is so far not working. Change to
test the 'usb stop' only.Signed-off-by: Bin Meng
-
Patch queue for efi - 2017-10-01
Lots of new things this time. High level highlights are:
- Shim support (to boot Fedora)
- Initial set of unit tests
- Preparations to support UEFI Shell
30 Sep, 2017
4 commits
-
The current code wraps a pre tag inside an a tag. For some reason, this
causes at least Firefox to attempt to drag the pre section content when
using a mouse drag to select text. Re-order the tags so that the text can
be selected using the mouse, at least if you start the drag outside the
text (after the end of the line, for example).Signed-off-by: Stephen Warren
-
Some tests use external tools (executables) during their operation. Add
a test.py mark to indicate this. This allows those tests to be skipped if
the required tool is not present.Signed-off-by: Stephen Warren
-
When skipping tests, explicitly mention the board type or config option
that caused the skip. This will help people understand/fix any issues.Signed-off-by: Stephen Warren
-
Some tests rely on external tools. Mention these in the test/py README.
Signed-off-by: Stephen Warren
29 Sep, 2017
2 commits
-
This adds tests for clear, set-cursor and color escape sequences.
Signed-off-by: Rob Clark
Signed-off-by: Anatolij Gustschin -
I'll need some more of this, let's not just copy-pasta the
vidconsole_put_char() loop.Named to match vidconsole_put_char() in case that is ever useful
outside of the tests.Signed-off-by: Rob Clark
19 Sep, 2017
1 commit
-
A Python test script is provided that runs the EFI selftest
if CONFIG_CMD_EFI_SELFTEST=y.Signed-off-by: Heinrich Schuchardt
Signed-off-by: Alexander Graf
16 Sep, 2017
2 commits
-
Make various changes to the GPT test:
1) Reference the disk image using an absolute path in all cases. This
allows test/py to operate correctly if it's run from a directory other
than the root of the U-Boot source tree.2) Store the disk image in the teswt/py persistent data directory. This
removes the need to re-generate it every time the tests are run.3) Execute sgdisk using u_boot_utils.run_and_log() so that its output is
captured in the test log. This allows debugging any problems running it.4) Make the disk image a test fixture. This removes the requirement to
always run all GPT tests, and run them in order. The current code doesn't
create the disk image if e.g. just test_gpt_uuid() is executed via the
test.py -k command-line option.5) Use @pytest.mark.buildconfigspec for all feature dependencies, rather
than manually implementing some of them.6) Make all tests depend on sandbox, since they use the sandbox-specific
host command.Fixes: a2f422555fc8 ("add pytests for 'gpt guid' command in sandbox")
Fixes: c5772188ede9 ("add pytests for 'gpt rename' and 'gpt swap'")
Signed-off-by: Stephen Warren
Tested-by: Tom Rini
15 Sep, 2017
6 commits
-
run_command does not have a parameter wait_for_each.
Signed-off-by: Heinrich Schuchardt
-
Verify that stacked overlays work.
Signed-off-by: Pantelis Antoniou
Acked-by: Simon Glass -
fdt_getprop_u32 is not exported and it's different than what the
unit test uses. Rename u32 prop access methods to something that's
unit test specific.Signed-off-by: Pantelis Antoniou
Acked-by: Simon Glass -
Add unit tests for the 'gpt rename' and 'gpt swap' commands that
rely on the block device created by test/py/make_test_disk.py.
Add CONFIG_CMD_GPT_RENAME to the sandbox_defconfig. Remove the
testdisk.raw test device at the end of the tests.Signed-off-by: Alison Chaiken
Reviewed-by: Simon Glass -
Run unit tests for the 'gpt guid' command, making use of the block
device created by test/py/make_test_disk.py. Remove this device at
the end of the tests.Signed-off-by: Alison Chaiken
Reviewed-by: Simon Glass -
Provide a Python function that creates a small block device for the
purpose of testing the cmd/gpt.c or cmd/part.c functions in the u-boot
sandbox.Signed-off-by: Alison Chaiken
Reviewed-by: Simon Glass
12 Sep, 2017
2 commits
-
This allows to scan the DT including all "clocks" node's sub-nodes
in which fixed-clock are defined.
All fixed-clock should be defined inside a clocks node which collect all
external oscillators. Until now, all clocks sub-nodes can't be binded except
if the "simple-bus" compatible string is added which is a hack.Update test.dts by moving clk_fixed node inside clocks.
Signed-off-by: Patrice Chotard
Reviewed-by: Simon Glass
Reviewed-by: Simon Glass -
There is an overflow problem when taking the size instead of the number
of blocks in blk_create_device(). This results in a wrong device size: the
device apparent size is its real size modulo 4GB.
Using the number of blocks instead of the device size fixes the problem and
is more coherent with the internals of the block layer.Signed-off-by: Jean-Jacques Hiblot
Reviewed-by: Simon Glass
16 Aug, 2017
2 commits
-
We are now using an env_ prefix for environment functions. Rename these
two functions for consistency. Also add function comments in common.h.Quite a few places use getenv() in a condition context, provoking a
warning from checkpatch. These are fixed up in this patch also.Suggested-by: Wolfgang Denk
Signed-off-by: Simon Glass -
We are now using an env_ prefix for environment functions. Rename setenv()
for consistency. Also add function comments in common.h.Suggested-by: Wolfgang Denk
Signed-off-by: Simon Glass