[PATCH v2 02/28] binman: Drop unused return variable for _DoTestFile()

Simon Glass sjg at chromium.org
Tue Oct 27 00:40:00 CET 2020


This function returns the exit code from binman, not any data. Fix up a
few callers in the tests.

Signed-off-by: Simon Glass <sjg at chromium.org>
---

Changes in v2:
- Add new patch to drop unused return variable for _DoTestFile()

 tools/binman/ftest.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/binman/ftest.py b/tools/binman/ftest.py
index 75f6ca3a895..d23967e603d 100644
--- a/tools/binman/ftest.py
+++ b/tools/binman/ftest.py
@@ -3737,14 +3737,14 @@ class TestFunctional(unittest.TestCase):
     def testEnvironmentNoSize(self):
         """Test that a missing 'size' property is detected"""
         with self.assertRaises(ValueError) as e:
-            data = self._DoTestFile('175_env_no_size.dts')
+            self._DoTestFile('175_env_no_size.dts')
         self.assertIn("'u-boot-env' entry must have a size property",
                       str(e.exception))
 
     def testEnvironmentTooSmall(self):
         """Test handling of an environment that does not fit"""
         with self.assertRaises(ValueError) as e:
-            data = self._DoTestFile('176_env_too_small.dts')
+            self._DoTestFile('176_env_too_small.dts')
 
         # checksum, start byte, environment with \0 terminator, final \0
         need = 4 + 1 + len(ENV_DATA) + 1 + 1
-- 
2.29.0.rc2.309.g374f81d7ae-goog



More information about the U-Boot mailing list