mirror of
https://github.com/dgibson/dtc.git
synced 2025-12-07 13:45:07 -05:00
dtc: Remove some redundant testcases
This patch removes a number of testcases from the testsuite that are extremely unlikely to find any bugs that won't be found by the other tests. This speeds up the testsuite. - Both loops across the various tree block layouts run the tree1_tests on the basic mangled tree. This is completely redundant, so remove the second copy. This removes 456 testcases. - We currently run tree1_tests on various trees manipulated by move_and_save. We replace those with just a dtbs_equal_ordered test to check that the manipulated tree is equal to the original. What we're testing here is that fdt_move() operates correctly - it's very unlikely it would succeed well enough for the ordered_equal test to succeed, but the tree1_tests would fail on the result. This removes 162 testcases. - Currently we re-ordered with mangle-layout both the basic test_tree1.dtb and sw_tree1.test.dtb. Since we've already checked that these dtbs are equivalent with dtbs_ordered_equal, it's very unlikely that the tests would fail on one but not the other. Therefore reduce this to only using test_tree1.dtb. This removes 828 testcases. Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
This commit is contained in:
parent
8eaf5e3583
commit
3ce5363387
1 changed files with 5 additions and 6 deletions
|
|
@ -76,13 +76,13 @@ libfdt_tests () {
|
||||||
for tree in test_tree1.dtb sw_tree1.test.dtb unfinished_tree1.test.dtb; do
|
for tree in test_tree1.dtb sw_tree1.test.dtb unfinished_tree1.test.dtb; do
|
||||||
rm -f moved.$tree shunted.$tree deshunted.$tree
|
rm -f moved.$tree shunted.$tree deshunted.$tree
|
||||||
run_test move_and_save $tree
|
run_test move_and_save $tree
|
||||||
tree1_tests moved.$tree
|
run_test dtbs_equal_ordered $tree moved.$tree
|
||||||
tree1_tests shunted.$tree
|
run_test dtbs_equal_ordered $tree shunted.$tree
|
||||||
tree1_tests deshunted.$tree
|
run_test dtbs_equal_ordered $tree deshunted.$tree
|
||||||
done
|
done
|
||||||
|
|
||||||
# v16 and alternate layout tests
|
# v16 and alternate layout tests
|
||||||
for tree in test_tree1.dtb sw_tree1.test.dtb; do
|
for tree in test_tree1.dtb; do
|
||||||
for version in 17 16; do
|
for version in 17 16; do
|
||||||
for layout in $ALL_LAYOUTS; do
|
for layout in $ALL_LAYOUTS; do
|
||||||
run_test mangle-layout $tree $version $layout
|
run_test mangle-layout $tree $version $layout
|
||||||
|
|
@ -93,13 +93,12 @@ libfdt_tests () {
|
||||||
done
|
done
|
||||||
|
|
||||||
# Read-write tests
|
# Read-write tests
|
||||||
for basetree in test_tree1.dtb sw_tree1.test.dtb; do
|
for basetree in test_tree1.dtb; do
|
||||||
for version in 17 16; do
|
for version in 17 16; do
|
||||||
for layout in $ALL_LAYOUTS; do
|
for layout in $ALL_LAYOUTS; do
|
||||||
tree=v$version.$layout.$basetree
|
tree=v$version.$layout.$basetree
|
||||||
rm -f opened.$tree repacked.$tree
|
rm -f opened.$tree repacked.$tree
|
||||||
run_test open_pack $tree
|
run_test open_pack $tree
|
||||||
tree1_tests $tree
|
|
||||||
tree1_tests opened.$tree
|
tree1_tests opened.$tree
|
||||||
tree1_tests repacked.$tree
|
tree1_tests repacked.$tree
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue