diff options
| author | Adrian Freihofer <adrian.freihofer@siemens.com> | 2025-10-19 18:44:11 +0200 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2025-10-27 11:37:43 +0000 |
| commit | 49e2a19a4b157d999812a907aeb5ab0b0847c831 (patch) | |
| tree | 1ec9b467a9a82f92d54ce4bee246c6c66e4cb72a | |
| parent | d950cfad3142e97aa7a430818d8205fe0e2571d6 (diff) | |
| download | poky-49e2a19a4b157d999812a907aeb5ab0b0847c831.tar.gz | |
oe-selftest: fitimage: fix req_its_fields last field_index
The last enrty of req_its_fields was not taken into account and the test
passed even if the last entry was not found in the its file.
(From OE-Core rev: 34e872ab23067231fb93c3b31ad5a439e9c17cb8)
Signed-off-by: Adrian Freihofer <adrian.freihofer@siemens.com>
Signed-off-by: Mathieu Dubois-Briand <mathieu.dubois-briand@bootlin.com>
Signed-off-by: Ross Burton <ross.burton@arm.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rw-r--r-- | meta/lib/oeqa/selftest/cases/fitimage.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/meta/lib/oeqa/selftest/cases/fitimage.py b/meta/lib/oeqa/selftest/cases/fitimage.py index 195b9ee8b5..9c2e10dd2b 100644 --- a/meta/lib/oeqa/selftest/cases/fitimage.py +++ b/meta/lib/oeqa/selftest/cases/fitimage.py | |||
| @@ -297,14 +297,16 @@ class FitImageTestCase(OESelftestTestCase): | |||
| 297 | if req_its_fields: | 297 | if req_its_fields: |
| 298 | field_index = 0 | 298 | field_index = 0 |
| 299 | field_index_last = len(req_its_fields) - 1 | 299 | field_index_last = len(req_its_fields) - 1 |
| 300 | found_all = False | ||
| 300 | with open(its_file_path) as its_file: | 301 | with open(its_file_path) as its_file: |
| 301 | for line in its_file: | 302 | for line in its_file: |
| 302 | if req_its_fields[field_index] in line: | 303 | if req_its_fields[field_index] in line: |
| 303 | if field_index < field_index_last: | 304 | if field_index < field_index_last: |
| 304 | field_index +=1 | 305 | field_index += 1 |
| 305 | else: | 306 | else: |
| 307 | found_all = True | ||
| 306 | break | 308 | break |
| 307 | self.assertEqual(field_index, field_index_last, | 309 | self.assertTrue(found_all, |
| 308 | "Fields in Image Tree Source File %s did not match, error in finding %s" | 310 | "Fields in Image Tree Source File %s did not match, error in finding %s" |
| 309 | % (its_file_path, req_its_fields[field_index])) | 311 | % (its_file_path, req_its_fields[field_index])) |
| 310 | 312 | ||
