With a newer pylint we get a warning about how offset could be used
before assigned. This is because when the underlying filesystem wasn't
one that is supported we would have runtime test failures. Address this
by raise'ing an Exception if fs is not supported.

Signed-off-by: Tom Rini <tr...@konsulko.com>
---
Cc: Love Kumar <love.ku...@amd.com>
Cc: Andrew Goodbody <andrew.goodb...@linaro.org>
---
 test/py/tests/test_usb.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/test/py/tests/test_usb.py b/test/py/tests/test_usb.py
index 566d73b7c647..9bef883325f0 100644
--- a/test/py/tests/test_usb.py
+++ b/test/py/tests/test_usb.py
@@ -580,6 +580,8 @@ def test_usb_load(u_boot_console):
                     elif fs in ['ext4', 'ext2']:
                         file, size, expected_crc32 = \
                             usb_ext4load_ext4write(u_boot_console, fs, x, part)
+                    else:
+                        raise Exception('Unsupported filesystem type %s' % fs)
 
                     offset = random.randrange(128, 1024, 128)
                     output = u_boot_console.run_command(
-- 
2.43.0

Reply via email to