Message ID | 20171113045231.15911-5-sjg@chromium.org |
---|---|
State | Accepted |
Delegated to: | Simon Glass |
Headers | show |
Series | test: Include Python tools in test coverage | expand |
At present these tests use the same filename as patman. This adds
confusion when running all tests, since error messages look very similar.
In fact binman tries to run the wrong tests at present.
Rename the tests.
Signed-off-by: Simon Glass <sjg@chromium.org>
---
tools/binman/binman.py | 4 ++--
tools/binman/{func_test.py => ftest.py} | 0
2 files changed, 2 insertions(+), 2 deletions(-)
rename tools/binman/{func_test.py => ftest.py} (100%)
Applied to u-boot-dm thanks!
diff --git a/tools/binman/binman.py b/tools/binman/binman.py index e75a59d9517..d264bcdfa8c 100755 --- a/tools/binman/binman.py +++ b/tools/binman/binman.py @@ -34,7 +34,7 @@ def RunTests(): """Run the functional tests and any embedded doctests""" import entry_test import fdt_test - import func_test + import ftest import test import doctest @@ -44,7 +44,7 @@ def RunTests(): suite.run(result) sys.argv = [sys.argv[0]] - for module in (func_test.TestFunctional, fdt_test.TestFdt, + for module in (ftest.TestFunctional, fdt_test.TestFdt, entry_test.TestEntry): suite = unittest.TestLoader().loadTestsFromTestCase(module) suite.run(result) diff --git a/tools/binman/func_test.py b/tools/binman/ftest.py similarity index 100% rename from tools/binman/func_test.py rename to tools/binman/ftest.py
At present these tests use the same filename as patman. This adds confusion when running all tests, since error messages look very similar. In fact binman tries to run the wrong tests at present. Rename the tests. Signed-off-by: Simon Glass <sjg@chromium.org> --- tools/binman/binman.py | 4 ++-- tools/binman/{func_test.py => ftest.py} | 0 2 files changed, 2 insertions(+), 2 deletions(-) rename tools/binman/{func_test.py => ftest.py} (100%)